From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH] i2c: i2c-st: Use %pa to print 'resource_size_t' type Date: Fri, 11 Jul 2014 09:16:35 +0200 Message-ID: <53BF8F53.1040701@st.com> References: <1405034605-21757-1-git-send-email-festevam@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Fabio Estevam , wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Fabio Estevam List-Id: linux-i2c@vger.kernel.org Hi Fabio, On 07/11/2014 01:23 AM, Fabio Estevam wrote: > From: Fabio Estevam > > When building multi_v7_defconfig with CONFIG_ARM_LPAE=y the following warning > is seen: > > drivers/i2c/busses/i2c-st.c:818:2: warning: format '%x' expects argument of type > 'unsigned int', but argument 4 has type 'resource_size_t' [-Wformat=] > > Use %pa to print 'resource_size_t' type to fix the warning. > > Reported-by: Olof's autobuilder > Signed-off-by: Fabio Estevam > --- > drivers/i2c/busses/i2c-st.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) You can add my: Acked-by: Maxime Coquelin Thanks! Maxime > > diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c > index 95b94767..fc767dd 100644 > --- a/drivers/i2c/busses/i2c-st.c > +++ b/drivers/i2c/busses/i2c-st.c > @@ -815,7 +815,7 @@ static int st_i2c_probe(struct platform_device *pdev) > > adap = &i2c_dev->adap; > i2c_set_adapdata(adap, i2c_dev); > - snprintf(adap->name, sizeof(adap->name), "ST I2C(0x%x)", res->start); > + snprintf(adap->name, sizeof(adap->name), "ST I2C(0x%pa)", &res->start); > adap->owner = THIS_MODULE; > adap->timeout = 2 * HZ; > adap->retries = 0; >