Changes in uspace/drv/bus/usb/usbmid/usbmid.c [317a463:5203e256] in mainline

File:

• uspace/drv/bus/usb/usbmid/usbmid.c (modified) (4 diffs)

uspace/drv/bus/usb/usbmid/usbmid.c

@@ -45 +45 @@
 
 /** Callback for DDF USB interface. */
-static int usb_iface_get_interface_impl(ddf_fun_t *fun, int *iface_no)
+static int usb_iface_get_address_impl(ddf_fun_t *fun, devman_handle_t handle,
+    usb_address_t *address)
+{
+        return usb_iface_get_address_hub_impl(fun, handle, address);
+}
+
+/** Callback for DDF USB interface. */
+static int usb_iface_get_interface_impl(ddf_fun_t *fun, devman_handle_t handle,
+    int *iface_no)
 {
         assert(fun);
@@ -61 +69 @@
 /** DDF interface of the child - interface function. */
 static usb_iface_t child_usb_iface = {
-        .get_hc_handle = usb_iface_get_hc_handle_device_impl,
-        .get_my_address = usb_iface_get_my_address_forward_impl,
-        .get_my_interface = usb_iface_get_interface_impl,
+        .get_hc_handle = usb_iface_get_hc_handle_hub_child_impl,
+        .get_address = usb_iface_get_address_impl,
+        .get_interface = usb_iface_get_interface_impl
 };
 
@@ -71 +79 @@
 };
 
-int usbmid_interface_destroy(usbmid_interface_t *mid_iface)
-{
-        assert(mid_iface);
-        assert_link_not_used(&mid_iface->link);
-        const int ret = ddf_fun_unbind(mid_iface->fun);
-        if (ret != EOK) {
-                return ret;
-        }
-        /* NOTE: usbmid->interface points somewhere, but we did not
-         * allocate that space, so don't touch */
-        ddf_fun_destroy(mid_iface->fun);
-        /* NOTE: mid_iface is invalid at this point, it was assigned to
-         * mid_iface->fun->driver_data and freed in ddf_fun_destroy */
-        return EOK;
-}
 
 /** Spawn new child device from one interface.
@@ -109 +102 @@
          * class name something humanly understandable.
          */
-        rc = asprintf(&child_name, "%s%hhu",
+        rc = asprintf(&child_name, "%s%d",
             usb_str_class(interface_descriptor->interface_class),
-            interface_descriptor->interface_number);
+            (int) interface_descriptor->interface_number);
         if (rc < 0) {
-                return ENOMEM;
+                goto error_leave;
         }
 
         /* Create the device. */
         child = ddf_fun_create(parent->ddf_dev, fun_inner, child_name);
-        free(child_name);
         if (child == NULL) {
-                return ENOMEM;
+                rc = ENOMEM;
+                goto error_leave;
         }
 
+        iface->fun = child;
+
+        child->driver_data = iface;
+        child->ops = &child_device_ops;
+
         rc = usb_device_create_match_ids_from_interface(device_descriptor,
-            interface_descriptor, &child->match_ids);
+            interface_descriptor,
+            &child->match_ids);
         if (rc != EOK) {
-                ddf_fun_destroy(child);
-                return rc;
+                goto error_leave;
         }
 
         rc = ddf_fun_bind(child);
         if (rc != EOK) {
+                goto error_leave;
+        }
+
+        return EOK;
+
+error_leave:
+        if (child != NULL) {
+                child->name = NULL;
                 /* This takes care of match_id deallocation as well. */
                 ddf_fun_destroy(child);
-                return rc;
+        }
+        if (child_name != NULL) {
+                free(child_name);
         }
 
-        iface->fun = child;
-        child->driver_data = iface;
-        child->ops = &child_device_ops;
-
-        return EOK;
+        return rc;
 }