Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
IronPort-SDR: 9aUIAjbMAKuXcmYiAVO6QX6QpgIywxQT5EtjyHyyz1o2vK7poSCeWoeXaKP9hnHMHTsCf4S114
 cWa7gqmAD7EJB47SvUmPUVxhSo4V9m0vJ6v8LXH1f0vrFnLpPJw/qILkUnSkdyfovZVpkTZvMe
 C7CSIhWzlNnkgjTMQMpoajpm3jE9w9AIkpz5YOzwRTm2zjVfku8EL+X6WrXX2wqWzk33QV0SwL
 T2vBpg00Th6QETjRqhFyjM7OoKz8ypV+tNNNjji6yAfLedDTDCu+7J4p0CXnSYO9TEe9IvsFPN
 D02xcpk0B8X+fwVkFCLSgAYB
IronPort-SDR: dUrTZSVA705o5iadGNlWYkloisIML9nddoqltHJ7r1mc+Nd/pDC+CtILUp2W8cmdGcdW8kvukd
 jOTG8Y/qJKQ8aEEmtpqve0hIdHDL1k3N5GGDlAnU2r8DrLRFykEpuEMN3YhI6fhyPy0zolXsPF
 EI861RHu/FKmXVZbCfntcIWxbpMWtwl5CV6HGGQlZtk9sGrP7+F01eYDDGheTC1k+iJiLYjHvn
 c5tpYIX9ag/1Oy87pjl3Wj2jMMYiaU4wQmzdg/AWz29DL22ZdbK4e7jFJK2meV7p8v6yrp/Dq6
 ZD8=
WDCIronportException: Internal
Message-ID: <0e293be4-ba99-379e-75dc-0ba5cf7793b7@opensource.wdc.com>
Date:   Mon, 20 Jun 2022 09:43:13 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: linux-next: build warning after merge of the libata tree
Content-Language: en-US
To:     Stephen Rothwell <sfr@canb.auug.org.au>
Cc:     Sergey Shtylyov <s.shtylyov@omp.ru>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
References: <20220620103400.6b03f8d9@canb.auug.org.au>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <20220620103400.6b03f8d9@canb.auug.org.au>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 6/20/22 09:34, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the libata tree, today's linux-next build (powercp
> ppc64_defconfig) produced this warning:

Will fix that.

> 
> In file included from include/linux/device.h:15,
>                  from arch/powerpc/include/asm/io.h:27,
>                  from include/linux/io.h:13,
>                  from include/linux/irq.h:20,
>                  from arch/powerpc/include/asm/hardirq.h:6,
>                  from include/linux/hardirq.h:11,
>                  from include/linux/highmem.h:11,
>                  from include/linux/bvec.h:10,
>                  from include/linux/blk_types.h:10,
>                  from include/linux/blkdev.h:9,
>                  from drivers/ata/pata_macio.c:18:
> drivers/ata/pata_macio.c: In function 'pmac_macio_calc_timing_masks':
> drivers/ata/pata_macio.c:1031:28: warning: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:129:41: note: in definition of macro 'dev_printk'
>   129 |                 _dev_printk(level, dev, fmt, ##__VA_ARGS__);            \
>       |                                         ^~~
> include/linux/dev_printk.h:163:45: note: in expansion of macro 'dev_fmt'
>   163 |                 dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
>       |                                             ^~~~~~~
> drivers/ata/pata_macio.c:1031:9: note: in expansion of macro 'dev_dbg'
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |         ^~~~~~~
> drivers/ata/pata_macio.c:1031:52: note: format string is defined here
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                                                  ~~^
>       |                                                    |
>       |                                                    long unsigned int
>       |                                                  %x
> In file included from include/linux/device.h:15,
>                  from arch/powerpc/include/asm/io.h:27,
>                  from include/linux/io.h:13,
>                  from include/linux/irq.h:20,
>                  from arch/powerpc/include/asm/hardirq.h:6,
>                  from include/linux/hardirq.h:11,
>                  from include/linux/highmem.h:11,
>                  from include/linux/bvec.h:10,
>                  from include/linux/blk_types.h:10,
>                  from include/linux/blkdev.h:9,
>                  from drivers/ata/pata_macio.c:18:
> drivers/ata/pata_macio.c:1031:28: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'unsigned int' [-Wformat=]
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:129:41: note: in definition of macro 'dev_printk'
>   129 |                 _dev_printk(level, dev, fmt, ##__VA_ARGS__);            \
>       |                                         ^~~
> include/linux/dev_printk.h:163:45: note: in expansion of macro 'dev_fmt'
>   163 |                 dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
>       |                                             ^~~~~~~
> drivers/ata/pata_macio.c:1031:9: note: in expansion of macro 'dev_dbg'
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |         ^~~~~~~
> drivers/ata/pata_macio.c:1031:63: note: format string is defined here
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                                                             ~~^
>       |                                                               |
>       |                                                               long unsigned int
>       |                                                             %x
> In file included from include/linux/device.h:15,
>                  from arch/powerpc/include/asm/io.h:27,
>                  from include/linux/io.h:13,
>                  from include/linux/irq.h:20,
>                  from arch/powerpc/include/asm/hardirq.h:6,
>                  from include/linux/hardirq.h:11,
>                  from include/linux/highmem.h:11,
>                  from include/linux/bvec.h:10,
>                  from include/linux/blk_types.h:10,
>                  from include/linux/blkdev.h:9,
>                  from drivers/ata/pata_macio.c:18:
> drivers/ata/pata_macio.c:1031:28: warning: format '%lx' expects argument of type 'long unsigned int', but argument 6 has type 'unsigned int' [-Wformat=]
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:129:41: note: in definition of macro 'dev_printk'
>   129 |                 _dev_printk(level, dev, fmt, ##__VA_ARGS__);            \
>       |                                         ^~~
> include/linux/dev_printk.h:163:45: note: in expansion of macro 'dev_fmt'
>   163 |                 dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
>       |                                             ^~~~~~~
> drivers/ata/pata_macio.c:1031:9: note: in expansion of macro 'dev_dbg'
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |         ^~~~~~~
> drivers/ata/pata_macio.c:1031:73: note: format string is defined here
>  1031 |         dev_dbg(priv->dev, "Supported masks: PIO=%lx, MWDMA=%lx, UDMA=%lx\n",
>       |                                                                       ~~^
>       |                                                                         |
>       |                                                                         long unsigned int
>       |                                                                       %x
> 
> Introduced by commit
> 
>   f0a6d77b351c ("ata: make transfer mode masks *unsigned int*")
> 


-- 
Damien Le Moal
Western Digital Research