On Monday, November 09, 2015 at 04:16:37 PM, Stephen Warren wrote:
> On 11/09/2015 02:03 AM, Ye.Li wrote:
> > All the i.MX6, i.MX23 and i.MX28 OTG controllers only support UTMI
> > interface. Set to ULPI is not correct, even the controller will reject
> > this setting in PORTSC register.
> > 
> > diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c
> > 
> > @@ -1018,18 +1018,10 @@ int usb_gadget_register_driver(struct
> > usb_gadget_driver *driver)
> > 
> >             return ret;
> >     
> >     ret = ci_udc_probe();
> 
> ...
> 
> > +    if (ret) {
> > +           DBG("udc probe failed, returned %d\n", ret);
> > +           return ret;
> > 
> >     }
> 
> The indentation of the "if" line is incorrect.

I'm also not quite fond of the DBG() statement, why not change it to plain
debug() ?

Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to