Changeset 8148ee3a in mainline for uspace/drv/ohci/hc.c

Timestamp:

2011-04-12T10:52:05Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a81736d5

Parents:

f324635

Message:

Root hub does not need ddf device pointer

Use endpoint manager registration wrapper

File:

• uspace/drv/ohci/hc.c (modified) (3 diffs)

uspace/drv/ohci/hc.c

@@ -55 +55 @@
         assert(hub_fun);
 
+        int ret;
+
         usb_address_t hub_address =
             device_keeper_get_free_address(&instance->manager, USB_SPEED_FULL);
+        if (hub_address <= 0) {
+                usb_log_error("Failed to get OHCI root hub address.\n");
+                return hub_address;
+        }
         instance->rh.address = hub_address;
         usb_device_keeper_bind(
             &instance->manager, hub_address, hub_fun->handle);
 
-        endpoint_t *ep = malloc(sizeof(endpoint_t));
-        assert(ep);
-        int ret = endpoint_init(ep, hub_address, 0, USB_DIRECTION_BOTH,
-            USB_TRANSFER_CONTROL, USB_SPEED_FULL, 64);
-        assert(ret == EOK);
-        ret = usb_endpoint_manager_register_ep(&instance->ep_manager, ep, 0);
-        assert(ret == EOK);
+        ret = usb_endpoint_manager_add_ep(&instance->ep_manager,
+            hub_address, 0, USB_DIRECTION_BOTH, USB_TRANSFER_CONTROL,
+            USB_SPEED_FULL, 64, 0);
+        if (ret != EOK) {
+                usb_log_error("Failed to add OHCI rh endpoint 0.\n");
+                usb_device_keeper_release(&instance->manager, hub_address);
+                return ret;
+        }
 
         char *match_str = NULL;
+        /* DDF needs heap allocated string */
         ret = asprintf(&match_str, "usb&class=hub");
-//      ret = (match_str == NULL) ? ret : EOK;
         if (ret < 0) {
                 usb_log_error(
                     "Failed(%d) to create root hub match-id string.\n", ret);
+                usb_device_keeper_release(&instance->manager, hub_address);
                 return ret;
         }
@@ -80 +88 @@
         ret = ddf_fun_add_match_id(hub_fun, match_str, 100);
         if (ret != EOK) {
-                usb_log_error("Failed add create root hub match-id.\n");
+                usb_log_error("Failed add root hub match-id.\n");
         }
         return ret;
@@ -115 +123 @@
         fibril_mutex_initialize(&instance->guard);
 
-        rh_init(&instance->rh, dev, instance->registers);
+        rh_init(&instance->rh, instance->registers);
 
         if (!interrupts) {