Changes in uspace/lib/usb/src/hiddescriptor.c [b9d7965:0a494fe] in mainline

File:

• uspace/lib/usb/src/hiddescriptor.c (modified) (10 diffs)

uspace/lib/usb/src/hiddescriptor.c

@@ -56 +56 @@
 #define USB_HID_UNKNOWN_TAG             -99
 
-usb_hid_report_path_t *usb_hid_report_path_try_insert(usb_hid_report_t *report, usb_hid_report_path_t *cmp_path)
-{
+
+/**
+ * Initialize the report descriptor parser structure
+ *
+ * @param parser Report descriptor parser structure
+ * @return Error code
+ */
+int usb_hid_report_init(usb_hid_report_t *report)
+{
+        if(report == NULL) {
+                return EINVAL;
+        }
+
+        memset(report, 0, sizeof(usb_hid_report_t));
+        list_initialize(&report->reports);
+        list_initialize(&report->collection_paths);
+
+        report->use_report_ids = 0;
+    return EOK;   
+}
+
+int usb_hid_report_append_fields(usb_hid_report_t *report, usb_hid_report_item_t *report_item)
+{
+        usb_hid_report_field_t *field;
+        int i;
+
+
         /* find or append current collection path to the list */
         link_t *path_it = report->collection_paths.next;
@@ -64 +89 @@
                 path = list_get_instance(path_it, usb_hid_report_path_t, link);
                 
-                if(usb_hid_report_compare_usage_path(path, cmp_path, USB_HID_PATH_COMPARE_STRICT) == EOK){
+                if(usb_hid_report_compare_usage_path(path, report_item->usage_path, USB_HID_PATH_COMPARE_STRICT) == EOK){
                         break;
                 }                       
@@ -70 +95 @@
         }
         if(path_it == &report->collection_paths) {
-                path = usb_hid_report_path_clone(cmp_path);                     
+                path = usb_hid_report_path_clone(report_item->usage_path);                      
                 list_append(&path->link, &report->collection_paths);                                    
                 report->collection_paths_count++;
-
-                return path;
-        }
-        else {
-                return list_get_instance(path_it, usb_hid_report_path_t, link);
-        }
-}
-
-/**
- * Initialize the report descriptor parser structure
- *
- * @param parser Report descriptor parser structure
- * @return Error code
- */
-int usb_hid_report_init(usb_hid_report_t *report)
-{
-        if(report == NULL) {
-                return EINVAL;
-        }
-
-        memset(report, 0, sizeof(usb_hid_report_t));
-        list_initialize(&report->reports);
-        list_initialize(&report->collection_paths);
-
-        report->use_report_ids = 0;
-    return EOK;   
-}
-
-int usb_hid_report_append_fields(usb_hid_report_t *report, usb_hid_report_item_t *report_item)
-{
-        usb_hid_report_field_t *field;
-        int i;
+        }
 
         for(i=0; i<report_item->usages_count; i++){
@@ -110 +104 @@
         }
 
-        usb_hid_report_path_t *path = report_item->usage_path;  
+        
         for(i=0; i<report_item->count; i++){
 
@@ -118 +112 @@
 
                 /* fill the attributes */               
+                field->collection_path = path;
                 field->logical_minimum = report_item->logical_minimum;
                 field->logical_maximum = report_item->logical_maximum;
@@ -173 +168 @@
                 }
                 
-                usb_hid_report_set_last_item(path, USB_HID_TAG_CLASS_GLOBAL, field->usage_page);
-                usb_hid_report_set_last_item(path, USB_HID_TAG_CLASS_LOCAL, field->usage);
-
-                field->collection_path = usb_hid_report_path_try_insert(report, path);
-
                 field->size = report_item->size;
                 field->offset = report_item->offset + (i * report_item->size);
@@ -359 +349 @@
                                         tmp_usage_path = list_get_instance(report_item->usage_path->link.prev, usb_hid_report_usage_path_t, link);
                                         
-                                        usb_hid_report_set_last_item(usage_path, USB_HID_TAG_CLASS_GLOBAL, tmp_usage_path->usage_page);
-                                        usb_hid_report_set_last_item(usage_path, USB_HID_TAG_CLASS_LOCAL, tmp_usage_path->usage);
+                                        usb_hid_report_set_last_item(usage_path, tmp_usage_path->usage_page, tmp_usage_path->usage);
 
                                         usb_hid_report_path_free(report_item->usage_path);
@@ -640 +629 @@
 
                 usb_log_debug("\t\tOFFSET: %X\n", report_item->offset);
-                usb_log_debug("\t\tSIZE: %X\n", report_item->size);                             
+                usb_log_debug("\t\tSIZE: %zu\n", report_item->size);
                 usb_log_debug("\t\tLOGMIN: %d\n", report_item->logical_minimum);
                 usb_log_debug("\t\tLOGMAX: %d\n", report_item->logical_maximum);                
@@ -651 +640 @@
                 usb_log_debug("\t\ttUSAGE: %X\n", report_item->usage);
                 usb_log_debug("\t\tUSAGE PAGE: %X\n", report_item->usage_page);
-                
-                //usb_hid_print_usage_path(report_item->collection_path);
-
-                usb_log_debug("\n");            
+                                                
+//              usb_log_debug("\n");            
 
         }
@@ -679 +666 @@
                 usb_log_debug("Report ID: %d\n", report_des->report_id);
                 usb_log_debug("\tType: %d\n", report_des->type);
-                usb_log_debug("\tLength: %d\n", report_des->bit_length);                
-                usb_log_debug("\tItems: %d\n", report_des->item_length);                
+                usb_log_debug("\tLength: %zu\n", report_des->bit_length);
+                usb_log_debug("\tItems: %zu\n", report_des->item_length);
 
                 usb_hid_descriptor_print_list(&report_des->report_items);