common: usb-hub: Reset hub port before scanning
authorShantur Rathore <i@shantur.com>
Sat, 9 Dec 2023 18:10:56 +0000 (18:10 +0000)
committerTom Rini <trini@konsulko.com>
Mon, 22 Jan 2024 17:49:03 +0000 (12:49 -0500)
commit33e06dcbe57a16e4f8b5b94dd8a3f89b87db2487
tree7463a941e8fe1a5f11a4d966119f2db3b0243526
parentc09ec8e1cb3baddef44458fb945fc258587a7d1b
common: usb-hub: Reset hub port before scanning

Currently when a hub is turned on, all the ports are powered on.
This works well for hubs which have individual power control.

For the hubs without individual power control this has no effect.
Mostly in these scenarios the hub port is powered before the USB
controller is enabled, this can lead to some devices in unexpected
state.

With this patch, we explicitly reset the port while powering up hub
This resets the port for hubs without port power control and has
no effect on hubs with port power control as the port is still off.

Before this patch AMicro AM8180 based NVME to USB adapter won't be
detected as a USB3.0 Mass Storage device but with this it works as
expected.

Tested working after this patch:
1. AMicro AM8180 based NVME to USB Adapter
2. Kingston DataTraveler 3.0
3. GenesysLogic USB3.0 Hub

The drives were tested while connected directly and via the hub.

Signed-off-by: Shantur Rathore <i@shantur.com>
Reviewed-by: Marek Vasut <marex@denx.de>
common/usb_hub.c