On 3/18/19 1:31 PM, Ismael Luceno Cortes wrote:
The name assigned to the USB host controller interface is a better indicator than the counter currently in use (which has no meaning other than being the order in which the interface is found).
Example of the original output:
USB0: USB EHCI 1.10 scanning bus 0 for devices... 2 USB Device(s) found scanning usb for storage devices... 1 Storage Device(s) found
Patched output:
USB0(usb@ee080100): USB EHCI 1.10 scanning bus 0 for devices... 2 USB Device(s) found scanning usb for storage devices... 1 Storage Device(s) found
Signed-off-by: Ismael Luceno ismael.luceno@silicon-gears.com
Notes: Changes since v1: - Improved commit message.
drivers/usb/host/usb-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c index 611ea97a72..0575f5393b 100644 --- a/drivers/usb/host/usb-uclass.c +++ b/drivers/usb/host/usb-uclass.c @@ -255,7 +255,7 @@ int usb_init(void)
uclass_foreach_dev(bus, uc) { /* init low_level USB */
printf("USB%d: ", count);
printf("USB%d(%s): ", count, bus->name);
Hmmm, why dont we just drop the count altogether then ?
The output would then be:
USB %s: USB EHCI... scanning bus %s for devices...
for %s being the bus->name.
With the improved commit message, this would actually make sense. Does that make sense to you too or am I missing something ?
#ifdef CONFIG_SANDBOX
base-commit: 9ba5e5bc261a16f51662490da0cf620dc7f29013