linux-kernel - Re: [PATCH] USB: ehci/mxc: compile fix

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 4 Nov 2010 11:28:18 +0100
From:	Uwe Kleine-König 
	<u.kleine-koenig@...gutronix.de>
To:	Greg KH <gregkh@...e.de>
Cc:	kernel@...gutronix.de, Sergei Shtylyov <sshtylyov@...sta.com>,
	matthieu.castet@...rot.com, linux-kernel@...r.kernel.org,
	linux-usb@...r.kernel.org, Alan Stern <stern@...land.harvard.edu>,
	Nguyen Dinh-R00091 <R00091@...escale.com>
Subject: Re: [PATCH] USB: ehci/mxc: compile fix

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@...gutronix.de]
> > Sent: Tuesday, November 02, 2010 4:31 AM
> > To: Greg KH
> > Cc: kernel@...gutronix.de; Sergei Shtylyov; matthieu.castet@...rot.com;
> > linux-kernel@...r.kernel.org; linux-usb@...r.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@...escale.com>
> > Cc: Alan Stern <stern@...land.harvard.edu>
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...gutronix.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"?

Just drop me a note if you need them resent.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/