This patch is to set xhci root hub's DeviceRemovable according to usb port's
connect type which currently comes from ACPI information rather than xhci PORTSC
register due to Windows prefers to ACPI information. If ACPI information was
different with PORTSC, there would be a warning.
Signed-off-by: Lan Tianyu <tianyu....@intel.com>
---
drivers/usb/host/xhci-hub.c | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/drivers/usb/host/xhci-hub.c b/drivers/usb/host/xhci-hub.c
index a686cf4..50a3c39 100644
--- a/drivers/usb/host/xhci-hub.c
+++ b/drivers/usb/host/xhci-hub.c
@@ -97,11 +97,18 @@ static void xhci_usb2_hub_descriptor(struct usb_hcd *hcd,
struct xhci_hcd *xhci,
/* If a device is removable, PORTSC reports a 0, same as in the
* hub descriptor DeviceRemovable bits.
*/
- if (portsc & PORT_DEV_REMOVE)
+ if (usb_get_hub_port_connect_type(
+ hcd->self.root_hub, i + 1)
+ == USB_PORT_CONNECT_TYPE_HARD_WIRED) {
+ if (!(portsc & PORT_DEV_REMOVE))
+ xhci_warn(xhci, "usb2.0 port%d's connect type
from ACPI is different with PORTSC register. ACPI returns hard-wired.\n",
+ i + 1);
+
/* This math is hairy because bit 0 of DeviceRemovable
* is reserved, and bit 1 is for port 1, etc.
*/
port_removable[(i + 1) / 8] |= 1 << ((i + 1) % 8);
+ }
}
/* ch11.h defines a hub descriptor that has room for USB_MAXCHILDREN
@@ -148,8 +155,16 @@ static void xhci_usb3_hub_descriptor(struct usb_hcd *hcd,
struct xhci_hcd *xhci,
/* bit 0 is reserved, bit 1 is for port 1, etc. */
for (i = 0; i < ports; i++) {
portsc = xhci_readl(xhci, xhci->usb3_ports[i]);
- if (portsc & PORT_DEV_REMOVE)
+
+ if (usb_get_hub_port_connect_type(
+ hcd->self.root_hub, i + 1)
+ == USB_PORT_CONNECT_TYPE_HARD_WIRED) {
+ if (!(portsc & PORT_DEV_REMOVE))
+ xhci_warn(xhci, "usb3.0 port%d's connect type
from ACPI is different with PORTSC register. ACPI returns hard-wired.\n",
+ i + 1);
+
port_removable |= 1 << (i + 1);
+ }
}
desc->u.ss.DeviceRemovable = cpu_to_le16(port_removable);
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
