Changes in uspace/lib/usbdev/src/pipesinit.c [4e732f1a:1cf26ab] in mainline

File:

• uspace/lib/usbdev/src/pipesinit.c (modified) (12 diffs)

uspace/lib/usbdev/src/pipesinit.c

@@ -34 +34 @@
  *
  */
+#include <usb/usb.h>
 #include <usb/dev/pipes.h>
 #include <usb/dev/dp.h>
 #include <usb/dev/request.h>
-#include <usb/usb.h>
-#include <usb/descriptor.h>
-
+#include <errno.h>
 #include <assert.h>
-#include <errno.h>
 
 #define DEV_DESCR_MAX_PACKET_SIZE_OFFSET 7
@@ -150 +148 @@
  * @param interface Interface descriptor under which belongs the @p endpoint.
  * @param endpoint Endpoint descriptor.
+ * @param wire Connection backing the endpoint pipes.
  * @return Error code.
  */
@@ -156 +155 @@
     usb_standard_interface_descriptor_t *interface,
     usb_standard_endpoint_descriptor_t *endpoint_desc,
-    usb_dev_session_t *bus_session)
+    usb_device_connection_t *wire)
 {
 
@@ -191 +190 @@
 
         if (ep_mapping->present) {
-                return EEXISTS;
-        }
-
-        int rc = usb_pipe_initialize(&ep_mapping->pipe,
+                return EEXIST;
+        }
+
+        int rc = usb_pipe_initialize(&ep_mapping->pipe, wire,
             ep_no, description.transfer_type,
-            ED_MPS_PACKET_SIZE_GET(
-                uint16_usb2host(endpoint_desc->max_packet_size)),
-            description.direction,
-            ED_MPS_TRANS_OPPORTUNITIES_GET(
-                uint16_usb2host(endpoint_desc->max_packet_size)), bus_session);
+            uint16_usb2host(endpoint_desc->max_packet_size),
+            description.direction);
         if (rc != EOK) {
                 return rc;
@@ -224 +220 @@
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
     const usb_dp_parser_t *parser, const usb_dp_parser_data_t *parser_data,
-    const uint8_t *interface_descriptor, usb_dev_session_t *bus_session)
+    const uint8_t *interface_descriptor)
 {
         const uint8_t *descriptor = usb_dp_get_nested_descriptor(parser,
@@ -240 +236 @@
                             (usb_standard_endpoint_descriptor_t *)
                                 descriptor,
-                            bus_session);
+                            (usb_device_connection_t *) parser_data->arg);
                 }
 
@@ -284 +280 @@
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
     const uint8_t *config_descriptor, size_t config_descriptor_size,
-    usb_dev_session_t *bus_session)
-{
+    usb_device_connection_t *connection)
+{
+        assert(connection);
 
         if (config_descriptor == NULL) {
@@ -309 +306 @@
                 .data = config_descriptor,
                 .size = config_descriptor_size,
+                .arg = connection
         };
 
@@ -321 +319 @@
         do {
                 (void) process_interface(mapping, mapping_count,
-                    &dp_parser, &dp_data, interface, bus_session);
+                    &dp_parser, &dp_data, interface);
                 interface = usb_dp_get_sibling_descriptor(&dp_parser, &dp_data,
                     config_descriptor, interface);
@@ -342 +340 @@
 {
         assert(pipe);
-        assert(DEV_DESCR_MAX_PACKET_SIZE_OFFSET < CTRL_PIPE_MIN_PACKET_SIZE);
+        static_assert(DEV_DESCR_MAX_PACKET_SIZE_OFFSET < CTRL_PIPE_MIN_PACKET_SIZE);
 
         if ((pipe->direction != USB_DIRECTION_BOTH) ||
@@ -349 +347 @@
                 return EINVAL;
         }
+
+
+        usb_pipe_start_long_transfer(pipe);
 
         uint8_t dev_descr_start[CTRL_PIPE_MIN_PACKET_SIZE];
@@ -366 +367 @@
                 }
         }
+        usb_pipe_end_long_transfer(pipe);
         if (rc != EOK) {
                 return rc;