Changes in uspace/drv/bus/usb/usbmid/usbmid.c [d3a1ad58:56fd7cf] in mainline

File:

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

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

@@ -31 +31 @@
  */
 
+/* XXX Fix this */
+#define _DDF_DATA_IMPLANT
+
 /**
  * @file
@@ -39 +42 @@
 #include <stdlib.h>
 #include <usb_iface.h>
+#include <usb/ddfiface.h>
 #include <usb/dev/pipes.h>
 #include <usb/classes/classes.h>
 #include <usb/dev/recognise.h>
 #include "usbmid.h"
-/** Get host controller handle by calling the parent usb_device_t.
- *
- * @param[in] fun Device function the operation is running on.
- * @param[out] handle Storage for the host controller handle.
- * @return Error code.
- */
-static int usb_iface_device_hc_handle(ddf_fun_t *fun, devman_handle_t *handle)
-{
-        assert(handle);
-        assert(fun);
-        usb_device_t *usb_dev = usb_device_get(ddf_fun_get_dev(fun));
-        assert(usb_dev);
-        *handle = usb_device_hc_handle(usb_dev);
-        return EOK;
-}
-
-/** Get USB device address by calling the parent usb_device_t.
- *
- * @param[in] fun Device function the operation is running on.
- * @param[in] handle Devman handle of USB device we want address of.
- * @param[out] address Storage for USB address of device with handle @p handle.
- * @return Error code.
- */
-static int usb_iface_device_address(ddf_fun_t *fun, usb_address_t *address)
-{
-        assert(address);
-        assert(fun);
-        usb_device_t *usb_dev = usb_device_get(ddf_fun_get_dev(fun));
-        assert(usb_dev);
-        *address = usb_device_address(usb_dev);
-        return EOK;
-}
 
 /** Callback for DDF USB interface. */
-static int usb_iface_iface(ddf_fun_t *fun, int *iface_no)
+static int usb_iface_get_interface_impl(ddf_fun_t *fun, int *iface_no)
 {
         usbmid_interface_t *iface = ddf_fun_data_get(fun);
@@ -89 +61 @@
 }
 
-static int usb_iface_register_endpoint(ddf_fun_t *fun, usb_endpoint_t ep,
-    usb_transfer_type_t type, usb_direction_t dir, size_t mps, unsigned inter)
-{
-        usb_device_t *usb_dev = usb_device_get(ddf_fun_get_dev(fun));
-        assert(usb_dev);
-        async_exch_t *exch = usb_device_bus_exchange_begin(usb_dev);
-        if (!exch)
-                return ENOMEM;
-        const int ret = usb_register_endpoint(exch, ep, type, dir, mps, inter);
-        usb_device_bus_exchange_end(exch);
-        return ret;
-}
-
-static int usb_iface_unregister_endpoint(ddf_fun_t *fun, usb_endpoint_t ep,
-    usb_direction_t dir)
-{
-        usb_device_t *usb_dev = usb_device_get(ddf_fun_get_dev(fun));
-        assert(usb_dev);
-        async_exch_t *exch = usb_device_bus_exchange_begin(usb_dev);
-        if (!exch)
-                return ENOMEM;
-        const int ret = usb_unregister_endpoint(exch, ep, dir);
-        usb_device_bus_exchange_end(exch);
-        return ret;
-}
-
 /** DDF interface of the child - interface function. */
 static usb_iface_t child_usb_iface = {
-        .get_hc_handle = usb_iface_device_hc_handle,
-        .get_my_address = usb_iface_device_address,
-        .get_my_interface = usb_iface_iface,
-        .register_endpoint = usb_iface_register_endpoint,
-        .unregister_endpoint = usb_iface_unregister_endpoint,
+        .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,
 };
 
@@ -137 +81 @@
                 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;
 }
@@ -150 +98 @@
  */
 int usbmid_spawn_interface_child(usb_device_t *parent,
-    usbmid_interface_t **iface_ret,
+    usbmid_interface_t *iface,
     const usb_standard_device_descriptor_t *device_descriptor,
     const usb_standard_interface_descriptor_t *interface_descriptor)
@@ -171 +119 @@
 
         /* Create the device. */
-        child = usb_device_ddf_fun_create(parent, fun_inner, child_name);
+        child = ddf_fun_create(parent->ddf_dev, fun_inner, child_name);
         free(child_name);
         if (child == NULL) {
@@ -197 +145 @@
         }
         clean_match_ids(&match_ids);
-        ddf_fun_set_ops(child, &child_device_ops);
-
-        usbmid_interface_t *iface = ddf_fun_data_alloc(child, sizeof(*iface));
-
-        iface->fun = child;
-        iface->interface_no = interface_descriptor->interface_number;
-        link_initialize(&iface->link);
 
         rc = ddf_fun_bind(child);
@@ -211 +152 @@
                 return rc;
         }
-        *iface_ret = iface;
+
+        iface->fun = child;
+        ddf_fun_data_implant(child, iface);
+        ddf_fun_set_ops(child, &child_device_ops);
 
         return EOK;