usb: Perform device detect polling on all usb controllers.

Move the 100ms (USB_TIME_SIGATT) device detect polling from the
ohci/uhci/usb-hub code to the generic usb_hub_port_setup() code.  This
extends the 100ms polling to ehci and xhci controllers.  The code in
usb_hub_port_setup() now compares USB_TIME_SIGATT to the start of
usb_enumerate(), which may make boots faster when threads are
disabled.

This patch also changes the meaning of the return code for
hub->op->detect() calls.  Now 1 indicates device found, 0 indicates
device not found, and -1 indicates permanent failure.

Also, the xhci controller generic delay of 100ms is replaced with a
20ms root hub power stabilize time.  This in combination with the
100ms for USB_TIME_SIGATT should be closer to the USB2 spec (the USB3
spec does not seem to declare an equivalent of USB_TIME_SIGATT).

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>

1 files changed, 2 insertions, 15 deletions

diff --git a/src/hw/usb-ohci.c b/src/hw/usb-ohci.c
index 8089e13b..7a220577 100644
--- a/src/hw/usb-ohci.c
+++ b/src/hw/usb-ohci.c
@@ -42,21 +42,8 @@ static int
 ohci_hub_detect(struct usbhub_s *hub, u32 port)
 {
     struct usb_ohci_s *cntl = container_of(hub->cntl, struct usb_ohci_s, usb);
-    u32 end = timer_calc(USB_TIME_SIGATT);
-    for (;;) {
-        u32 sts = readl(&cntl->regs->roothub_portstatus[port]);
-        if (sts & RH_PS_CCS)
-            // Device connected.
-            break;
-        if (timer_check(end))
-            // No device found.
-            return -1;
-        msleep(5);
-    }
-
-    // XXX - need to wait for USB_TIME_ATTDB if just powered up?
-
-    return 0;
+    u32 sts = readl(&cntl->regs->roothub_portstatus[port]);
+    return (sts & RH_PS_CCS) ? 1 : 0;
 }
 
 // Disable port