Changeset 90994fa in mainline for uspace/drv/bus/usb/usbhub/port.c

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.

File:

• uspace/drv/bus/usb/usbhub/port.c (modified) (5 diffs)

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: