Changeset b9e3aa3 in mainline for uspace/lib

Timestamp:

2011-05-30T14:13:53Z (15 years ago)

Author:

Lubos Slovak <lubos.slovak@…>

Branches:

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

Children:

f3f9733

Parents:

2002595 (diff), 8357fc9 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Development changes (a lot of stuff)

Location:

uspace/lib

Files:

• drv/generic/driver.c (modified) (1 diff)
• drv/include/remote_pci.h (modified) (1 diff)
• drv/include/remote_usb.h (modified) (1 diff)
• drv/include/remote_usbhc.h (modified) (1 diff)
• drv/include/usbhc_iface.h (modified) (1 diff)
• usbdev/include/usb/dev/driver.h (modified) (1 diff)
• usbdev/src/devdrv.c (modified) (1 diff)
• usbdev/src/hub.c (modified) (1 diff)
• usbhid/src/hiddescriptor.c (modified) (4 diffs)
• usbhid/src/hidparser.c (modified) (9 diffs)
• usbhid/src/hidpath.c (modified) (1 diff)

uspace/lib/drv/generic/driver.c

@@ -405 +405 @@
                                 /* The interface has not such method */
                                 printf("%s: driver_connection_gen error - "
-                                    "invalid interface method (%d).\n",
+                                    "invalid interface method "
+                                    "(index %" PRIun ").\n",
                                     driver->name, iface_method_idx);
                                 async_answer_0(callid, ENOTSUP);

uspace/lib/drv/include/remote_pci.h

@@ -36 +36 @@
 #define LIBDRV_REMOTE_PCI_H_
 
-remote_iface_t remote_pci_iface;
+extern remote_iface_t remote_pci_iface;
 
 #endif

uspace/lib/drv/include/remote_usb.h

@@ -36 +36 @@
 #define LIBDRV_REMOTE_USB_H_
 
-remote_iface_t remote_usb_iface;
+extern remote_iface_t remote_usb_iface;
 
 #endif

uspace/lib/drv/include/remote_usbhc.h

@@ -36 +36 @@
 #define LIBDRV_REMOTE_USBHC_H_
 
-remote_iface_t remote_usbhc_iface;
+extern remote_iface_t remote_usbhc_iface;
 
 #endif

uspace/lib/drv/include/usbhc_iface.h

@@ -212 +212 @@
 /** USB host controller communication interface. */
 typedef struct {
-        int (*reserve_default_address)(ddf_fun_t *, usb_speed_t);
-        int (*release_default_address)(ddf_fun_t *);
         int (*request_address)(ddf_fun_t *, usb_speed_t, usb_address_t *);
         int (*bind_address)(ddf_fun_t *, usb_address_t, devman_handle_t);

uspace/lib/usbdev/include/usb/dev/driver.h

@@ -168 +168 @@
 int usb_device_destroy_pipes(ddf_dev_t *, usb_endpoint_mapping_t *, size_t);
 int usb_device_create(ddf_dev_t *, usb_endpoint_description_t **, usb_device_t **, const char **);
+void usb_device_destroy(usb_device_t *);
 
 size_t usb_interface_count_alternates(uint8_t *, size_t, uint8_t);

uspace/lib/usbdev/src/devdrv.c

@@ -533 +533 @@
 }
 
