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

File:

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

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

@@ -34 +34 @@
  * Exploration of available interfaces in the USB device.
  */
-#include <ddf/driver.h>
 #include <errno.h>
 #include <str_error.h>
@@ -57 +56 @@
 static bool interface_in_list(const list_t *list, int interface_no)
 {
-        list_foreach(*list, link, usbmid_interface_t, iface) {
+        list_foreach(*list, l) {
+                usbmid_interface_t *iface = usbmid_interface_from_link(l);
                 if (iface->interface_no == interface_no) {
                         return true;
@@ -71 +71 @@
  * @param config_descriptor_size Size of configuration descriptor in bytes.
  * @param list List where to add the interfaces.
- * @return EOK on success, ENOMEM if out of memory.
- */
-static int create_interfaces(usb_mid_t *mid, const uint8_t *config_descriptor,
-    size_t config_descriptor_size)
+ */
+static void create_interfaces(const uint8_t *config_descriptor,
+    size_t config_descriptor_size, list_t *list)
 {
         const usb_dp_parser_data_t data = {
@@ -103 +102 @@
 
                 /* Skip alternate interfaces. */
-                if (interface_in_list(&mid->interface_list,
-                    interface->interface_number)) {
+                if (interface_in_list(list, interface->interface_number)) {
                         /* TODO: add the alternatives and create match ids
                          * for them. */
                         continue;
                 }
-
-                /* Create the function */
-                ddf_fun_t *fun = ddf_fun_create(mid->dev, fun_inner, NULL);
-                if (fun == NULL)
-                        goto error;
-
-                usbmid_interface_t *iface = ddf_fun_data_alloc(fun,
-                    sizeof(usbmid_interface_t));
-                if (iface == NULL)
-                        goto error;
+                usbmid_interface_t *iface = malloc(sizeof(usbmid_interface_t));
+                if (iface == NULL) {
+                        //TODO: Do something about that failure.
+                        break;
+                }
 
                 link_initialize(&iface->link);
-                iface->fun = fun;
+                iface->fun = NULL;
                 iface->interface_no = interface->interface_number;
                 iface->interface = interface;
 
-                list_append(&iface->link, &mid->interface_list);
-        }
-
-        return EOK;
-error:
-        while (!list_empty(&mid->interface_list)) {
-                link_t *link = list_first(&mid->interface_list);
-                usbmid_interface_t *iface = list_get_instance(link,
-                    usbmid_interface_t, link);
-
-                ddf_fun_destroy(iface->fun);
-        }
-
-        return ENOMEM;
+                list_append(&iface->link, list);
+        }
 }
 
@@ -185 +166 @@
         }
 
-        usb_mid->dev = dev->ddf_dev;
-
         /* Create control function. */
         usb_mid->ctl_fun = ddf_fun_create(dev->ddf_dev, fun_exposed, "ctl");
@@ -204 +183 @@
         }
 
+
         /* Create interface children. */
         list_initialize(&usb_mid->interface_list);
-        create_interfaces(usb_mid, config_descriptor_raw, config_descriptor_size);
+        create_interfaces(config_descriptor_raw, config_descriptor_size,
+            &usb_mid->interface_list);
 
         /* Start child function for every interface. */
-        list_foreach(usb_mid->interface_list, link, usbmid_interface_t, iface) {
+        list_foreach(usb_mid->interface_list, link) {
+                usbmid_interface_t *iface = usbmid_interface_from_link(link);
+
                 usb_log_info("Creating child for interface %d (%s).\n",
                     iface->interface_no,