Changeset b3433a2 in mainline for uspace/drv/bus/usb/usbhub/ports.c

Timestamp:

2011-09-23T10:17:25Z (14 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

bba0f1fc

Parents:

8d3cd30

Message:

usbhub: Create GET_PORT_STATUS request inline isntead of creation function.

File:

• uspace/drv/bus/usb/usbhub/ports.c (modified) (3 diffs)

uspace/drv/bus/usb/usbhub/ports.c

@@ -98 +98 @@
 {
         usb_log_debug("Interrupt at port %zu\n", (size_t) port);
-        //determine type of change
-        //usb_pipe_t *pipe = hub->control_pipe;
-
-        int opResult;
 
         usb_port_status_t status;
-        opResult = get_port_status(&hub->usb_device->ctrl_pipe, port, &status);
+        const int opResult =
+            get_port_status(&hub->usb_device->ctrl_pipe, port, &status);
         if (opResult != EOK) {
                 usb_log_error("Failed to get port %zu status: %s.\n",
@@ -119 +116 @@
 
                 if (device_connected) {
-                        opResult = create_add_device_fibril(hub, port,
+                        const int opResult = create_add_device_fibril(hub, port,
                             usb_port_speed(status));
                         if (opResult != EOK) {
@@ -300 +297 @@
  */
 static int get_port_status(usb_pipe_t *ctrl_pipe, size_t port,
-    usb_port_status_t *status) {
+    usb_port_status_t *status)
+{
         size_t recv_size;
-        usb_device_request_setup_packet_t request;
         usb_port_status_t status_tmp;
-
-        usb_hub_set_port_status_request(&request, port);
-        int rc = usb_pipe_control_read(ctrl_pipe,
-            &request, sizeof (usb_device_request_setup_packet_t),
-            &status_tmp, sizeof (status_tmp), &recv_size);
+        /* USB hub specific GET_PORT_STATUS request. See USB Spec 11.16.2.6 */
+        const usb_device_request_setup_packet_t request = {
+                .request_type = USB_HUB_REQ_TYPE_GET_PORT_STATUS,
+                .request = USB_HUB_REQUEST_GET_STATUS,
+                .value = 0,
+                .index = port,
+                .length = sizeof(usb_port_status_t),
+        };
+
+        const int rc = usb_pipe_control_read(ctrl_pipe,
+            &request, sizeof(usb_device_request_setup_packet_t),
+            &status_tmp, sizeof(status_tmp), &recv_size);
         if (rc != EOK) {
                 return rc;