Changeset 02fc5c4 in mainline for uspace/lib/usbdev/src/hub.c

Timestamp:

2011-11-25T17:41:23Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

095bddfc

Parents:

56bdd9a4

Message:

usbhc: Export IPC wrapper instead of IPC call numbers.

Hide IPC protocol in one file, instead of multiple client implementations.
Rename 'find_by_address' ⇒ 'get_handle'

File:

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

uspace/lib/usbdev/src/hub.c

@@ -76 +76 @@
 {
         CHECK_CONNECTION(connection);
-        
+
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        
-        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);
-        
+        if (!exch)
+                return (usb_address_t)ENOMEM;
+
+        usb_address_t address = preferred;
+        const int ret = usbhc_request_address(exch, &address, strict, speed);
+
         async_exchange_end(exch);
-        
-        if (rc != EOK)
-                return (usb_address_t) rc;
-        
-        return (usb_address_t) address;
+        return ret == EOK ? address : ret;
 }
 
@@ -97 +94 @@
  * @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)
-                return EBADMEM;
-        
+        if (attached_device == NULL || attached_device->fun == NULL)
+                return EINVAL;
+
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        int rc = async_req_3_0(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
-            IPC_M_USBHC_BIND_ADDRESS,
+        if (!exch)
+                return ENOMEM;
+        const int ret = usbhc_bind_address(exch,
             attached_device->address, attached_device->fun->handle);
         async_exchange_end(exch);
-        
-        return rc;
+
+        return ret;
 }
 
@@ -124 +121 @@
 {
         CHECK_CONNECTION(connection);
-        
+
         async_exch_t *exch = async_exchange_begin(connection->hc_sess);
-        int rc = async_req_2_0(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
-            IPC_M_USBHC_RELEASE_ADDRESS, address);
+        if (!exch)
+                return ENOMEM;
+        const int ret = usbhc_release_address(exch, address);
         async_exchange_end(exch);
-        
-        return rc;
+
+        return ret;
 }