Re: [PATCH -next] i2c: fix i2c-ismt.c printk format warning

Re: [PATCH -next] i2c: fix i2c-ismt.c printk format warning

[Neil Horman]

>Fix printk format warning.  dma_addr_t can be 32-bit or 64-bit,
>so cast it to long long for printing.  This also matches the
>printk format specifier that is already used.
>
>drivers/i2c/busses/i2c-ismt.c:532:3: warning: format '%llX' expects argument of type \
>'long long unsigned int', but argument 4 has type 'dma_addr_t' [-Wformat]
>
>Signed-off-by: Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
>Cc:	Bill E. Brown <bill.e.brown-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>Cc:	Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>
>Cc:	Seth Heasley <seth.heasley-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>Cc:	Neil Horman <nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
>Cc:	Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
>Cc:	linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

Acked-by: Neil Horman <nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>

Re: [PATCH -next] i2c: fix i2c-ismt.c printk format warning

[Wolfram Sang]

On Fri, Feb 15, 2013 at 10:51:40AM -0800, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
> 
> Fix printk format warning.  dma_addr_t can be 32-bit or 64-bit,
> so cast it to long long for printing.  This also matches the
> printk format specifier that is already used.
> 
> drivers/i2c/busses/i2c-ismt.c:532:3: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' [-Wformat]
> 
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>

Thanks, applied to next.

[PATCH -next] i2c: fix i2c-ismt.c printk format warning

[Randy Dunlap]

From: Randy Dunlap <rdunlap@infradead.org>

Fix printk format warning.  dma_addr_t can be 32-bit or 64-bit,
so cast it to long long for printing.  This also matches the
printk format specifier that is already used.

drivers/i2c/busses/i2c-ismt.c:532:3: warning: format '%llX' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' [-Wformat]

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc:	Bill E. Brown <bill.e.brown@intel.com>
Cc:	Jean Delvare <khali@linux-fr.org>
Cc:	Seth Heasley <seth.heasley@intel.com>
Cc:	Neil Horman <nhorman@tuxdriver.com>
Cc:	Wolfram Sang <w.sang@pengutronix.de>
Cc:	linux-i2c@vger.kernel.org
---
 drivers/i2c/busses/i2c-ismt.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-next-20130215.orig/drivers/i2c/busses/i2c-ismt.c
+++ linux-next-20130215/drivers/i2c/busses/i2c-ismt.c
@@ -530,7 +530,7 @@ static int ismt_access(struct i2c_adapte
 		}
 
 		dev_dbg(dev, " dma_addr = 0x%016llX\n",
-			dma_addr);
+			(unsigned long long)dma_addr);
 
 		desc->dptr_low = lower_32_bits(dma_addr);
 		desc->dptr_high = upper_32_bits(dma_addr);