Changeset 8a121b1 in mainline for uspace/lib/usbdev/src/pipesinit.c

Timestamp:

2011-10-15T11:58:16Z (14 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

7c95d6f5

Parents:

2a5b62b

Message:

usb: Make descriptors readonly.

File:

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

uspace/lib/usbdev/src/pipesinit.c

@@ -68 +68 @@
  * @return Whether the given descriptor is endpoint descriptor.
  */
-static inline bool is_endpoint_descriptor(uint8_t *descriptor)
+static inline bool is_endpoint_descriptor(const uint8_t *descriptor)
 {
         return descriptor[1] == USB_DESCTYPE_ENDPOINT;
@@ -80 +80 @@
  */
 static bool endpoint_fits_description(const usb_endpoint_description_t *wanted,
-    usb_endpoint_description_t *found)
+    const usb_endpoint_description_t *found)
 {
 #define _SAME(fieldname) ((wanted->fieldname) == (found->fieldname))
@@ -120 +120 @@
 static usb_endpoint_mapping_t *find_endpoint_mapping(
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
-    usb_endpoint_description_t *found_endpoint,
+    const usb_endpoint_description_t *found_endpoint,
     int interface_number, int interface_setting)
 {
@@ -160 +160 @@
     usb_device_connection_t *wire)
 {
-        usb_endpoint_description_t description;
 
         /*
@@ -167 +166 @@
 
         /* Actual endpoint number is in bits 0..3 */
-        usb_endpoint_t ep_no = endpoint->endpoint_address & 0x0F;
-
-        /* Endpoint direction is set by bit 7 */
-        description.direction = (endpoint->endpoint_address & 128)
-            ? USB_DIRECTION_IN : USB_DIRECTION_OUT;
-        /* Transfer type is in bits 0..2 and the enum values corresponds 1:1 */
-        description.transfer_type = endpoint->attributes & 3;
-
-        /*
-         * Get interface characteristics.
-         */
-        description.interface_class = interface->interface_class;
-        description.interface_subclass = interface->interface_subclass;
-        description.interface_protocol = interface->interface_protocol;
+        const usb_endpoint_t ep_no = endpoint->endpoint_address & 0x0F;
+
+        const usb_endpoint_description_t description = {
+                /* Endpoint direction is set by bit 7 */
+                .direction = (endpoint->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,
+
+                /* Get interface characteristics. */
+                .interface_class = interface->interface_class,
+                .interface_subclass = interface->interface_subclass,
+                .interface_protocol = interface->interface_protocol,
+        };
 
         /*
@@ -225 +225 @@
     usb_endpoint_mapping_t *mapping, size_t mapping_count,
     usb_dp_parser_t *parser, usb_dp_parser_data_t *parser_data,
-    uint8_t *interface_descriptor)
-{
-        uint8_t *descriptor = usb_dp_get_nested_descriptor(parser,
+    const uint8_t *interface_descriptor)
+{
+        const uint8_t *descriptor = usb_dp_get_nested_descriptor(parser,
             parser_data, interface_descriptor);
 
@@ -322 +322 @@
          * Iterate through all interfaces.
          */
-        uint8_t *interface = usb_dp_get_nested_descriptor(&dp_parser,
+        const uint8_t *interface = usb_dp_get_nested_descriptor(&dp_parser,
             &dp_data, configuration_descriptor);
         if (interface == NULL) {
@@ -329 +329 @@
         do {
                 (void) process_interface(mapping, mapping_count,
-                    &dp_parser, &dp_data,
-                    interface);
+                    &dp_parser, &dp_data, interface);
                 interface = usb_dp_get_sibling_descriptor(&dp_parser, &dp_data,
                     configuration_descriptor, interface);