[U-Boot] [PATCH 0/3] add dfu support for at91 sam9260 based boards

[Marek Vasut <marex@denx.de>]

On Monday, July 20, 2015 at 04:03:08 PM, Lukasz Majewski wrote:
> Hi Marek,
> 
> > add dfu supprt for at91 sam9260 based boards. The USB
> > gadget driver is ported from linux:
> > 
> > b2ba27a5c56ff: usb: gadget: at91_udc: move prepare clk into process
> > context
> > 
> > it drops a lot of checkpatch warnings/errors:
> > 
> > checkpatch.pl found 12 error(s), 31 warning(s), 43 checks(s)
> > 
> > but for further updates I did not fix them.
> > The errors are all from this sort:
> > 
> > error: drivers/usb/gadget/at91_udc.c,87: space prohibited before open
> > square bracket '['
> > 
> > a lot of "line over 80 characters" warnings ...
> > 
> > Heiko Schocher (3):
> >   ARM: at91: add cpu.h
> >   usb: gadget: at91_udc: add support for at91_udc
> >   at91, taurus, smartweb: add dfu support
> 
> Marek, do you plan to review this patch series?
> 
> (I would need at least one review before I pull the code).

I took a brief look. One thing I don't quite like is the omnipresent #ifdef 
__UBOOT__ stuff. I'd much rather love to see some kind of a porting layer
so the driver can just be picked from linux as-is, but that might be too much
hassle.

Otherwise, I'm fine with it, though I nitpicked a bit.

Best regards,
Marek Vasut