Changeset 3e23316 in mainline for uspace/drv/bus/usb/usbmast/main.c

Timestamp:

2013-01-26T22:17:49Z (12 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

bb70637

Parents:

a6a5b25

Message:

usbmast: Don't access mapped endpoints directly.

Use provided functions.

File:

• uspace/drv/bus/usb/usbmast/main.c (modified) (3 diffs)

uspace/drv/bus/usb/usbmast/main.c

@@ -150 +150 @@
         unsigned i;
 
+        usb_endpoint_mapping_t *epm_in =
+            usb_device_get_mapped_ep_desc(dev, &bulk_in_ep);
+        usb_endpoint_mapping_t *epm_out =
+            usb_device_get_mapped_ep_desc(dev, &bulk_out_ep);
+        if (!epm_in || !epm_out || !epm_in->present || !epm_out->present) {
+                usb_log_error("Required EPs were not mapped.\n");
+                return ENOENT;
+        }
+
         /* Allocate softstate */
         mdev = usb_device_data_alloc(dev, sizeof(usbmast_dev_t));
@@ -162 +171 @@
             usb_device_get_name(dev));
         usb_log_debug("Bulk in endpoint: %d [%zuB].\n",
-            dev->pipes[BULK_IN_EP].pipe.endpoint_no,
-            dev->pipes[BULK_IN_EP].pipe.max_packet_size);
+            epm_in->pipe.endpoint_no, epm_in->pipe.max_packet_size);
         usb_log_debug("Bulk out endpoint: %d [%zuB].\n",
-            dev->pipes[BULK_OUT_EP].pipe.endpoint_no,
-            dev->pipes[BULK_OUT_EP].pipe.max_packet_size);
+            epm_out->pipe.endpoint_no, epm_out->pipe.max_packet_size);
 
         usb_log_debug("Get LUN count...\n");
@@ -182 +189 @@
                         goto error;
         }
+        mdev->bulk_in_pipe = &epm_in->pipe;
+        mdev->bulk_out_pipe = &epm_out->pipe;
 
         return EOK;