Changes in uspace/lib/usbdev/src/hub.c [7711296:2179cf95] in mainline

File:

• uspace/lib/usbdev/src/hub.c (modified) (5 diffs)

uspace/lib/usbdev/src/hub.c

@@ -76 +76 @@
 {
         CHECK_CONNECTION(connection);
-
+        
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        if (!exch)
-                return (usb_address_t)ENOMEM;
-
-        usb_address_t address = preferred;
-        const int ret = usbhc_request_address(exch, &address, strict, speed);
-
+        
+        sysarg_t address;
+        int rc = async_req_4_1(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_REQUEST_ADDRESS, preferred, strict, speed, &address);
+        
         async_exchange_end(exch);
-        return ret == EOK ? address : ret;
+        
+        if (rc != EOK)
+                return (usb_address_t) rc;
+        
+        return (usb_address_t) address;
 }
 
@@ -94 +97 @@
  * @return Error code.
  */
-int usb_hc_register_device(usb_hc_connection_t *connection,
+int usb_hc_register_device(usb_hc_connection_t * connection,
     const usb_hub_attached_device_t *attached_device)
 {
         CHECK_CONNECTION(connection);
-        if (attached_device == NULL || attached_device->fun == NULL)
-                return EINVAL;
-
+        
+        if (attached_device == NULL)
+                return EBADMEM;
+        
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        if (!exch)
-                return ENOMEM;
-        const int ret = usbhc_bind_address(exch,
+        int rc = async_req_3_0(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_BIND_ADDRESS,
             attached_device->address, attached_device->fun->handle);
         async_exchange_end(exch);
-
-        return ret;
+        
+        return rc;
 }
 
@@ -121 +124 @@
 {
         CHECK_CONNECTION(connection);
-
+        
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        if (!exch)
-                return ENOMEM;
-        const int ret = usbhc_release_address(exch, address);
+        int rc = async_req_2_0(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_RELEASE_ADDRESS, address);
         async_exchange_end(exch);
-
-        return ret;
+        
+        return rc;
 }
 
@@ -219 +221 @@
  *      request or requests for descriptors when creating match ids).
  */
-int usb_hc_new_device_wrapper(ddf_dev_t *parent,
-    usb_hc_connection_t *connection, usb_speed_t dev_speed,
+int usb_hc_new_device_wrapper(ddf_dev_t *parent, usb_hc_connection_t *connection,
+    usb_speed_t dev_speed,
     int (*enable_port)(void *arg), void *arg, usb_address_t *assigned_address,
     ddf_dev_ops_t *dev_ops, void *new_dev_data, ddf_fun_t **new_fun)
@@ -227 +229 @@
                 return EINVAL;
 
+        // FIXME: this is awful, we are accessing directly the structure.
         // TODO: Why not use provided connection?
-        usb_hc_connection_t hc_conn;
-        usb_hc_connection_initialize(&hc_conn, connection->hc_handle);
+        usb_hc_connection_t hc_conn = {
+                .hc_handle = connection->hc_handle,
+                .hc_sess = NULL
+        };
 
         int rc;