Changes in uspace/lib/usbdev/src/pipesinit.c [9d58539:d93f5afb] in mainline

File:

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

uspace/lib/usbdev/src/pipesinit.c

@@ -148 +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.
  */
@@ -154 +153 @@
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
     usb_standard_interface_descriptor_t *interface,
-    usb_standard_endpoint_descriptor_t *endpoint,
-    usb_device_connection_t *wire)
+    usb_standard_endpoint_descriptor_t *endpoint_desc,
+    usb_dev_session_t *bus_session)
 {
 
@@ -163 +162 @@
 
         /* Actual endpoint number is in bits 0..3 */
-        const usb_endpoint_t ep_no = endpoint->endpoint_address & 0x0F;
+        const usb_endpoint_t ep_no = endpoint_desc->endpoint_address & 0x0F;
 
         const usb_endpoint_description_t description = {
                 /* Endpoint direction is set by bit 7 */
-                .direction = (endpoint->endpoint_address & 128)
+                .direction = (endpoint_desc->endpoint_address & 128)
                     ? USB_DIRECTION_IN : USB_DIRECTION_OUT,
                 /* Transfer type is in bits 0..2 and
                  * the enum values corresponds 1:1 */
-                .transfer_type = endpoint->attributes & 3,
+                .transfer_type = endpoint_desc->attributes & 3,
 
                 /* Get interface characteristics. */
@@ -193 +192 @@
         }
 
-        int rc = usb_pipe_initialize(&ep_mapping->pipe, wire,
-            ep_no, description.transfer_type, endpoint->max_packet_size,
-            description.direction);
+        int rc = usb_pipe_initialize(&ep_mapping->pipe,
+            ep_no, description.transfer_type,
+            uint16_usb2host(endpoint_desc->max_packet_size),
+            description.direction, bus_session);
         if (rc != EOK) {
                 return rc;
@@ -201 +201 @@
 
         ep_mapping->present = true;
-        ep_mapping->descriptor = endpoint;
+        ep_mapping->descriptor = endpoint_desc;
         ep_mapping->interface = interface;
 
@@ -219 +219 @@
     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)
+    const uint8_t *interface_descriptor, usb_dev_session_t *bus_session)
 {
         const uint8_t *descriptor = usb_dp_get_nested_descriptor(parser,
@@ -235 +235 @@
                             (usb_standard_endpoint_descriptor_t *)
                                 descriptor,
-                            (usb_device_connection_t *) parser_data->arg);
+                            bus_session);
                 }
 
@@ -279 +279 @@
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
     const uint8_t *config_descriptor, size_t config_descriptor_size,
-    usb_device_connection_t *connection)
-{
-        assert(connection);
+    usb_dev_session_t *bus_session)
+{
 
         if (config_descriptor == NULL) {
@@ -305 +304 @@
                 .data = config_descriptor,
                 .size = config_descriptor_size,
-                .arg = connection
         };
 
@@ -318 +316 @@
         do {
                 (void) process_interface(mapping, mapping_count,
-                    &dp_parser, &dp_data, interface);
+                    &dp_parser, &dp_data, interface, bus_session);
                 interface = usb_dp_get_sibling_descriptor(&dp_parser, &dp_data,
                     config_descriptor, interface);
@@ -346 +344 @@
                 return EINVAL;
         }
-
-
-        usb_pipe_start_long_transfer(pipe);
 
         uint8_t dev_descr_start[CTRL_PIPE_MIN_PACKET_SIZE];
@@ -366 +361 @@
                 }
         }
-        usb_pipe_end_long_transfer(pipe);
         if (rc != EOK) {
                 return rc;