USB: UHCI: adjust zhaoxin UHCI controllers OverCurrent bit value

OverCurrent condition is not standardized in the UHCI spec.
Zhaoxin UHCI controllers report OverCurrent bit active off.
In order to handle OverCurrent condition correctly, the uhci-hcd
driver needs to be told to expect the active-off behavior.

Suggested-by: Alan Stern <stern@rowland.harvard.edu>
Cc: stable@vger.kernel.org
Signed-off-by: Weitao Wang <WeitaoWang-oc@zhaoxin.com>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Link: https://lore.kernel.org/r/20230423105952.4526-1-WeitaoWang-oc@zhaoxin.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: LeoLiu-oc <leoliu-oc@zhaoxin.com>
This commit is contained in:
Weitao Wang 2023-04-23 18:59:52 +08:00 committed by LeoLiu-oc
parent 74935aa083
commit 5ab633d0b5
1 changed files with 6 additions and 4 deletions

View File

@ -119,11 +119,13 @@ static int uhci_pci_init(struct usb_hcd *hcd)
uhci->rh_numports = uhci_count_ports(hcd); uhci->rh_numports = uhci_count_ports(hcd);
/* Intel controllers report the OverCurrent bit active on. /*
* VIA controllers report it active off, so we'll adjust the * Intel controllers report the OverCurrent bit active on. VIA
* bit value. (It's not standardized in the UHCI spec.) * and ZHAOXIN controllers report it active off, so we'll adjust
* the bit value. (It's not standardized in the UHCI spec.)
*/ */
if (to_pci_dev(uhci_dev(uhci))->vendor == PCI_VENDOR_ID_VIA) if (to_pci_dev(uhci_dev(uhci))->vendor == PCI_VENDOR_ID_VIA ||
to_pci_dev(uhci_dev(uhci))->vendor == PCI_VENDOR_ID_ZHAOXIN)
uhci->oc_low = 1; uhci->oc_low = 1;
/* HP's server management chip requires a longer port reset delay. */ /* HP's server management chip requires a longer port reset delay. */