Changes in uspace/drv/bus/usb/usbmid/usbmid.c [56fd7cf:d3a1ad58] 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
@@ -42 +39 @@
 #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_get_interface_impl(ddf_fun_t *fun, int *iface_no)
+static int usb_iface_iface(ddf_fun_t *fun, int *iface_no)
 {
         usbmid_interface_t *iface = ddf_fun_data_get(fun);
@@ -61 +89 @@
 }
 
+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_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_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,
 };
 
@@ -81 +137 @@
                 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;
 }
@@ -98 +150 @@
  */
 int usbmid_spawn_interface_child(usb_device_t *parent,
-    usbmid_interface_t *iface,
+    usbmid_interface_t **iface_ret,
     const usb_standard_device_descriptor_t *device_descriptor,
     const usb_standard_interface_descriptor_t *interface_descriptor)
@@ -119 +171 @@
 
         /* Create the device. */
-        child = ddf_fun_create(parent->ddf_dev, fun_inner, child_name);
+        child = usb_device_ddf_fun_create(parent, fun_inner, child_name);
         free(child_name);
         if (child == NULL) {
@@ -145 +197 @@
         }
         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);
@@ -152 +211 @@
                 return rc;
         }
-
-        iface->fun = child;
-        ddf_fun_data_implant(child, iface);
-        ddf_fun_set_ops(child, &child_device_ops);
+        *iface_ret = iface;
 
         return EOK;