+/** Destroy instance of a USB device.
+ *
+ * @param dev Device to be destroyed.
+ */
+void usb_device_destroy(usb_device_t *dev)
+{
+        if (dev == NULL) {
+                return;
+        }
+
+        /* Ignore errors and hope for the best. */
+        usb_device_destroy_pipes(dev->ddf_dev, dev->pipes, dev->pipes_count);
+        if (dev->descriptors.configuration != NULL) {
+                free(dev->descriptors.configuration);
+        }
+
+        if (dev->alternate_interfaces != NULL) {
+                if (dev->alternate_interfaces->alternatives != NULL) {
+                        free(dev->alternate_interfaces->alternatives);
+                }
+                free(dev->alternate_interfaces);
+        }
+
+        free(dev);
+}
+
 /**
  * @}

uspace/lib/usbdev/src/hub.c

@@ -331 +331 @@
                 goto leave_release_free_address;
         }
+        
+        usb_hc_connection_close(&hc_conn);
 
         /*

uspace/lib/usbhid/src/hiddescriptor.c

@@ -187 +187 @@
 
                 field = malloc(sizeof(usb_hid_report_field_t));
+                if(field == NULL) {
+                        return ENOMEM;
+                }
+
                 memset(field, 0, sizeof(usb_hid_report_field_t));
                 list_initialize(&field->link);
@@ -216 +220 @@
                         }
                         else {
-                                usage = report_item->usages[
+                                usage = report_item->usages[
                                         report_item->usages_count- 1]; 
                         }
@@ -241 +245 @@
 
                 field->size = report_item->size;
-                
-                size_t offset_byte = (report_item->offset + (i *
-                        report_item->size)) / 8; 
-
-                size_t offset_bit = 8 - ((report_item->offset + (i *
-                        report_item->size)) % 8) - report_item->size;
-
-                field->offset = 8 * offset_byte + offset_bit;
-                if(report_item->id != 0) {
+
+                if(report_item->type == USB_HID_REPORT_TYPE_INPUT) {
+                        int offset = report_item->offset + report_item->size * i;
+                        int field_offset = (offset/8)*8 + (offset/8 + 1) * 8 - 
+                                offset - report_item->size;
+                        if(field_offset < 0) {
+                                field->offset = 0;
+                        }
+                        else {
+                                field->offset = field_offset;
+                        }
+                }
+                else {
+                        field->offset = report_item->offset + (i * report_item->size);
+                }
+
+
+                if(report->use_report_ids != 0) {
                         field->offset += 8;
                         report->use_report_ids = 1;
                 }
+
                 field->item_flags = report_item->item_flags;
 
@@ -322 +336 @@
                                 usb_hid_report_description_t, link);
 
-                if((report_des->report_id == report_id) && 
+                // if report id not set, return the first of the type
+                if(((report_des->report_id == report_id) || (report_id == 0)) && 
                    (report_des->type == type)) { 
                         return report_des;

uspace/lib/usbhid/src/hidparser.c

@@ -153 +153 @@
 
 
-        report_des = usb_hid_report_find_description(report, *report_id, type);
+        report_des = usb_hid_report_find_description(report, *report_id, 
+                type);
+
         if(report_des == NULL) {
                 return EINVAL;
@@ -167 +169 @@
                 if(USB_HID_ITEM_FLAG_CONSTANT(item->item_flags) == 0) {
                         
-                        if(USB_HID_ITEM_FLAG_VARIABLE(item->item_flags) == 0) {
+                        if(USB_HID_ITEM_FLAG_VARIABLE(item->item_flags) == 0){
 
                                 // array
@@ -174 +176 @@
                 
                                 item->usage = USB_HID_EXTENDED_USAGE(
-                                    item->usages[item->value - item->physical_minimum]);
-
-                                item->usage_page = USB_HID_EXTENDED_USAGE_PAGE(
-                                    item->usages[item->value - item->physical_minimum]);
-
-                                usb_hid_report_set_last_item (item->collection_path, 
-                                    USB_HID_TAG_CLASS_GLOBAL, item->usage_page);
-
-                                usb_hid_report_set_last_item (item->collection_path, 
+                                    item->usages[
+                                    item->value - item->physical_minimum]);
+
+                                item->usage_page = 
+                                    USB_HID_EXTENDED_USAGE_PAGE(
+                                    item->usages[
+                                    item->value - item->physical_minimum]);
+
+                                usb_hid_report_set_last_item (
+                                    item->collection_path, 
+                                    USB_HID_TAG_CLASS_GLOBAL, 
+                                    item->usage_page);
+
+                                usb_hid_report_set_last_item (
+                                    item->collection_path, 
                                     USB_HID_TAG_CLASS_LOCAL, item->usage);
                                 
@@ -188 +196 @@
                         else {
                                 // variable item
-                                item->value = usb_hid_translate_data(item, data);                               
-                        }                               
+                                item->value = usb_hid_translate_data(item, 
+                                    data);                              
+                        }                       
                 }
                 list_item = list_item->next;
@@ -213 +222 @@
         
         int32_t value=0;
-        int32_t mask;
-        const uint8_t *foo;
+        int32_t mask=0;
+        const uint8_t *foo=0;
 
         // now only shot tags are allowed
@@ -240 +249 @@
         if((size_t)(offset/8) != (size_t)((offset+item->size-1)/8)) {
                 
-                part_size = ((offset+item->size)%8);
+                part_size = 0;
 
                 size_t i=0;
@@ -246 +255 @@
                         if(i == (size_t)(offset/8)) {
                                 // the higher one
+                                part_size = 8 - (offset % 8);
                                 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;
                         }
                 }
@@ -375 +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);
 
@@ -617 +619 @@
                         return report_des->report_id;
                 }
+
+                report_it = report_it->next;
         }
 

uspace/lib/usbhid/src/hidpath.c

@@ -211 +211 @@
 
         if(report_path->report_id != path->report_id) {
-                return 1;
+                if(path->report_id != 0) {
+                        return 1;
+                }
         }