Patchwork [U-Boot] usb: musb: fix printf warning

login
register
mail settings
Submitter Mike Frysinger
Date Feb. 3, 2012, 1:03 p.m.
Message ID <1328274184-23581-1-git-send-email-vapier@gentoo.org>
Download mbox | patch
Permalink /patch/139366/
State Accepted
Commit b9743081058d847a9150dc8bd9dd2b74d78105a2
Delegated to: Marek Vasut
Headers show

Comments

Mike Frysinger - Feb. 3, 2012, 1:03 p.m.
musb_hcd.c: In function 'musb_submit_rh_msg':
musb_hcd.c:827:2: warning: format '%d' expects type 'int',
	but argument 3 has type 'long unsigned int'

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
 drivers/usb/musb/musb_hcd.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Marek Vasut - Feb. 26, 2012, 11:13 p.m.
> musb_hcd.c: In function 'musb_submit_rh_msg':
> musb_hcd.c:827:2: warning: format '%d' expects type 'int',
> 	but argument 3 has type 'long unsigned int'
> 
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
>  drivers/usb/musb/musb_hcd.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 

Hi,

what's the status of this patch/patchset?

Thanks
M
Mike Frysinger - Feb. 26, 2012, 11:38 p.m.
On Sunday 26 February 2012 18:13:58 Marek Vasut wrote:
> > musb_hcd.c: In function 'musb_submit_rh_msg':
> > musb_hcd.c:827:2: warning: format '%d' expects type 'int',
> > 
> > 	but argument 3 has type 'long unsigned int'
> 
> what's the status of this patch/patchset?

the usb ones are waiting for someone to pick up
-mike
Marek Vasut - Feb. 26, 2012, 11:48 p.m.
> On Sunday 26 February 2012 18:13:58 Marek Vasut wrote:
> > > musb_hcd.c: In function 'musb_submit_rh_msg':
> > > musb_hcd.c:827:2: warning: format '%d' expects type 'int',
> > > 
> > > 	but argument 3 has type 'long unsigned int'
> > 
> > what's the status of this patch/patchset?
> 
> the usb ones are waiting for someone to pick up
> -mike

Ok, can you please resend and Cc me so I can pick them up?

M
Mike Frysinger - Feb. 27, 2012, 12:24 a.m.
On Sunday 26 February 2012 18:48:26 Marek Vasut wrote:
> > On Sunday 26 February 2012 18:13:58 Marek Vasut wrote:
> > > > musb_hcd.c: In function 'musb_submit_rh_msg':
> > > > musb_hcd.c:827:2: warning: format '%d' expects type 'int',
> > > > 
> > > > 	but argument 3 has type 'long unsigned int'
> > > 
> > > what's the status of this patch/patchset?
> > 
> > the usb ones are waiting for someone to pick up
> 
> Ok, can you please resend and Cc me so I can pick them up?

http://patchwork.ozlabs.org/patch/139366/
-mike
Marek Vasut - Feb. 27, 2012, 1:17 p.m.
> On Sunday 26 February 2012 18:48:26 Marek Vasut wrote:
> > > On Sunday 26 February 2012 18:13:58 Marek Vasut wrote:
> > > > > musb_hcd.c: In function 'musb_submit_rh_msg':
> > > > > musb_hcd.c:827:2: warning: format '%d' expects type 'int',
> > > > > 
> > > > > 	but argument 3 has type 'long unsigned int'
> > > > 
> > > > what's the status of this patch/patchset?
> > > 
> > > the usb ones are waiting for someone to pick up
> > 
> > Ok, can you please resend and Cc me so I can pick them up?
> 
> http://patchwork.ozlabs.org/patch/139366/
> -mike

Assigned, thanks!

Patch

diff --git a/drivers/usb/musb/musb_hcd.c b/drivers/usb/musb/musb_hcd.c
index 325edb9..b6b70ec 100644
--- a/drivers/usb/musb/musb_hcd.c
+++ b/drivers/usb/musb/musb_hcd.c
@@ -824,7 +824,7 @@  static int musb_submit_rh_msg(struct usb_device *dev, unsigned long pipe,
 
 	dev->act_len = len;
 	dev->status = stat;
-	debug("dev act_len %d, status %d\n", dev->act_len, dev->status);
+	debug("dev act_len %d, status %lu\n", dev->act_len, dev->status);
 
 	return stat;
 }