Changes in uspace/drv/uhci-rhd/port.c [11349a85:4125b7d] in mainline

File:

• uspace/drv/uhci-rhd/port.c (modified) (5 diffs)

uspace/drv/uhci-rhd/port.c

@@ -32 +32 @@
  * @brief UHCI root hub port routines
  */
-#include <libarch/ddi.h>  /* pio_read and pio_write */
-#include <fibril_synch.h> /* async_usleep */
+#include <libarch/ddi.h> /* pio_read and pio_write */
 #include <errno.h>
 #include <str_error.h>
+#include <fibril_synch.h>
 
 #include <usb/usb.h>    /* usb_address_t */
-#include <usb/hub.h>    /* usb_hc_new_device_wrapper */
+#include <usb/hub.h>
 #include <usb/debug.h>
 
@@ -212 +212 @@
 
         /*
-         * Resets from root ports should be nominally 50ms (USB spec 7.1.7.3)
+         * The host then waits for at least 100 ms to allow completion of
+         * an insertion process and for power at the device to become stable.
+         */
+        async_usleep(100000);
+
+        /*
+         * Resets from root ports should be nominally 50ms
          */
         {
@@ -223 +229 @@
                 port_status &= ~STATUS_IN_RESET;
                 uhci_port_write_status(port, port_status);
-                while (uhci_port_read_status(port) & STATUS_IN_RESET);
-                // TODO: find a better way to waste time (it should be less than
-                // 10ms, if we reschedule it takes too much time (random
-                // interrupts can be solved by multiple attempts).
-                usb_log_debug2("%s: Reset Signal stop.\n", port->id_string);
-        }
+                usb_log_debug("%s: Reset Signal stop.\n", port->id_string);
+        }
+
+        /* the reset recovery time 10ms */
+        async_usleep(10000);
+
         /* Enable the port. */
         uhci_port_set_enabled(port, true);
-
-        /* Reset recovery period,
-         * devices do not have to respond during this period
-         */
-        async_usleep(10000);
         return EOK;
 }
@@ -254 +255 @@
         usb_log_debug("%s: Detected new device.\n", port->id_string);
 
-        int ret, count = 0;
         usb_address_t dev_addr;
-        do {
-                ret = usb_hc_new_device_wrapper(port->rh, &port->hc_connection,
-                    speed, uhci_port_reset_enable, port->number, port,
-                    &dev_addr, &port->attached_device, NULL, NULL, NULL);
-        } while (ret != EOK && ++count < 4);
+        int ret = usb_hc_new_device_wrapper(port->rh, &port->hc_connection,
+            speed, uhci_port_reset_enable, port->number, port,
+            &dev_addr, &port->attached_device, NULL, NULL, NULL);
 
         if (ret != EOK) {
@@ -315 +313 @@
         /* Wait for port to become enabled */
         do {
+                async_usleep(1000);
                 port_status = uhci_port_read_status(port);
         } while ((port_status & STATUS_CONNECTED) &&