Changes in uspace/drv/bus/usb/vhc/connhost.c [02fc5c4:82a31261] in mainline

File:

• uspace/drv/bus/usb/vhc/connhost.c (modified) (13 diffs)

uspace/drv/bus/usb/vhc/connhost.c

@@ -57 +57 @@
  * @return Error code.
  */
-static int request_address(ddf_fun_t *fun, usb_address_t *address, bool strict,
-    usb_speed_t speed)
-{
-        VHC_DATA(vhc, fun);
-
-        assert(address);
-        return usb_device_manager_request_address(
-            &vhc->dev_manager, address, strict, speed);
+static int request_address(ddf_fun_t *fun, usb_speed_t speed,
+    usb_address_t *address)
+{
+        VHC_DATA(vhc, fun);
+
+        usb_address_t addr = usb_device_manager_get_free_address(
+            &vhc->dev_manager, USB_SPEED_HIGH);
+        if (addr < 0) {
+                return addr;
+        }
+
+        if (address != NULL) {
+                *address = addr;
+        }
+
+        return EOK;
 }
 
@@ -80 +88 @@
         usb_log_debug("Binding handle %" PRIun " to address %d.\n",
             handle, address);
-        usb_device_manager_bind_address(&vhc->dev_manager, address, handle);
+        usb_device_manager_bind(&vhc->dev_manager, address, handle);
 
         return EOK;
@@ -96 +104 @@
 {
         VHC_DATA(vhc, fun);
-        return usb_device_manager_get_info_by_address(
-            &vhc->dev_manager, address, handle, NULL);
+        bool found =
+            usb_device_manager_find_by_address(&vhc->dev_manager, address, handle);
+        return found ? EOK : ENOENT;
 }
 
@@ -110 +119 @@
         VHC_DATA(vhc, fun);
         usb_log_debug("Releasing address %d...\n", address);
-        usb_device_manager_release_address(&vhc->dev_manager, address);
+        usb_device_manager_release(&vhc->dev_manager, address);
 
         return ENOTSUP;
@@ -128 +137 @@
  */
 static int register_endpoint(ddf_fun_t *fun,
-    usb_address_t address, usb_endpoint_t endpoint,
+    usb_address_t address, usb_speed_t speed, usb_endpoint_t endpoint,
     usb_transfer_type_t transfer_type, usb_direction_t direction,
     size_t max_packet_size, unsigned int interval)
 {
-        VHC_DATA(vhc, fun);
-
-        return usb_endpoint_manager_add_ep(&vhc->ep_manager,
-            address, endpoint, direction, transfer_type, USB_SPEED_FULL, 1, 0,
-            NULL, NULL);
-
+        /* TODO: Use usb_endpoint_manager_add_ep */
+        VHC_DATA(vhc, fun);
+
+        endpoint_t *ep = endpoint_get(
+            address, endpoint, direction, transfer_type, USB_SPEED_FULL, 1);
+        if (ep == NULL) {
+                return ENOMEM;
+        }
+
+        int rc = usb_endpoint_manager_register_ep(&vhc->ep_manager, ep, 1);
+        if (rc != EOK) {
+                endpoint_destroy(ep);
+                return rc;
+        }
+
+        return EOK;
 }
 
@@ -153 +172 @@
         VHC_DATA(vhc, fun);
 
-        int rc = usb_endpoint_manager_remove_ep(&vhc->ep_manager,
-            address, endpoint, direction, NULL, NULL);
+        int rc = usb_endpoint_manager_unregister_ep(&vhc->ep_manager,
+            address, endpoint, direction);
 
         return rc;
@@ -395 +414 @@
         VHC_DATA(vhc, fun);
 
-        endpoint_t *ep = usb_endpoint_manager_find_ep(&vhc->ep_manager,
-            target.address, target.endpoint, USB_DIRECTION_IN);
+        endpoint_t *ep = usb_endpoint_manager_get_ep(&vhc->ep_manager,
+            target.address, target.endpoint, USB_DIRECTION_IN, NULL);
         if (ep == NULL) {
                 return ENOENT;
@@ -421 +440 @@
         int rc = vhc_virtdev_add_transfer(vhc, transfer);
         if (rc != EOK) {
-                if (transfer->setup_buffer != NULL) {
-                        free(transfer->setup_buffer);
-                }
+                free(transfer->setup_buffer);
                 free(transfer);
                 return rc;
@@ -437 +454 @@
         VHC_DATA(vhc, fun);
 
-        endpoint_t *ep = usb_endpoint_manager_find_ep(&vhc->ep_manager,
-            target.address, target.endpoint, USB_DIRECTION_OUT);
+        endpoint_t *ep = usb_endpoint_manager_get_ep(&vhc->ep_manager,
+            target.address, target.endpoint, USB_DIRECTION_OUT, NULL);
         if (ep == NULL) {
                 return ENOENT;
@@ -471 +488 @@
 }
 
-static int tell_address(ddf_fun_t *fun, usb_address_t *address)
+static int tell_address(ddf_fun_t *fun, devman_handle_t handle,
+    usb_address_t *address)
 {
         UNSUPPORTED("tell_address");
@@ -488 +506 @@
 }
 
-static int tell_address_rh(ddf_fun_t *root_hub_fun, usb_address_t *address)
+static int tell_address_rh(ddf_fun_t *root_hub_fun, devman_handle_t handle,
+    usb_address_t *address)
 {
         VHC_DATA(vhc, root_hub_fun);
 
-        devman_handle_t handle = root_hub_fun->handle;
+        if (handle == 0) {
+                handle = root_hub_fun->handle;
+        }
 
         usb_log_debug("tell_address_rh(handle=%" PRIun ")\n", handle);
-        const usb_address_t addr =
-            usb_device_manager_find_address(&vhc->dev_manager, handle);
+        usb_address_t addr = usb_device_manager_find(&vhc->dev_manager, handle);
         if (addr < 0) {
                 return addr;
@@ -508 +528 @@
         .request_address = request_address,
         .bind_address = bind_address,
-        .get_handle = find_by_address,
+        .find_by_address = find_by_address,
         .release_address = release_address,
 
@@ -520 +540 @@
 usb_iface_t vhc_usb_iface = {
         .get_hc_handle = usb_iface_get_hc_handle_hc_impl,
-        .get_my_address = tell_address
+        .get_address = tell_address
 };
 
 usb_iface_t rh_usb_iface = {
         .get_hc_handle = usb_iface_get_hc_handle_rh_impl,
-        .get_my_address = tell_address_rh
+        .get_address = tell_address_rh
 };