Changeset 8f8ae1fd in mainline for uspace/drv/vhc/hub.c

Timestamp:

2010-12-13T09:57:22Z (13 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

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

Children:

7feeb84

Parents:

97e660e

Message:

Better root hub initialization

We connect to ourselves and use standard usb_drv_ requests.

File:

• uspace/drv/vhc/hub.c (modified) (3 diffs)

uspace/drv/vhc/hub.c

@@ -40 +40 @@
 #include <stdlib.h>
 #include <driver.h>
+#include <usb/usbdrv.h>
 
 #include "vhcd.h"
@@ -151 +152 @@
 hub_device_t hub_dev;
 
-static usb_address_t hub_set_address(usbvirt_device_t *hub)
-{
-        usb_address_t new_address;
-        int rc = vhc_iface.request_address(NULL, &new_address);
-        if (rc != EOK) {
-                return rc;
-        }
-        
-        usb_device_request_setup_packet_t setup_packet = {
-                .request_type = 0,
-                .request = USB_DEVREQ_SET_ADDRESS,
-                .index = 0,
-                .length = 0,
-        };
-        setup_packet.value = new_address;
-
-        hub->transaction_setup(hub, 0, &setup_packet, sizeof(setup_packet));
-        hub->transaction_in(hub, 0, NULL, 0, NULL);
-        
-        return new_address;
+/** Register root hub in devman.
+ *
+ * @param arg Host controller device (type <code>device_t *</code>).
+ * @return Error code.
+ */
+static int hub_register_in_devman_fibril(void *arg)
+{
+        device_t *hc_dev = (device_t *) arg;
+
+        int hc = usb_drv_hc_connect(hc_dev, IPC_FLAG_BLOCKING);
+        if (hc < 0) {
+                printf(NAME ": failed to register root hub\n");
+                return hc;
+        }
+
+        usb_drv_reserve_default_address(hc);
+
+        usb_address_t hub_address = usb_drv_request_address(hc);
+        usb_drv_req_set_address(hc, USB_ADDRESS_DEFAULT, hub_address);
+
+        usb_drv_release_default_address(hc);
+
+        devman_handle_t hub_handle;
+        usb_drv_register_child_in_devman(hc, hc_dev, hub_address, &hub_handle);
+        usb_drv_bind_address(hc, hub_address, hub_handle);
+
+        return EOK;
 }
 
@@ -190 +198 @@
         usbvirt_connect_local(&virthub_dev);
         
-        dprintf(1, "virtual hub (%d ports) created", HUB_PORT_COUNT);
-
-        usb_address_t hub_address = hub_set_address(&virthub_dev);
-        if (hub_address < 0) {
-                dprintf(1, "problem changing hub address (%s)",
-                    str_error(hub_address));
-        }
-
-        dprintf(2, "virtual hub address changed to %d", hub_address);
-
-        char *id;
-        int rc = asprintf(&id, "usb&hub");
-        if (rc <= 0) {
+        /*
+         * We need to register the root hub.
+         * This must be done in separate fibril because the device
+         * we are connecting to are ourselves and we cannot connect
+         * before leaving the add_device() function.
+         */
+        fid_t root_hub_registration
+            = fibril_create(hub_register_in_devman_fibril, hc_dev);
+        if (root_hub_registration == 0) {
+                printf(NAME ": failed to register root hub\n");
                 return;
         }
-        devman_handle_t hub_handle;
-        rc = child_device_register_wrapper(hc_dev, "hub", id, 10, &hub_handle);
-        if (rc != EOK) {
-                free(id);
-        }
-
-        vhc_iface.bind_address(NULL, hub_address, hub_handle);  
-
-        dprintf(2, "virtual hub has devman handle %d", (int) hub_handle);
+
+        fibril_add_ready(root_hub_registration);
 }