[U-Boot] [PATCH v2 9/9] usbd driver support for SPEAr SoCs
Remy Bohmer
linux at bohmer.net
Sat Jan 9 10:50:00 CET 2010
Hi,
Some small coding style remarks below.
2010/1/7 Vipin KUMAR <vipin.kumar at st.com>:
>
> Signed-off-by: Vipin <vipin.kumar at st.com>
> ---
> drivers/serial/usbtty.h | 2 +
> drivers/usb/gadget/Makefile | 1 +
> drivers/usb/gadget/spr_udc.c | 1004 ++++++++++++++++++++++++++++++++++++++++++
> include/configs/spear.h | 7 +-
> include/usb/spr_udc.h | 227 ++++++++++
> 5 files changed, 1240 insertions(+), 1 deletions(-)
> mode change 100644 => 100755 drivers/serial/usbtty.h
> mode change 100644 => 100755 drivers/usb/gadget/Makefile
> create mode 100755 drivers/usb/gadget/spr_udc.c
> create mode 100755 include/usb/spr_udc.h
> + for (i = 0; i < nb; i++) {
> + writeb(readb(fifo_ptr), bytp);
> + fifo_ptr++;
> + bytp++;
> + }
> + readl(&outep_regs_p[epNum].write_done);
> +
> + return 0;
> +}
> +
> +static void usbputpcktofifo(int epNum, u8 * bufp, u32 len)
Should be u8 *bufp. remove the extra space
Same comment for usbgetpckfromfifo() routine
> +
> + /* SetInterface command received */
> + if (readl(&udc_regs_p->dev_int) & DEV_INT_SETINTF) {
> + writel(DEV_INT_SETINTF, &udc_regs_p->dev_int);
> + }
No brackets for single statement blocks
> + /* USB Start-Of-Frame detected on cable */
> + if (readl(&udc_regs_p->dev_int) & DEV_INT_SOF) {
> + writel(DEV_INT_SOF, &udc_regs_p->dev_int);
> + }
No brackets for single statement blocks
That's all, so after fixing these issues:
Acked-by: Remy Bohmer <linux at bohmer.net>
I suggest the series are being kept together, so Tom do you take it
in your tree?
Kind regards,
Remy
More information about the U-Boot
mailing list