[U-Boot] [PATCH v2 2/6] pxa25x: Add support for USB ethernet gadget

Marek Vasut marex at denx.de
Sun Sep 23 21:32:56 CEST 2012


Dear Łukasz Dałek,

> On 23.09.2012 20:24, Marek Vasut wrote:
> > Dear Łukasz Dałek,
> > 
> >> On 23.09.2012 17:44, Marek Vasut wrote:
> >>> Dear Lukasz Dalek,
> >>> 
> >>>> Add to pxa25x based devices support for USB ethernet gadget. This is a
> >>>> port of pxa25x UDC driver from Linux kernel.
> >>>> 
> >>>> Signed-off-by: Lukasz Dalek<luk0104 at gmail.com>
> >>>> ---
> >>>> 
> >>>>    drivers/usb/gadget/Makefile     |    1 +
> >>>>    drivers/usb/gadget/pxa25x_udc.c | 2059
> >>>> 
> >>>> +++++++++++++++++++++++++++++++++++++++
> >>>> drivers/usb/gadget/pxa25x_udc.h
> >>>> 
> >>>> | 162 +++
> >>>> | 
> >>>>    3 files changed, 2222 insertions(+), 0 deletions(-)
> >>>>    create mode 100644 drivers/usb/gadget/pxa25x_udc.c
> >>>>    create mode 100644 drivers/usb/gadget/pxa25x_udc.h
> >>> 
> >>> [...]
> >>> 
> >>> So what's the change here? Or is it a repost?
> >>> 
> >>> Best regards,
> >>> Marek Vasut
> >> 
> >> Everything was fine except DEBUG_NOISY defintions. I fixed them and
> >> I have resend patch.
> > 
> > http://www.denx.de/wiki/U-Boot/Patches did you read this? Worth reading
> > so I don't have to repeat myself ;-)
> 
> Yes.
> 
> >> Can I ask what about patches 3,4,5,6?
> > 
> > I think they're fine ... did you do any changes to them ?
> 
> If they are ok then my job is done :-)

You did not answer my question ... and no, your job isn't done ... once your 
code is mainline, you're the maintainer, every change to common code will also 
be your responsibility to at least review/test on your hardware etc ... you know 
the drill.

Even though PXA is semi-dead, so I don't expect much happening there.

> Łukasz Dałek

Best regards,
Marek Vasut


More information about the U-Boot mailing list