From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Date: Fri, 23 Jun 2017 10:02:34 +0000 Subject: Re: [PATCH][mtd-next] mtd: parser: print hex size_t value using correct %zx printk format specifier Message-Id: List-Id: References: <20170623090023.9482-1-colin.king@canonical.com> In-Reply-To: <20170623090023.9482-1-colin.king@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Colin King Cc: David Woodhouse , Brian Norris , Boris Brezillon , Marek Vasut , Richard Weinberger , Cyrille Pitchen , linux-mtd@lists.infradead.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org On 2017-06-23 11:00, Colin King wrote: > From: Colin Ian King > > 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 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 :)