Changeset 90994fa in mainline for uspace/drv

Timestamp:

2011-10-12T20:03:36Z (14 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

162726b

Parents:

205f0766

Message:

usb: Hubs should store child functions not just handles.

Location:

uspace/drv/bus/usb

Files:

• uhcirh/port.c (modified) (7 diffs)
• usbhub/port.c (modified) (5 diffs)
• usbhub/port.h (modified) (1 diff)

uspace/drv/bus/usb/uhcirh/port.c

@@ -100 +100 @@
         port->number = number;
         port->wait_period_usec = usec;
-        port->attached_device.handle = 0;
+        port->attached_device.fun = NULL;
         port->attached_device.address = -1;
         port->rh = rh;
@@ -169 +169 @@
 
                 /* Remove any old device */
-                if (instance->attached_device.handle) {
+                if (instance->attached_device.fun) {
                         usb_log_debug2("%s: Removing device.\n",
                             instance->id_string);
@@ -261 +261 @@
                 ret = usb_hc_new_device_wrapper(port->rh, &port->hc_connection,
                     speed, uhci_port_reset_enable, port->number, port,
-                    &port->attached_device.address,
-                    &port->attached_device.handle, NULL, NULL, NULL);
+                    &port->attached_device.address, NULL, NULL, NULL,
+                    &port->attached_device.fun);
         } while (ret != EOK && ++count < 4);
 
@@ -274 +274 @@
         usb_log_info("New device at port %u, address %d (handle %" PRIun ").\n",
             port->number, port->attached_device.address,
-            port->attached_device.handle);
+            port->attached_device.fun->handle);
         return EOK;
 }
@@ -287 +287 @@
         assert(port);
         /* There is nothing to remove. */
-        if (port->attached_device.handle == 0) {
+        if (port->attached_device.fun == NULL) {
                 usb_log_warning("%s: Removed a ghost device.\n",
                     port->id_string);
@@ -297 +297 @@
 
         /* Stop driver first */
-        int ret = devman_remove_function(port->attached_device.handle);
+        int ret = ddf_fun_unbind(port->attached_device.fun);
         if (ret != EOK) {
                 usb_log_error("%s: Failed to remove child function: %s.\n",
@@ -303 +303 @@
                 return ret;
         }
-        port->attached_device.handle = 0;
+        ddf_fun_destroy(port->attached_device.fun);
+        port->attached_device.fun = NULL;
 
         /* Driver stopped, free used address */

uspace/drv/bus/usb/usbhub/port.c

@@ -257 +257 @@
                 usb_log_debug("Removing device on port %zu.\n",
                     port->port_number);
-                const int ret =
-                    devman_remove_function(port->attached_device.handle);
+                const int ret = ddf_fun_unbind(port->attached_device.fun);
                 if (ret == EOK) {
+                        ddf_fun_destroy(port->attached_device.fun);
                         const int ret =
                             usb_hc_unregister_device(&hub->connection,
@@ -269 +269 @@
                         }
                 } else {
-                        usb_log_error("Failed to remove child function on port"
+                        usb_log_error("Failed to unbind child function on port"
                            " %zu: %s.\n", port->port_number, str_error(ret));
                 }
                 port->attached_device.address = -1;
-                port->attached_device.handle = 0;
+                port->attached_device.fun = NULL;
                 fibril_mutex_unlock(&port->mutex);
                 usb_log_info("Removed device on port %zu.\n",
@@ -418 +418 @@
 
         usb_address_t new_address;
-        devman_handle_t child_handle;
+        ddf_fun_t *child_fun;
 
         const int rc = usb_hc_new_device_wrapper(data->hub->usb_device->ddf_dev,
             &data->hub->connection, data->speed,
             enable_port_callback, (int) data->port->port_number,
-            data->port, &new_address, &child_handle,
-            NULL, NULL, NULL);
+            data->port, &new_address, NULL,
+            NULL, NULL, &child_fun);
 
         if (rc != EOK) {
@@ -433 +433 @@
 
         fibril_mutex_lock(&data->port->mutex);
-        data->port->attached_device.handle = child_handle;
+        data->port->attached_device.fun = child_fun;
         data->port->attached_device.address = new_address;
         fibril_mutex_unlock(&data->port->mutex);
@@ -440 +440 @@
             "address %d (handle %" PRIun ").\n",
             data->hub->usb_device->ddf_dev->name, data->port->port_number,
-            new_address, child_handle);
+            new_address, child_fun->handle);
 
 leave:

uspace/drv/bus/usb/usbhub/port.h

@@ -70 +70 @@
         assert(port);
         port->attached_device.address = -1;
-        port->attached_device.handle = 0;
+        port->attached_device.fun = NULL;
         port->port_number = port_number;
         port->control_pipe = control_pipe;