* [PATCH] i2c: i2c-st: Use %pa to print 'resource_size_t' type
@ 2014-07-10 23:23 Fabio Estevam
[not found] ` <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Fabio Estevam @ 2014-07-10 23:23 UTC (permalink / raw)
To: wsa-z923LK4zBo2bacvFa/9K2g
Cc: maxime.coquelin-qxv4g6HH51o, linux-i2c-u79uwXL29TY76Z2rM5mHXA,
Fabio Estevam
From: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
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 <build-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>
Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
---
drivers/i2c/busses/i2c-st.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
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;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 3+ messages in thread[parent not found: <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH] i2c: i2c-st: Use %pa to print 'resource_size_t' type [not found] ` <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> @ 2014-07-11 7:16 ` Maxime Coquelin 2014-07-17 13:32 ` Wolfram Sang 1 sibling, 0 replies; 3+ messages in thread From: Maxime Coquelin @ 2014-07-11 7:16 UTC (permalink / raw) To: Fabio Estevam, wsa-z923LK4zBo2bacvFa/9K2g Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, Fabio Estevam Hi Fabio, On 07/11/2014 01:23 AM, Fabio Estevam wrote: > From: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > > 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 <build-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> > Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > --- > drivers/i2c/busses/i2c-st.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) You can add my: Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org> 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; > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] i2c: i2c-st: Use %pa to print 'resource_size_t' type [not found] ` <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2014-07-11 7:16 ` Maxime Coquelin @ 2014-07-17 13:32 ` Wolfram Sang 1 sibling, 0 replies; 3+ messages in thread From: Wolfram Sang @ 2014-07-17 13:32 UTC (permalink / raw) To: Fabio Estevam Cc: maxime.coquelin-qxv4g6HH51o, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Fabio Estevam [-- Attachment #1: Type: text/plain, Size: 680 bytes --] On Thu, Jul 10, 2014 at 08:23:25PM -0300, Fabio Estevam wrote: > From: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > > 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 <build-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> > Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Applied to for-current, thanks! [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-07-17 13:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-10 23:23 [PATCH] i2c: i2c-st: Use %pa to print 'resource_size_t' type Fabio Estevam
[not found] ` <1405034605-21757-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-07-11 7:16 ` Maxime Coquelin
2014-07-17 13:32 ` Wolfram Sang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).