Re: [PATCH v2 3/4] i2c: highlander: Use proper printk format for iomem pointer

From: Krzysztof Kozlowski
Date: Wed Jan 15 2020 - 14:57:12 EST


On Wed, Jan 15, 2020 at 06:43:37PM +0100, Wolfram Sang wrote:
> On Tue, Jan 14, 2020 at 10:06:04AM +0100, Krzysztof Kozlowski wrote:
> > iomem pointers should be printed with pointer format to hide the
> > actual value and fix warnings when compiling on 64-bit platform (e.g. with
> > COMPILE_TEST):
> >
> > drivers/i2c/busses/i2c-highlander.c: In function âhighlander_i2c_smbus_xferâ:
> > drivers/i2c/busses/i2c-highlander.c:325:22: warning:
> > format â%dâ expects argument of type âintâ,
> > but argument 3 has type âsize_t {aka long unsigned int}â [-Wformat=]
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
>
> Wrong commit message. buf_len is size_t and not an iomem pointer.

I'll fix it up.

Best regards,
Krzysztof