lkml.org 
[lkml]   [2010]   [Nov]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] USB: ehci/mxc: compile fix
On Thu, Nov 04, 2010 at 11:28:18AM +0100, Uwe Kleine-K?nig wrote:
> Hi Greg,
>
> On Tue, Nov 02, 2010 at 07:05:27AM -0700, Nguyen Dinh-R00091 wrote:
> > Hi Uwe,
> >
> > > -----Original Message-----
> > > From: Uwe Kleine-K?nig [mailto:u.kleine-koenig@pengutronix.de]
> > > Sent: Tuesday, November 02, 2010 4:31 AM
> > > To: Greg KH
> > > Cc: kernel@pengutronix.de; Sergei Shtylyov; matthieu.castet@parrot.com;
> > > linux-kernel@vger.kernel.org; linux-usb@vger.kernel.org; Nguyen Dinh-
> > > R00091; Nguyen Dinh-R00091; Alan Stern
> > > Subject: [PATCH] USB: ehci/mxc: compile fix
> > >
> > > Commit
> > > 65fd427 (USB: ehci tdi : let's tdi_reset set host mode)
> > >
> > > broke the build using ARM's mx51_defconfig:
> > >
> > > CC drivers/usb/host/ehci-hcd.o
> > > In file included from drivers/usb/host/ehci-hcd.c:1166:
> > > drivers/usb/host/ehci-mxc.c: In function 'ehci_mxc_drv_probe':
> > > drivers/usb/host/ehci-mxc.c:192: error: 'ehci' undeclared (first
> > > use in this function)
> > > drivers/usb/host/ehci-mxc.c:192: error: (Each undeclared identifier
> > > is reported only once
> > > drivers/usb/host/ehci-mxc.c:192: error: for each function it
> > > appears in.)
> > > drivers/usb/host/ehci-mxc.c:117: warning: unused variable 'temp'
> > > make[3]: *** [drivers/usb/host/ehci-hcd.o] Error 1
> > > make[2]: *** [drivers/usb/host/ehci-hcd.o] Error 2
> > > make[1]: *** [sub-make] Error 2
> > > make: *** [all] Error 2
> > >
> > > Fix it together with the warning about the unused variable and use
> > > msleep instead of mdelay as requested by Alan Stern.
> > >
> > > Cc: Dinh Nguyen <Dinh.Nguyen@freescale.com>
> > > Cc: Alan Stern <stern@rowland.harvard.edu>
> > > Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>
> > > ---
> > > Hello,
> > >
> > > I currently have no hardware to test it, so I only verified it to
> > > compile.
> > >
> > > Dinh: I changed your Tested-by to Cc for the new patch, do you care to
> > > retest?
> >
> > Retested - check!
> Do you take this patch and "USB: Fix FSL USB driver on non Open Firmware
> systems"?

Not yet, I'm at the Plumbers conference and have had limited time to be
committing patches at the moment. It's in my to-apply queue, so don't
worry, it's not lost.

thanks,

greg k-h


\
 
 \ /
  Last update: 2010-11-04 14:29    [W:0.038 / U:0.536 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site