Changeset 574f276 in mainline for uspace/lib/usbhid/src/hidparser.c

Timestamp:

2011-05-27T17:37:21Z (14 years ago)

Author:

Matej Klonfar <maklf@…>

Branches:

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

Children:

4967c1e

Parents:

0c904a3

Message:

parsing of customer controls bug fixed

File:

• uspace/lib/usbhid/src/hidparser.c (modified) (2 diffs)

uspace/lib/usbhid/src/hidparser.c

@@ -258 +258 @@
                                 foo = data + i;
                                 mask =  ((1 << (item->size-part_size))-1);
-                                value = (*foo & mask) << part_size;
+                                value = (*foo & mask);
                         }
                         else if(i == ((offset+item->size-1)/8)){
                                 // the lower one
                                 foo = data + i;
-                                mask =  ((1 << part_size)-1) << (8-part_size);
-                                value += ((*foo & mask) >> (8-part_size));
+                                mask = ((1 << (item->size - part_size)) - 1) 
+                                        << (8 - (item->size - part_size));
+
+                                value = (((*foo & mask) >> (8 - 
+                                    (item->size - part_size))) << part_size ) 
+                                    + value;
                         }
                         else {
-                                value = value << 8;
-                                value += *(data + 1);
+                                value = (*(data + 1) << (part_size + 8)) + value;
                                 part_size += 8;
                         }
@@ -386 +389 @@
                 report_item = list_get_instance(item, usb_hid_report_field_t, link);
 
-                if(USB_HID_ITEM_FLAG_VARIABLE(report_item->item_flags) == 0) {
-                                        
-                        // array
-                        value = usb_hid_translate_data_reverse(report_item, 
-                                report_item->value);
-
-                        offset = report_item->offset;
-                        length = report_item->size;
-                }
-                else {
-                        // variable item
-                        value  = usb_hid_translate_data_reverse(report_item, 
-                                report_item->value);
-
-                        offset = report_item->offset;
-                        length = report_item->size;
-                }
-
+                value = usb_hid_translate_data_reverse(report_item, 
+                        report_item->value);
+
+                offset = report_des->bit_length - report_item->offset - 1;
+                length = report_item->size;
+                
                 usb_log_debug("\ttranslated value: %x\n", value);