Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754517AbdFWSDP (ORCPT ); Fri, 23 Jun 2017 14:03:15 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:48413 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752878AbdFWSDN (ORCPT ); Fri, 23 Jun 2017 14:03:13 -0400 Subject: Re: [PATCH][mtd-next] mtd: parser: print hex size_t value using correct %zx printk format specifier To: Brian Norris , =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= References: <20170623090023.9482-1-colin.king@canonical.com> <20170623175124.GI14148@google.com> Cc: David Woodhouse , Boris Brezillon , Marek Vasut , Richard Weinberger , Cyrille Pitchen , linux-mtd@lists.infradead.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org From: Colin Ian King Message-ID: Date: Fri, 23 Jun 2017 19:03:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <20170623175124.GI14148@google.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1363 Lines: 48 On 23/06/17 18:51, Brian Norris wrote: > 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 >>> >>> 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 :) > > 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. FYI, I used %zx rather than %zX as couldn't find any instances of it in the kernel and I wasn't 100% sure if it was supported or not. linux-next: $ git grep "%zx" | wc -l 161 $ git grep "%zX" | wc -l 0 Colin > > Regards, > Brian > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >