Changeset 56bdd9a4 in mainline for uspace/lib/usbdev/src/pipes.c

Timestamp:

2011-11-25T15:20:04Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

02fc5c4

Parents:

317a463

Message:

usb: Provide IPC wrappers instead of IPC call numbers.

Put IPC protocol in one file.
This is does not make much difference for usb_iface, but will be useful
for more complicated interfaces.

File:

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

uspace/lib/usbdev/src/pipes.c

@@ -54 +54 @@
 static usb_address_t get_my_address(async_sess_t *sess, const ddf_dev_t *dev)
 {
+        assert(sess);
         async_exch_t *exch = async_exchange_begin(sess);
-        
-        sysarg_t address;
-        int rc = async_req_1_1(exch, DEV_IFACE_ID(USB_DEV_IFACE),
-            IPC_M_USB_GET_MY_ADDRESS, &address);
-        
+        if (!exch)
+                return ENOMEM;
+
+        usb_address_t address;
+        const int ret = usb_get_my_address(exch, &address);
+
         async_exchange_end(exch);
-        
-        if (rc != EOK)
-                return rc;
-        
-        return (usb_address_t) address;
+
+        return (ret == EOK) ? address : ret;
 }
 
@@ -71 +70 @@
  *
  * @param device Device in question.
- * @return Interface number (negative code means any).
+ * @return Error code (ENOTSUP means any).
  */
 int usb_device_get_assigned_interface(const ddf_dev_t *device)
@@ -80 +79 @@
             IPC_FLAG_BLOCKING);
         if (!parent_sess)
-                return -1;
-        
+                return ENOMEM;
+
         async_exch_t *exch = async_exchange_begin(parent_sess);
-        
-        sysarg_t iface_no;
-        int rc = async_req_1_1(exch, DEV_IFACE_ID(USB_DEV_IFACE),
-            IPC_M_USB_GET_MY_INTERFACE, &iface_no);
-        
-        async_exchange_end(exch);
-        async_hangup(parent_sess);
-        
-        if (rc != EOK)
-                return -1;
-        
-        return (int) iface_no;
+        if (!exch) {
+                async_hangup(parent_sess);
+                return ENOMEM;
+        }
+
+        int iface_no;
+        const int ret = usb_get_my_interface(exch, &iface_no);
+
+        return ret == EOK ? iface_no : ret;
 }