Changes in uspace/drv/uhci-rhd/port.c [563ead9:5f94a0c] in mainline

File:

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

uspace/drv/uhci-rhd/port.c

@@ -36 +36 @@
 #include <errno.h>
 #include <str_error.h>
+#include <time.h>
 #include <async.h>
 
 #include <usb/usb.h>    /* usb_address_t */
-#include <usb/dev/hub.h>    /* usb_hc_new_device_wrapper */
+#include <usb/hub.h>    /* usb_hc_new_device_wrapper */
 #include <usb/debug.h>
 
@@ -81 +82 @@
  * @param[in] number Port number.
  * @param[in] usec Polling interval.
- * @param[in] rh Pointer to ddf instance of the root hub driver.
+ * @param[in] rh Pointer to ddf instance fo the root hub driver.
  * @return Error code.
  *
@@ -90 +91 @@
 {
         assert(port);
-        char *id_string;
-        asprintf(&id_string, "Port (%p - %u)", port, number);
-        if (id_string == NULL) {
+        asprintf(&port->id_string, "Port (%p - %u)", port, number);
+        if (port->id_string == NULL) {
                 return ENOMEM;
         }
 
-        port->id_string = id_string;
         port->address = address;
         port->number = number;
@@ -106 +105 @@
             usb_hc_connection_initialize_from_device(&port->hc_connection, rh);
         if (ret != EOK) {
-                usb_log_error("%s: failed to initialize connection to HC.",
-                    port->id_string);
-                free(id_string);
+                usb_log_error("Failed to initialize connection to HC.");
                 return ret;
         }
@@ -116 +113 @@
                 usb_log_error("%s: failed to create polling fibril.",
                     port->id_string);
-                free(id_string);
                 return ENOMEM;
         }
@@ -136 +132 @@
         assert(port);
         free(port->id_string);
-        // TODO: Kill fibril here
+        /* TODO: Kill fibril here */
         return;
 }
@@ -154 +150 @@
 
                 /* Read register value */
-                const port_status_t port_status =
-                    uhci_port_read_status(instance);
+                port_status_t port_status = uhci_port_read_status(instance);
 
                 /* Print the value if it's interesting */
@@ -166 +161 @@
                 usb_log_debug("%s: Connected change detected: %x.\n",
                     instance->id_string, port_status);
+
+                int rc =
+                    usb_hc_connection_open(&instance->hc_connection);
+                if (rc != EOK) {
+                        usb_log_error("%s: Failed to connect to HC.",
+                            instance->id_string);
+                        continue;
+                }
 
                 /* Remove any old device */
@@ -172 +175 @@
                             instance->id_string);
                         uhci_port_remove_device(instance);
-                }
-
-                int ret =
-                    usb_hc_connection_open(&instance->hc_connection);
-                if (ret != EOK) {
-                        usb_log_error("%s: Failed to connect to HC.",
-                            instance->id_string);
-                        continue;
                 }
 
@@ -195 +190 @@
                 }
 
-                ret = usb_hc_connection_close(&instance->hc_connection);
-                if (ret != EOK) {
+                rc = usb_hc_connection_close(&instance->hc_connection);
+                if (rc != EOK) {
                         usb_log_error("%s: Failed to disconnect.",
                             instance->id_string);
@@ -214 +209 @@
 int uhci_port_reset_enable(int portno, void *arg)
 {
-        uhci_port_t *port = arg;
-        assert(port);
+        uhci_port_t *port = (uhci_port_t *) arg;
 
         usb_log_debug2("%s: new_device_enable_port.\n", port->id_string);
@@ -289 +283 @@
         usb_log_error("%s: Don't know how to remove device %" PRIun ".\n",
             port->id_string, port->attached_device);
-        port->attached_device = 0;
         return ENOTSUP;
 }