Changes in uspace/drv/uhci-rhd/port.c [6427cf67:97e87de] in mainline

File:

• uspace/drv/uhci-rhd/port.c (modified) (10 diffs)

uspace/drv/uhci-rhd/port.c

@@ -33 +33 @@
  */
 #include <errno.h>
-#include <str_error.h>
 
 #include <usb/usb.h>    /* usb_address_t */
-#include <usb/usbdevice.h>
-#include <usb/hub.h>
-#include <usb/request.h>
+#include <usb/usbdrv.h> /* usb_drv_*     */
 #include <usb/debug.h>
-#include <usb/recognise.h>
 
 #include "port.h"
@@ -60 +56 @@
         port->attached_device = 0;
         port->rh = rh;
-        int rc = usb_hc_connection_initialize_from_device(
-            &port->hc_connection, rh);
-        if (rc != EOK) {
-                usb_log_error("Failed to initialize connection to HC.");
-                return rc;
-        }
+        port->hc_phone = parent_phone;
 
         port->checker = fibril_create(uhci_port_check, port);
@@ -105 +96 @@
 
                 if (port_status & STATUS_CONNECTED_CHANGED) {
-                        int rc = usb_hc_connection_open(
-                            &port_instance->hc_connection);
-                        if (rc != EOK) {
-                                usb_log_error("Failed to connect to HC.");
-                                goto next;
-                        }
-
                         if (port_status & STATUS_CONNECTED) {
                                 /* new device */
@@ -118 +102 @@
                                 uhci_port_remove_device(port_instance);
                         }
-
-                        rc = usb_hc_connection_close(
-                            &port_instance->hc_connection);
-                        if (rc != EOK) {
-                                usb_log_error("Failed to disconnect from HC.");
-                                goto next;
-                        }
                 }
-        next:
                 async_usleep(port_instance->wait_period_usec);
         }
@@ -135 +111 @@
 {
         assert(port);
-        assert(usb_hc_connection_is_opened(&port->hc_connection));
+        assert(port->hc_phone);
 
         usb_log_info("Adding new device on port %d.\n", port->number);
 
         /* get address of the future device */
-        const usb_address_t usb_address = usb_hc_request_address(
-            &port->hc_connection, true);
+        const usb_address_t usb_address = usb_drv_request_address(port->hc_phone);
 
         if (usb_address <= 0) {
@@ -151 +126 @@
 
         /* get default address */
-        int ret = usb_hc_reserve_default_address(&port->hc_connection, true);
+        int ret = usb_drv_reserve_default_address(port->hc_phone);
         if (ret != EOK) {
                 usb_log_error("Failed to reserve default address on port %d.\n",
                     port->number);
-                int ret2 = usb_hc_unregister_device(&port->hc_connection,
-                    usb_address);
+                int ret2 =
+                  usb_drv_release_address(port->hc_phone, usb_address);
                 if (ret2 != EOK) {
                         usb_log_fatal("Failed to return requested address on port %d.\n",
@@ -197 +172 @@
         }
 
-        /*
-         * Initialize connection to the device.
-         */
-        /* FIXME: check for errors. */
-        usb_device_connection_t new_dev_connection;
-        usb_endpoint_pipe_t new_dev_ctrl_pipe;
-        usb_device_connection_initialize_on_default_address(
-            &new_dev_connection, &port->hc_connection);
-        usb_endpoint_pipe_initialize_default_control(&new_dev_ctrl_pipe,
-            &new_dev_connection);
-
-        /*
-         * Assign new address to the device. This function updates
-         * the backing connection to still point to the same device.
-         */
-        /* FIXME: check for errors. */
-        usb_endpoint_pipe_start_session(&new_dev_ctrl_pipe);
-        ret = usb_request_set_address(&new_dev_ctrl_pipe, usb_address);
-        usb_endpoint_pipe_end_session(&new_dev_ctrl_pipe);
+        /* assign address to device */
+        ret = usb_drv_req_set_address(port->hc_phone, 0, usb_address);
 
         if (ret != EOK) { /* address assigning went wrong */
                 usb_log_error("Failed(%d) to assign address to the device.\n", ret);
                 uhci_port_set_enabled(port, false);
-                int release = usb_hc_release_default_address(&port->hc_connection);
+                int release = usb_drv_release_default_address(port->hc_phone);
                 if (release != EOK) {
                         usb_log_error("Failed to release default address on port %d.\n",
@@ -234 +192 @@
 
         /* release default address */
-        ret = usb_hc_release_default_address(&port->hc_connection);
+        ret = usb_drv_release_default_address(port->hc_phone);
         if (ret != EOK) {
                 usb_log_error("Failed to release default address on port %d.\n",
@@ -246 +204 @@
         assert(port->attached_device == 0);
 
-        ret = usb_device_register_child_in_devman(new_dev_connection.address,
-            new_dev_connection.hc_handle, port->rh, &port->attached_device);
+        ret = usb_drv_register_child_in_devman(port->hc_phone, port->rh,
+          usb_address, &port->attached_device);
 
         if (ret != EOK) { /* something went wrong */
@@ -257 +215 @@
                 port->number, usb_address, port->attached_device);
 
-        /*
-         * Register the device in the host controller.
-         */
-        usb_hc_attached_device_t new_device = {
-                .address = new_dev_connection.address,
-                .handle = port->attached_device
-        };
-
-        ret = usb_hc_register_device(&port->hc_connection, &new_device);
+        ret =
+          usb_drv_bind_address(port->hc_phone, usb_address, port->attached_device);
         // TODO: proper error check here
         assert(ret == EOK);