Subject: Re: [PATCH 4/6] powerpc: add USB peripheral support to MPC8272ADS

From: Li Yang <[email-masked]>
Date: Thu, 07 Aug 2008 11:50:14 +0800

On Wed, 2008-08-06 at 12:19 -0500, Scott Wood wrote:
> On Wed, Aug 06, 2008 at 03:04:43PM +0800, Li Yang wrote:
> > Signed-off-by: Li Yang <[email-masked]>
> > ---
> >  arch/powerpc/boot/dts/mpc8272ads.dts      |    8 ++++++++
> >  arch/powerpc/platforms/82xx/mpc8272_ads.c |   25 +++++++++++++++++++++++++
> >  arch/powerpc/platforms/82xx/pq2ads.h      |    3 +++
> >  3 files changed, 36 insertions(+), 0 deletions(-)
> > 
> > diff --git a/arch/powerpc/boot/dts/mpc8272ads.dts b/arch/powerpc/boot/dts/mpc8272ads.dts
> > index b2ce4c0..75cc94c 100644
> > --- a/arch/powerpc/boot/dts/mpc8272ads.dts
> > +++ b/arch/powerpc/boot/dts/mpc8272ads.dts
> > @@ -256,6 +256,14 @@
> >  				#address-cells = <1>;
> >  				#size-cells = <0>;
> >  			};
> > +
> > +			usb@11b60 {
> > +				compatible = "fsl,qe_udc";
> 
> "fsl,mpc8272-usb", "fsl,cpm2-usb".
> 
> Where is fsl,qe_udc documented or used (other than in the BCSR setup)?
> What does "udc" mean (the only reference to it in the 8272 manual is
> under ATM)?

udc is a common name for USB device mode drivers to work with Linux
gadget.  We will have two separate drivers for USB host and device.
Probably we can have two compatibles "fsl,qe_udc" and "fsl,qe-usb-host"

- Leo