Changes in uspace/lib/usb/src/hidreport.c [4125b7d:175ad13e] in mainline

File:

• uspace/lib/usb/src/hidreport.c (modified) (7 diffs)

uspace/lib/usb/src/hidreport.c

@@ -80 +80 @@
                 d = usb_dp_get_sibling_descriptor(&parser, &parser_data, 
                     dev->descriptors.configuration, d);
-                ++i;
         }
         
@@ -109 +108 @@
         
         if (*d != sizeof(usb_standard_hid_descriptor_t)) {
-                usb_log_error("HID descriptor has wrong size (%u, expected %zu"
+                usb_log_error("HID descriptor hass wrong size (%u, expected %u"
                     ")\n", *d, sizeof(usb_standard_hid_descriptor_t));
                 return EINVAL;
@@ -119 +118 @@
         uint16_t length =  hid_desc->report_desc_info.length;
         size_t actual_size = 0;
+        
+        /*
+         * Start session for the control transfer.
+         */
+        int sess_rc = usb_pipe_start_session(&dev->ctrl_pipe);
+        if (sess_rc != EOK) {
+                usb_log_warning("Failed to start a session: %s.\n",
+                    str_error(sess_rc));
+                return sess_rc;
+        }
 
         /*
@@ -149 +158 @@
                 free(*report_desc);
                 *report_desc = NULL;
-                usb_log_error("Report descriptor has wrong size (%zu, expected "
+                usb_log_error("Report descriptor has wrong size (%u, expected "
                     "%u)\n", actual_size, length);
                 return EINVAL;
         }
         
+        /*
+         * End session for the control transfer.
+         */
+        sess_rc = usb_pipe_end_session(&dev->ctrl_pipe);
+        if (sess_rc != EOK) {
+                usb_log_warning("Failed to end a session: %s.\n",
+                    str_error(sess_rc));
+                free(*report_desc);
+                *report_desc = NULL;
+                return sess_rc;
+        }
+        
         *size = length;
         
@@ -164 +185 @@
 
 int usb_hid_process_report_descriptor(usb_device_t *dev, 
-    usb_hid_report_parser_t *parser)
+    usb_hid_report_t *report)
 {
-        if (dev == NULL || parser == NULL) {
+        if (dev == NULL || report == NULL) {
                 usb_log_error("Failed to process Report descriptor: wrong "
                     "parameters given.\n");
@@ -189 +210 @@
         assert(report_desc != NULL);
         
-        rc = usb_hid_parse_report_descriptor(parser, report_desc, report_size);
+        rc = usb_hid_parse_report_descriptor(report, report_desc, report_size);
         if (rc != EOK) {
                 usb_log_error("Problem parsing Report descriptor: %s.\n",
@@ -197 +218 @@
         }
         
-        usb_hid_descriptor_print(parser);
+        usb_hid_descriptor_print(report);
         free(report_desc);