Changeset f4138ac in mainline for uspace/lib/usbdev/src/devdrv.c

Timestamp:

2013-01-26T19:22:41Z (11 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

b06d35a

Parents:

2dc5a9f

Message:

libusbdev: Use alternate_interfaces.current.

It's initialized to zero if there are no alternatives.

File:

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

uspace/lib/usbdev/src/devdrv.c

@@ -76 +76 @@
  * @return Error code.
  */
-int usb_device_select_interface(usb_device_t *dev, uint8_t alternate_setting,
-    const usb_endpoint_description_t **endpoints)
-{
-        if (dev->interface_no < 0) {
+int usb_device_select_interface(usb_device_t *usb_dev,
+    uint8_t alternate_setting, const usb_endpoint_description_t **endpoints)
+{
+        assert(usb_dev);
+
+        if (usb_dev->interface_no < 0) {
                 return EINVAL;
         }
 
+        /* Change the interface itself. */
+        int rc = usb_request_set_interface(&usb_dev->ctrl_pipe,
+            usb_dev->interface_no, alternate_setting);
+        if (rc != EOK) {
+                return rc;
+        }
+
+        /* Change current alternative */
+        usb_dev->alternate_interfaces.current = alternate_setting;
+
         /* Destroy existing pipes. */
-        usb_device_destroy_pipes(dev);
-
-        /* Change the interface itself. */
-        int rc = usb_request_set_interface(&dev->ctrl_pipe, dev->interface_no,
-            alternate_setting);
-        if (rc != EOK) {
-                return rc;
-        }
+        usb_device_destroy_pipes(usb_dev);
 
         /* Create new pipes. */
-        rc = usb_device_create_pipes(&dev->wire, endpoints,
-            dev->descriptors.configuration, dev->descriptors.configuration_size,
-            dev->interface_no, (int)alternate_setting,
-            &dev->pipes, &dev->pipes_count);
+        rc = usb_device_create_pipes(&usb_dev->wire, endpoints,
+            usb_dev->descriptors.configuration,
+            usb_dev->descriptors.configuration_size,
+            usb_dev->interface_no, usb_dev->alternate_interfaces.current,
+            &usb_dev->pipes, &usb_dev->pipes_count);
 
         return rc;
@@ -381 +387 @@
             usb_dev->descriptors.configuration,
             usb_dev->descriptors.configuration_size, usb_dev->interface_no);
-        const int alternate_iface =
-            (rc == EOK) ? usb_dev->alternate_interfaces.current : 0;
 
         /* Create and register other pipes than default control (EP 0) */
@@ -388 +392 @@
             usb_dev->descriptors.configuration,
             usb_dev->descriptors.configuration_size,
-            usb_dev->interface_no, (int)alternate_iface,
+            usb_dev->interface_no, usb_dev->alternate_interfaces.current,
             &usb_dev->pipes, &usb_dev->pipes_count);
         if (rc != EOK) {