[U-Boot] [RFC PATCH V2] drivers/usb: regorganisation

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Thu Apr 2 21:27:55 CEST 2009


On 20:07 Thu 02 Apr     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <1238691332-27548-1-git-send-email-plagnioj at jcrosoft.com> you wrote:
> > move to linux usb driver organisation
> > 
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> > Cc: Remy Bohmer <linux at bohmer.net>
> > ---
> > Remy is this one ok for you?
> 
> Looks OK to me, but..
> 
> 
> > diff --git a/drivers/serial/usbtty.c b/drivers/serial/usbtty.c
> > index d8b1387..71f0e5b 100644
> > --- a/drivers/serial/usbtty.c
> > +++ b/drivers/serial/usbtty.c
> > @@ -94,7 +94,7 @@ static char serial_number[16];
> >   * Descriptors, Strings, Local variables.
> >   */
> >  
> > -/* defined and used by usbdcore_ep0.c */
> > +/* defined and used by ep0.c */
> >  extern struct usb_string_descriptor **usb_strings;
> 
> This is now too short. Where might ep0.c be? Please use "gadget/ep0.c"
> here.
ok
> 
> 
> > rename to drivers/usb/gadget/mpc8xx_udc.c
> > index 0e311ad..8d7bd65 100644
> > --- a/drivers/usb/usbdcore_mpc8xx.c
> > +++ b/drivers/usb/gadget/mpc8xx_udc.c
> > @@ -3,7 +3,7 @@
> >   * bodonoghue at CodeHermit.ie
> >   *
> >   * References
> > - * DasUBoot/drivers/usb/usbdcore_omap1510.c, for design and implementation
> > + * DasUBoot/drivers/usb/omap1510_udc.c, for design and implementation
> 
> Please change into "drivers/usb/omap1510_udc.c"
drivers/usb/gadget/omap1510_udc.c
ok

if Remy is ok
could apply the at91 pull request then I will generate the final usb
reorganisation patch?

Best Regards,
J.


More information about the U-Boot mailing list