[U-Boot] [PATCH] dm: usb: Do not use bus->seq before device_probe(bus)

[Simon Glass]

On 4 May 2015 at 13:37, Simon Glass <sjg at chromium.org> wrote:
> Hi Hans,
>
> On 4 May 2015 at 13:33, Hans de Goede <hdegoede at redhat.com> wrote:
>> Do not use bus->seq before device_probe(bus), as bus->seq is not set until
>> after the device_probe() call. This fixes u-boot printing: "USB-1:   " for
>> each bus it scans.
>>
>> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
>> ---
>>  drivers/usb/host/usb-uclass.c | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c
>> index 3655435..dd721cb 100644
>> --- a/drivers/usb/host/usb-uclass.c
>> +++ b/drivers/usb/host/usb-uclass.c
>> @@ -181,9 +181,8 @@ int usb_init(void)
>>
>>         uclass_foreach_dev(bus, uc) {
>>                 /* init low_level USB */
>> +               printf("USB%d:   ", count);
>
> This might be confusing if bus->seq ends up being something else. But
> I don't have a better idea and I've been thinking about it for a
> while.
>
> Acked-by: Simon Glass <sjg at chromium.org>
>
>>                 count++;
>> -               printf("USB");
>> -               printf("%d:   ", bus->seq);
>>                 ret = device_probe(bus);
>>                 if (ret == -ENODEV) {   /* No such device. */
>>                         puts("Port not available.\n");
>> --
>> 2.3.6
>>
>
> Regards,
> Simon

Applied to u-boot-dm, thanks!