Re: [PATCH][mtd-next] mtd: parser: print hex size_t value using correct %zx printk format specifier

From: Brian Norris
Date: Fri Jun 23 2017 - 13:51:52 EST


On Fri, Jun 23, 2017 at 12:02:34PM +0200, RafaÅ MiÅecki wrote:
> On 2017-06-23 11:00, Colin King wrote:
> >From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
> >
> >Use %zx instead of %X for size_t variable offset, fixes build warning:
> >
> >warning: format '%X' expects argument of type 'unsigned int', but
> >argument
> >2 has type 'size_t {aka long unsigned int}' [-Wformat=]
> >
> >Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> I sent similar patch few hours earlier:
> [PATCH] mtd: parsers: trx: fix pr_err format for printing offset
> https://patchwork.ozlabs.org/patch/779789/
>
> Brian: you may pick the one with nicer commit message, whichever one you
> prefer :)

I'll go with:
(a) the earlier one and
(b) the one that doesn't change 'X' to 'x'

That means RafaÅ, you're our lucky winner today! Thanks for playing,
Colin.

Regards,
Brian