Changeset bdd8ad2f in mainline for uspace/lib/usbdev/src/pipesio.c

Timestamp:

2011-09-14T20:15:03Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

bbce2c2

Parents:

365e29e2

Message:

usb: Send any setup data during initial opening call.

NOTE: receiving function uses those parameters even if they were not sent, this is rather ugly but it does not matter, because any value that might appear in setup data will be ignored.

File:

• uspace/lib/usbdev/src/pipesio.c (modified) (5 diffs)

uspace/lib/usbdev/src/pipesio.c

@@ -219 +219 @@
          * Make call identifying target USB device and type of transfer.
          */
-        aid_t opening_request = async_send_2(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
-            IPC_M_USBHC_DATA_WRITE, target.packed, NULL);
+        aid_t opening_request = async_send_3(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_DATA_WRITE, target.packed, size, NULL);
         
         if (opening_request == 0) {
@@ -337 +337 @@
             {{ .address = pipe->wire->address, .endpoint = pipe->endpoint_no }};
 
+        assert(setup_buffer_size == 8);
+        uint64_t setup_packet;
+        memcpy(&setup_packet, setup_buffer, 8);
         /*
          * Make call identifying target USB device and control transfer type.
          */
         async_exch_t *exch = async_exchange_begin(pipe->hc_sess);
-        aid_t opening_request = async_send_2(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
-            IPC_M_USBHC_CONTROL_READ, target.packed, NULL);
-        
+        aid_t opening_request = async_send_4(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_DATA_READ, target.packed,
+            (setup_packet & UINT32_MAX), (setup_packet >> 32), NULL);
+
         if (opening_request == 0) {
                 async_exchange_end(exch);
                 return ENOMEM;
         }
-        
-        /*
-         * Send the setup packet.
-         */
-        int rc = async_data_write_start(exch, setup_buffer, setup_buffer_size);
-        if (rc != EOK) {
-                async_exchange_end(exch);
-                pipe_end_transaction(pipe);
-                async_wait_for(opening_request, NULL);
-                return rc;
-        }
-        
+
         /*
          * Retrieve the data.
@@ -484 +477 @@
         const usb_target_t target =
             {{ .address = pipe->wire->address, .endpoint = pipe->endpoint_no }};
+        assert(setup_buffer_size == 8);
+        uint64_t setup_packet;
+        memcpy(&setup_packet, setup_buffer, 8);
 
         /*
@@ -489 +485 @@
          */
         async_exch_t *exch = async_exchange_begin(pipe->hc_sess);
-        aid_t opening_request = async_send_3(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
-            IPC_M_USBHC_CONTROL_WRITE, target.packed,
-            data_buffer_size, NULL);
+        aid_t opening_request = async_send_5(exch, DEV_IFACE_ID(USBHC_DEV_IFACE),
+            IPC_M_USBHC_DATA_WRITE, target.packed, data_buffer_size,
+            (setup_packet & UINT32_MAX), (setup_packet >> 32), NULL);
         
         if (opening_request == 0) {
@@ -498 +494 @@
                 return ENOMEM;
         }
-        
-        /*
-         * Send the setup packet.
-         */
-        int rc = async_data_write_start(exch, setup_buffer, setup_buffer_size);
-        if (rc != EOK) {
-                async_exchange_end(exch);
-                pipe_end_transaction(pipe);
-                async_wait_for(opening_request, NULL);
-                return rc;
-        }
-        
+
         /*
          * Send the data (if any).
          */
         if (data_buffer_size > 0) {
-                rc = async_data_write_start(exch, data_buffer, data_buffer_size);
+                int rc = async_data_write_start(exch, data_buffer, data_buffer_size);
                 
                 /* All data sent, pipe can be released. */