[U-Boot] [PATCH] usb: udc-uclass: Fixed problem when no alias is defined in DT
Lukasz Majewski
lukma at denx.de
Tue Feb 26 10:15:16 UTC 2019
Hi Vignesh,
> Hi Lukasz,
>
> On 24/01/19 8:14 PM, Jean-Jacques Hiblot wrote:
> > commit 801f1fa442 "dm: usb: udc: Use SEQ_ALIAS to index the USB
> > gadget ports" changed the way the udevice if found. It uses the
> > alias to find a udevice for a given USB port number. In the commit
> > log it was stated that if no alias is provided, the bind order will
> > be used instead. However it doesn't work. Fixing this by adding a
> > call to uclass_get_device() if uclass_get_device_by_seq() fails.
> >
> > Signed-off-by: Jean-Jacques Hiblot <jjhiblot at ti.com>
> > ---
>
> This fixes DFU gadget failing to find USB device on DRA7xx/AM57xx
> platforms. Since, this patch fixes a regression, could this be picked
> up for next -rc?
Yes, I will pick it up. Sorry for delay.
>
> Tested-by: Vignesh R <vigneshr at ti.com>
>
> > drivers/usb/gadget/udc/udc-uclass.c | 7 +++++--
> > 1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/usb/gadget/udc/udc-uclass.c
> > b/drivers/usb/gadget/udc/udc-uclass.c index 8d7864797a..3053ccf7d9
> > 100644 --- a/drivers/usb/gadget/udc/udc-uclass.c
> > +++ b/drivers/usb/gadget/udc/udc-uclass.c
> > @@ -23,8 +23,11 @@ int usb_gadget_initialize(int index)
> > return 0;
> > ret = uclass_get_device_by_seq(UCLASS_USB_GADGET_GENERIC,
> > index, &dev); if (!dev || ret) {
> > - pr_err("No USB device found\n");
> > - return -ENODEV;
> > + ret = uclass_get_device(UCLASS_USB_GADGET_GENERIC,
> > index, &dev);
> > + if (!dev || ret) {
> > + pr_err("No USB device found\n");
> > + return -ENODEV;
> > + }
> > }
> > dev_array[index] = dev;
> > return 0;
> >
>
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190226/42ff546c/attachment-0001.sig>
More information about the U-Boot
mailing list