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

Krzysztof Kozlowski
2020-01-15T19:57:12Z

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@kernel.org>
>
> Wrong commit message. buf_len is size_t and not an iomem pointer.

I'll fix it up.

Best regards,
Krzysztof