On 03.05.2016 22:51, Marek Vasut wrote:
The code shouldn't continue probing the port if get_port_status() failed.

Signed-off-by: Marek Vasut <ma...@denx.de>
Cc: Chin Liang See <cl...@altera.com>
Cc: Dinh Nguyen <dingu...@opensource.altera.com>
Cc: Hans de Goede <hdego...@redhat.com>
Cc: Stefan Roese <s...@denx.de>
Cc: Stephen Warren <swar...@nvidia.com>
---
  common/usb_hub.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/common/usb_hub.c b/common/usb_hub.c
index 4f59802..0f39c9f 100644
--- a/common/usb_hub.c
+++ b/common/usb_hub.c
@@ -402,6 +402,7 @@ static int usb_scan_port(struct usb_device_scan *usb_scan)
                        free(usb_scan);
                        return 0;
                }
+               return 0;
        }

        portstatus = le16_to_cpu(portsts->wPortStatus);


Thanks for spotting this:

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to