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

Timestamp:

2013-08-02T14:04:51Z (11 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

6fe7683

Parents:

9d15d1b

Message:

Implement and use usb_get_device_handle.

Enables object instantiation on usb interface functions.
avoids forwarding in usbmid driver.

File:

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

uspace/lib/usbdev/src/devdrv.c

@@ -78 +78 @@
         void *driver_data;
 
+        /** Connection to device on USB bus */
         usb_dev_session_t *bus_session;
+        /** devman handle */
+        devman_handle_t handle;
 } usb_device_t;
 
@@ -331 +334 @@
 }
 
+devman_handle_t usb_device_get_devman_handle(usb_device_t *usb_dev)
+{
+        assert(usb_dev);
+        return usb_dev->handle;
+}
+
 const usb_device_descriptors_t *usb_device_descriptors(usb_device_t *usb_dev)
 {
@@ -405 +414 @@
 static int usb_device_init(usb_device_t *usb_dev, ddf_dev_t *ddf_dev,
     const usb_endpoint_description_t **endpoints, const char **errstr_ptr,
-    devman_handle_t handle)
+    devman_handle_t handle, int interface_no)
 {
         assert(usb_dev != NULL);
@@ -413 +422 @@
 
         usb_dev->ddf_dev = ddf_dev;
+        usb_dev->handle = handle;
+        usb_dev->interface_no = interface_no;
         usb_dev->driver_data = NULL;
         usb_dev->descriptors.full_config = NULL;
@@ -419 +430 @@
         usb_dev->pipes = NULL;
 
-        if (ddf_dev)
-                usb_dev->bus_session = usb_dev_connect_to_self(ddf_dev);
-        else
-                usb_dev->bus_session = usb_dev_connect(handle);
+        usb_dev->bus_session = usb_dev_connect(handle);
 
         if (!usb_dev->bus_session) {
@@ -433 +441 @@
         usb_address_t address;
 
-        int rc = usb_dev_get_info(usb_dev,
-            &hc_handle, &address, &usb_dev->interface_no);
+        int rc = usb_dev_get_info(usb_dev, &hc_handle, &address, NULL);
         if (rc != EOK) {
                 usb_dev_disconnect(usb_dev->bus_session);
@@ -503 +510 @@
 }
 
+static int usb_device_get_info(async_sess_t *sess, devman_handle_t *handle,
+        int *iface_no)
+{
+        assert(handle);
+        assert(iface_no);
+        async_exch_t *exch = async_exchange_begin(sess);
+        if (!exch)
+                return EPARTY;
+        int ret = usb_get_device_handle(exch, handle);
+        if (ret == EOK) {
+                ret = usb_get_my_interface(exch, iface_no);
+                if (ret == ENOTSUP) {
+                        *iface_no = -1;
+                        ret = EOK;
+                }
+        }
+        async_exchange_end(exch);
+        return ret;
+}
+
 int usb_device_create_ddf(ddf_dev_t *ddf_dev,
     const usb_endpoint_description_t **desc, const char **err)
@@ -508 +535 @@
         assert(ddf_dev);
         assert(err);
+
+        devman_handle_t h = 0;
+        int iface_no = -1;
+
+        async_sess_t *sess = devman_parent_device_connect(EXCHANGE_ATOMIC,
+            ddf_dev_get_handle(ddf_dev), IPC_FLAG_BLOCKING);
+        const int ret = usb_device_get_info(sess, &h, &iface_no);
+        async_hangup(sess);
+        if (ret != EOK)
+                return ret;
+
         usb_device_t *usb_dev =
             ddf_dev_data_alloc(ddf_dev, sizeof(usb_device_t));
@@ -514 +552 @@
                 return ENOMEM;
         }
-        return usb_device_init(usb_dev, ddf_dev, desc, err, 0);
+        
+        return usb_device_init(usb_dev, ddf_dev, desc, err, h, iface_no);
 }
 
@@ -528 +567 @@
 usb_device_t * usb_device_create(devman_handle_t handle)
 {
+        devman_handle_t h = 0;
+        int iface_no = -1;
+
+        async_sess_t *sess = devman_device_connect(
+            EXCHANGE_ATOMIC, handle, IPC_FLAG_BLOCKING);
+        int ret = usb_device_get_info(sess, &h, &iface_no);
+        async_hangup(sess);
+        if (ret != EOK)
+                return NULL;
+
         usb_device_t *usb_dev = malloc(sizeof(usb_device_t));
         if (!usb_dev)
                 return NULL;
+
         const char* dummy = NULL;
-        const int ret = usb_device_init(usb_dev, NULL, NULL, &dummy, handle);
+        ret = usb_device_init(usb_dev, NULL, NULL, &dummy, handle, iface_no);
         if (ret != EOK) {
                 free(usb_dev);