vserver 1.9.5.x5
[linux-2.6.git] / drivers / usb / input / hid-core.c
index 663a6b7..2280d13 100644 (file)
@@ -94,7 +94,8 @@ static struct hid_field *hid_register_field(struct hid_report *report, unsigned
        memset(field, 0, sizeof(struct hid_field) + usages * sizeof(struct hid_usage)
                + values * sizeof(unsigned));
 
-       report->field[report->maxfield++] = field;
+       field->index = report->maxfield++;
+       report->field[field->index] = field;
        field->usage = (struct hid_usage *)(field + 1);
        field->value = (unsigned *)(field->usage + usages);
        field->report = report;
@@ -219,17 +220,13 @@ static int hid_add_field(struct hid_parser *parser, unsigned report_type, unsign
                dbg("logical range invalid %d %d", parser->global.logical_minimum, parser->global.logical_maximum);
                return -1;
        }
-       usages = parser->local.usage_index;
+       
+       if (!(usages = max_t(int, parser->local.usage_index, parser->global.report_count)))
+               return 0; /* Ignore padding fields */
 
        offset = report->size;
        report->size += parser->global.report_size * parser->global.report_count;
 
-       if (usages < parser->global.report_count)
-               usages = parser->global.report_count;
-
-       if (usages == 0)
-               return 0; /* ignore padding fields */
-
        if ((field = hid_register_field(report, usages, parser->global.report_count)) == NULL)
                return 0;
 
@@ -609,16 +606,16 @@ static u8 *fetch_item(__u8 *start, __u8 *end, struct hid_item *item)
                case 2:
                        if ((end - start) < 2) 
                                return NULL;
-                       item->data.u16 = le16_to_cpu(get_unaligned((__u16*)start));
-                       start = (__u8 *)((__u16 *)start + 1);
+                       item->data.u16 = le16_to_cpu(get_unaligned((__le16*)start));
+                       start = (__u8 *)((__le16 *)start + 1);
                        return start;
 
                case 3:
                        item->size++;
                        if ((end - start) < 4)
                                return NULL;
-                       item->data.u32 = le32_to_cpu(get_unaligned((__u32*)start));
-                       start = (__u8 *)((__u32 *)start + 1);
+                       item->data.u32 = le32_to_cpu(get_unaligned((__le32*)start));
+                       start = (__u8 *)((__le32 *)start + 1);
                        return start;
        }
 
@@ -679,7 +676,7 @@ static struct hid_device *hid_parse_report(__u8 *start, unsigned size)
        parser->device = device;
 
        end = start + size;
-       while ((start = fetch_item(start, end, &item)) != 0) {
+       while ((start = fetch_item(start, end, &item)) != NULL) {
 
                if (item.format != HID_ITEM_FORMAT_SHORT) {
                        dbg("unexpected long global item");
@@ -760,15 +757,15 @@ static __inline__ __u32 s32ton(__s32 value, unsigned n)
 static __inline__ __u32 extract(__u8 *report, unsigned offset, unsigned n)
 {
        report += (offset >> 5) << 2; offset &= 31;
-       return (le64_to_cpu(get_unaligned((__u64*)report)) >> offset) & ((1 << n) - 1);
+       return (le64_to_cpu(get_unaligned((__le64*)report)) >> offset) & ((1 << n) - 1);
 }
 
 static __inline__ void implement(__u8 *report, unsigned offset, unsigned n, __u32 value)
 {
        report += (offset >> 5) << 2; offset &= 31;
-       put_unaligned((get_unaligned((__u64*)report)
+       put_unaligned((get_unaligned((__le64*)report)
                & cpu_to_le64(~((((__u64) 1 << n) - 1) << offset)))
-               | cpu_to_le64((__u64)value << offset), (__u64*)report);
+               | cpu_to_le64((__u64)value << offset), (__le64*)report);
 }
 
 /*
@@ -807,7 +804,11 @@ static void hid_input_field(struct hid_device *hid, struct hid_field *field, __u
        unsigned size = field->report_size;
        __s32 min = field->logical_minimum;
        __s32 max = field->logical_maximum;
-       __s32 value[count]; /* WARNING: gcc specific */
+       __s32 *value;
+
+       value = kmalloc(sizeof(__s32)*count, GFP_ATOMIC);
+       if (!value)
+               return;
 
        for (n = 0; n < count; n++) {
 
@@ -817,7 +818,7 @@ static void hid_input_field(struct hid_device *hid, struct hid_field *field, __u
                        if (!(field->flags & HID_MAIN_ITEM_VARIABLE) /* Ignore report if ErrorRollOver */
                            && value[n] >= min && value[n] <= max
                            && field->usage[value[n] - min].hid == HID_UP_KEYBOARD + 1)
-                               return;
+                               goto exit;
        }
 
        for (n = 0; n < count; n++) {
@@ -847,6 +848,8 @@ static void hid_input_field(struct hid_device *hid, struct hid_field *field, __u
        }
 
        memcpy(field->value, value, count * sizeof(__s32));
+exit:
+       kfree(value);
 }
 
 static int hid_input_report(int type, struct urb *urb, struct pt_regs *regs)
@@ -917,20 +920,24 @@ static void hid_irq_in(struct urb *urb, struct pt_regs *regs)
        int                     status;
 
        switch (urb->status) {
-       case 0:                 /* success */
-               hid_input_report(HID_INPUT_REPORT, urb, regs);
-               break;
-       case -ECONNRESET:       /* unlink */
-       case -ENOENT:
-       case -ESHUTDOWN:
-               return;
-       default:                /* error */
-               dbg("nonzero status in input irq %d", urb->status);
+               case 0:                 /* success */
+                       hid_input_report(HID_INPUT_REPORT, urb, regs);
+                       break;
+               case -ECONNRESET:       /* unlink */
+               case -ENOENT:
+               case -EPERM:
+               case -ESHUTDOWN:        /* unplug */
+               case -EILSEQ:           /* unplug timeout on uhci */
+                       return;
+               case -ETIMEDOUT:        /* NAK */
+                       break;
+               default:                /* error */
+                       warn("input irq status %d received", urb->status);
        }
        
-       status = usb_submit_urb (urb, SLAB_ATOMIC);
+       status = usb_submit_urb(urb, SLAB_ATOMIC);
        if (status)
-               err ("can't resubmit intr, %s-%s/input%d, status %d",
+               err("can't resubmit intr, %s-%s/input%d, status %d",
                                hid->dev->bus->bus_name, hid->dev->devpath,
                                hid->ifnum, status);
 }
@@ -1130,24 +1137,38 @@ static void hid_irq_out(struct urb *urb, struct pt_regs *regs)
 {
        struct hid_device *hid = urb->context;
        unsigned long flags;
+       int unplug = 0;
 
-       if (urb->status)
-               warn("output irq status %d received", urb->status);
+       switch (urb->status) {
+               case 0:                 /* success */
+               case -ESHUTDOWN:        /* unplug */
+               case -EILSEQ:           /* unplug timeout on uhci */
+                       unplug = 1;
+               case -ECONNRESET:       /* unlink */
+               case -ENOENT:
+                       break;
+               default:                /* error */
+                       warn("output irq status %d received", urb->status);
+       }
 
        spin_lock_irqsave(&hid->outlock, flags);
 
-       hid->outtail = (hid->outtail + 1) & (HID_OUTPUT_FIFO_SIZE - 1);
+       if (unplug)
+               hid->outtail = hid->outhead;
+       else
+               hid->outtail = (hid->outtail + 1) & (HID_OUTPUT_FIFO_SIZE - 1);
 
        if (hid->outhead != hid->outtail) {
-               hid_submit_out(hid);
+               if (hid_submit_out(hid)) {
+                       clear_bit(HID_OUT_RUNNING, &hid->iofl);;
+                       wake_up(&hid->wait);
+               }
                spin_unlock_irqrestore(&hid->outlock, flags);
                return;
        }
 
        clear_bit(HID_OUT_RUNNING, &hid->iofl);
-
        spin_unlock_irqrestore(&hid->outlock, flags);
-
        wake_up(&hid->wait);
 }
 
@@ -1159,27 +1180,41 @@ static void hid_ctrl(struct urb *urb, struct pt_regs *regs)
 {
        struct hid_device *hid = urb->context;
        unsigned long flags;
-
-       if (urb->status)
-               warn("ctrl urb status %d received", urb->status);
+       int unplug = 0;
 
        spin_lock_irqsave(&hid->ctrllock, flags);
 
-       if (hid->ctrl[hid->ctrltail].dir == USB_DIR_IN) 
-               hid_input_report(hid->ctrl[hid->ctrltail].report->type, urb, regs);
+       switch (urb->status) {
+               case 0:                 /* success */
+                       if (hid->ctrl[hid->ctrltail].dir == USB_DIR_IN) 
+                               hid_input_report(hid->ctrl[hid->ctrltail].report->type, urb, regs);
+               case -ESHUTDOWN:        /* unplug */
+               case -EILSEQ:           /* unplug timectrl on uhci */
+                       unplug = 1;
+               case -ECONNRESET:       /* unlink */
+               case -ENOENT:
+               case -EPIPE:            /* report not available */
+                       break;
+               default:                /* error */
+                       warn("ctrl urb status %d received", urb->status);
+       }
 
-       hid->ctrltail = (hid->ctrltail + 1) & (HID_CONTROL_FIFO_SIZE - 1);
+       if (unplug)
+               hid->ctrltail = hid->ctrlhead;
+       else
+               hid->ctrltail = (hid->ctrltail + 1) & (HID_CONTROL_FIFO_SIZE - 1);
 
        if (hid->ctrlhead != hid->ctrltail) {
-               hid_submit_ctrl(hid);
+               if (hid_submit_ctrl(hid)) {
+                       clear_bit(HID_CTRL_RUNNING, &hid->iofl);
+                       wake_up(&hid->wait);
+               }
                spin_unlock_irqrestore(&hid->ctrllock, flags);
                return;
        }
 
        clear_bit(HID_CTRL_RUNNING, &hid->iofl);
-
        spin_unlock_irqrestore(&hid->ctrllock, flags);
-
        wake_up(&hid->wait);
 }
 
@@ -1205,7 +1240,8 @@ void hid_submit_report(struct hid_device *hid, struct hid_report *report, unsign
                hid->outhead = head;
 
                if (!test_and_set_bit(HID_OUT_RUNNING, &hid->iofl))
-                       hid_submit_out(hid);
+                       if (hid_submit_out(hid))
+                               clear_bit(HID_OUT_RUNNING, &hid->iofl);
 
                spin_unlock_irqrestore(&hid->outlock, flags);
                return;
@@ -1224,7 +1260,8 @@ void hid_submit_report(struct hid_device *hid, struct hid_report *report, unsign
        hid->ctrlhead = head;
 
        if (!test_and_set_bit(HID_CTRL_RUNNING, &hid->iofl))
-               hid_submit_ctrl(hid);
+               if (hid_submit_ctrl(hid))
+                       clear_bit(HID_CTRL_RUNNING, &hid->iofl);
 
        spin_unlock_irqrestore(&hid->ctrllock, flags);
 }
@@ -1238,8 +1275,10 @@ int hid_wait_io(struct hid_device *hid)
        add_wait_queue(&hid->wait, &wait);
 
        while (timeout && (test_bit(HID_CTRL_RUNNING, &hid->iofl) ||
-                          test_bit(HID_OUT_RUNNING, &hid->iofl)))
+                          test_bit(HID_OUT_RUNNING, &hid->iofl))) {
+               set_current_state(TASK_UNINTERRUPTIBLE);
                timeout = schedule_timeout(timeout);
+       }
 
        set_current_state(TASK_RUNNING);
        remove_wait_queue(&hid->wait, &wait);
@@ -1255,9 +1294,17 @@ int hid_wait_io(struct hid_device *hid)
 static int hid_get_class_descriptor(struct usb_device *dev, int ifnum,
                unsigned char type, void *buf, int size)
 {
-       return usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
-               USB_REQ_GET_DESCRIPTOR, USB_RECIP_INTERFACE | USB_DIR_IN,
-               (type << 8), ifnum, buf, size, HZ * USB_CTRL_GET_TIMEOUT);
+       int result, retries = 4;
+
+       memset(buf,0,size);     // Make sure we parse really received data
+
+       do {
+               result = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
+                               USB_REQ_GET_DESCRIPTOR, USB_RECIP_INTERFACE | USB_DIR_IN,
+                               (type << 8), ifnum, buf, size, HZ * USB_CTRL_GET_TIMEOUT);
+               retries--;
+       } while (result < size && retries);
+       return result;
 }
 
 int hid_open(struct hid_device *hid)
@@ -1276,7 +1323,7 @@ int hid_open(struct hid_device *hid)
 void hid_close(struct hid_device *hid)
 {
        if (!--hid->open)
-               usb_unlink_urb(hid->urbin);
+               usb_kill_urb(hid->urbin);
 }
 
 /*
@@ -1288,7 +1335,7 @@ void hid_init_reports(struct hid_device *hid)
        struct hid_report_enum *report_enum;
        struct hid_report *report;
        struct list_head *list;
-       int err, ret;
+       int err, ret, size;
 
        /*
         * The Set_Idle request is supposed to affect only the
@@ -1311,6 +1358,10 @@ void hid_init_reports(struct hid_device *hid)
        list = report_enum->report_list.next;
        while (list != &report_enum->report_list) {
                report = (struct hid_report *) list;
+               size = ((report->size - 1) >> 3) + 1 + report_enum->numbered;
+               if (size > HID_BUFFER_SIZE) size = HID_BUFFER_SIZE;
+               if (size > hid->urbin->transfer_buffer_length)
+                       hid->urbin->transfer_buffer_length = size;
                hid_submit_report(hid, report, USB_DIR_IN);
                list = list->next;
        }
@@ -1324,12 +1375,14 @@ void hid_init_reports(struct hid_device *hid)
        }
 
        err = 0;
-       while ((ret = hid_wait_io(hid))) {
+       ret = hid_wait_io(hid);
+       while (ret) {
                err |= ret;
                if (test_bit(HID_CTRL_RUNNING, &hid->iofl))
-                       usb_unlink_urb(hid->urbctrl);
+                       usb_kill_urb(hid->urbctrl);
                if (test_bit(HID_OUT_RUNNING, &hid->iofl))
-                       usb_unlink_urb(hid->urbout);
+                       usb_kill_urb(hid->urbout);
+               ret = hid_wait_io(hid);
        }
 
        if (err)
@@ -1404,6 +1457,7 @@ void hid_init_reports(struct hid_device *hid)
 
 #define USB_VENDOR_ID_CYPRESS          0x04b4
 #define USB_DEVICE_ID_CYPRESS_MOUSE    0x0001
+#define USB_DEVICE_ID_CYPRESS_HIDCOM   0x5500
 
 #define USB_VENDOR_ID_BERKSHIRE                0x0c98
 #define USB_DEVICE_ID_BERKSHIRE_PCWD   0x1140
@@ -1423,11 +1477,23 @@ void hid_init_reports(struct hid_device *hid)
 #define USB_VENDOR_ID_GLAB             0x06c2
 #define USB_DEVICE_ID_4_PHIDGETSERVO_30        0x0038
 #define USB_DEVICE_ID_1_PHIDGETSERVO_30        0x0039
+#define USB_DEVICE_ID_8_8_8_IF_KIT     0x0045
+#define USB_DEVICE_ID_0_0_4_IF_KIT     0x0040
+#define USB_DEVICE_ID_0_8_8_IF_KIT     0x0053
 
 #define USB_VENDOR_ID_WISEGROUP                0x0925
 #define USB_DEVICE_ID_1_PHIDGETSERVO_20        0x8101
 #define USB_DEVICE_ID_4_PHIDGETSERVO_20        0x8104
 
+#define USB_VENDOR_ID_CODEMERCS                0x07c0
+#define USB_DEVICE_ID_CODEMERCS_IOW40  0x1500
+#define USB_DEVICE_ID_CODEMERCS_IOW24  0x1501
+#define USB_DEVICE_ID_CODEMERCS_IOW48  0x1502
+#define USB_DEVICE_ID_CODEMERCS_IOW28  0x1503
+
+#define USB_VENDOR_ID_DELORME          0x1163
+#define USB_DEVICE_ID_DELORME_EARTHMATE 0x0100
+
 static struct hid_blacklist {
        __u16 idVendor;
        __u16 idProduct;
@@ -1442,20 +1508,28 @@ static struct hid_blacklist {
        { USB_VENDOR_ID_AIPTEK, USB_DEVICE_ID_AIPTEK_23, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_AIPTEK, USB_DEVICE_ID_AIPTEK_24, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_BERKSHIRE, USB_DEVICE_ID_BERKSHIRE_PCWD, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW40, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW24, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW48, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW28, HID_QUIRK_IGNORE },
+       
        { USB_VENDOR_ID_ESSENTIAL_REALITY, USB_DEVICE_ID_ESSENTIAL_REALITY_P5, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_KBGEAR, USB_DEVICE_ID_KBGEAR_JAMSTUDIO, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_4_PHIDGETSERVO_30, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_1_PHIDGETSERVO_30, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_8_8_8_IF_KIT, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_0_0_4_IF_KIT, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_0_8_8_IF_KIT, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_GRIFFIN, USB_DEVICE_ID_POWERMATE, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_GRIFFIN, USB_DEVICE_ID_SOUNDKNOB, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_MGE, USB_DEVICE_ID_MGE_UPS, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_MGE, USB_DEVICE_ID_MGE_UPS1, HID_QUIRK_IGNORE },
-
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100 + 100, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100 + 200, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100 + 300, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100 + 400, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_ONTRAK, USB_DEVICE_ID_ONTRAK_ADU100 + 500, HID_QUIRK_IGNORE },
-
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_PENPARTNER, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_GRAPHIRE, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_GRAPHIRE + 1, HID_QUIRK_IGNORE },
@@ -1481,10 +1555,9 @@ static struct hid_blacklist {
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_INTUOS2 + 7, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_VOLITO, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WACOM, USB_DEVICE_ID_WACOM_PTU, HID_QUIRK_IGNORE },
-       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_4_PHIDGETSERVO_30, HID_QUIRK_IGNORE },
-       { USB_VENDOR_ID_GLAB, USB_DEVICE_ID_1_PHIDGETSERVO_30, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WISEGROUP, USB_DEVICE_ID_4_PHIDGETSERVO_20, HID_QUIRK_IGNORE },
        { USB_VENDOR_ID_WISEGROUP, USB_DEVICE_ID_1_PHIDGETSERVO_20, HID_QUIRK_IGNORE },
+       
 
        { USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_UC100KM, HID_QUIRK_NOGET },
        { USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_CS124U, HID_QUIRK_NOGET },
@@ -1493,8 +1566,9 @@ static struct hid_blacklist {
        { USB_VENDOR_ID_ATEN, USB_DEVICE_ID_ATEN_4PORTKVMC, HID_QUIRK_NOGET },
        { USB_VENDOR_ID_TANGTOP, USB_DEVICE_ID_TANGTOP_USBPS2, HID_QUIRK_NOGET },
 
-       { USB_VENDOR_ID_A4TECH, USB_DEVICE_ID_A4TECH_WCP32PU, HID_QUIRK_2WHEEL_MOUSE_HACK_BACK },
-       { USB_VENDOR_ID_CYPRESS, USB_DEVICE_ID_CYPRESS_MOUSE, HID_QUIRK_2WHEEL_MOUSE_HACK_EXTRA },
+       { USB_VENDOR_ID_A4TECH, USB_DEVICE_ID_A4TECH_WCP32PU, HID_QUIRK_2WHEEL_MOUSE_HACK_7 },
+       { USB_VENDOR_ID_CYPRESS, USB_DEVICE_ID_CYPRESS_MOUSE, HID_QUIRK_2WHEEL_MOUSE_HACK_5 },
+       { USB_VENDOR_ID_CYPRESS, USB_DEVICE_ID_CYPRESS_HIDCOM, HID_QUIRK_IGNORE },
 
        { USB_VENDOR_ID_ALPS, USB_DEVICE_ID_IBM_GAMEPAD, HID_QUIRK_BADPAD },
        { USB_VENDOR_ID_CHIC, USB_DEVICE_ID_CHIC_GAMEPAD, HID_QUIRK_BADPAD },
@@ -1505,6 +1579,13 @@ static struct hid_blacklist {
        { USB_VENDOR_ID_SAITEK, USB_DEVICE_ID_SAITEK_RUMBLEPAD, HID_QUIRK_BADPAD },
        { USB_VENDOR_ID_TOPMAX, USB_DEVICE_ID_TOPMAX_COBRAPAD, HID_QUIRK_BADPAD },
 
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW40, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW24, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW48, HID_QUIRK_IGNORE },
+       { USB_VENDOR_ID_CODEMERCS, USB_DEVICE_ID_CODEMERCS_IOW28, HID_QUIRK_IGNORE },
+
+       { USB_VENDOR_ID_DELORME, USB_DEVICE_ID_DELORME_EARTHMATE, HID_QUIRK_IGNORE },
+
        { 0, 0 }
 };
 
@@ -1545,8 +1626,8 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
        int n;
 
        for (n = 0; hid_blacklist[n].idVendor; n++)
-               if ((hid_blacklist[n].idVendor == dev->descriptor.idVendor) &&
-                       (hid_blacklist[n].idProduct == dev->descriptor.idProduct))
+               if ((hid_blacklist[n].idVendor == le16_to_cpu(dev->descriptor.idVendor)) &&
+                       (hid_blacklist[n].idProduct == le16_to_cpu(dev->descriptor.idProduct)))
                                quirks = hid_blacklist[n].quirks;
 
        if (quirks & HID_QUIRK_IGNORE)
@@ -1585,7 +1666,7 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
        printk("\n");
 #endif
 
-       if (!(hid = hid_parse_report(rdesc, rsize))) {
+       if (!(hid = hid_parse_report(rdesc, n))) {
                dbg("parsing report descriptor failed");
                kfree(rdesc);
                return NULL;
@@ -1603,26 +1684,27 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
 
                struct usb_endpoint_descriptor *endpoint;
                int pipe;
+               int interval;
 
                endpoint = &interface->endpoint[n].desc;
                if ((endpoint->bmAttributes & 3) != 3)          /* Not an interrupt endpoint */
                        continue;
 
-               if (endpoint->bEndpointAddress & USB_DIR_IN) {
-                       int len;
+               /* handle potential highspeed HID correctly */
+               interval = endpoint->bInterval;
+               if (dev->speed == USB_SPEED_HIGH)
+                       interval = 1 << (interval - 1);
 
+               if (endpoint->bEndpointAddress & USB_DIR_IN) {
                        if (hid->urbin)
                                continue;
                        if (!(hid->urbin = usb_alloc_urb(0, GFP_KERNEL)))
                                goto fail;
                        pipe = usb_rcvintpipe(dev, endpoint->bEndpointAddress);
-                       len = usb_maxpacket(dev, pipe, 0);
-                       if (len > HID_BUFFER_SIZE)
-                               len = HID_BUFFER_SIZE;
-                       usb_fill_int_urb(hid->urbin, dev, pipe, hid->inbuf, len,
-                                        hid_irq_in, hid, endpoint->bInterval);
+                       usb_fill_int_urb(hid->urbin, dev, pipe, hid->inbuf, 0,
+                                        hid_irq_in, hid, interval);
                        hid->urbin->transfer_dma = hid->inbuf_dma;
-                       hid->urbin->transfer_flags |= URB_NO_TRANSFER_DMA_MAP;
+                       hid->urbin->transfer_flags |=(URB_NO_TRANSFER_DMA_MAP | URB_ASYNC_UNLINK);
                } else {
                        if (hid->urbout)
                                continue;
@@ -1630,9 +1712,9 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
                                goto fail;
                        pipe = usb_sndintpipe(dev, endpoint->bEndpointAddress);
                        usb_fill_int_urb(hid->urbout, dev, pipe, hid->outbuf, 0,
-                                         hid_irq_out, hid, 1);
+                                        hid_irq_out, hid, interval);
                        hid->urbout->transfer_dma = hid->outbuf_dma;
-                       hid->urbout->transfer_flags |= URB_NO_TRANSFER_DMA_MAP;
+                       hid->urbout->transfer_flags |= (URB_NO_TRANSFER_DMA_MAP | URB_ASYNC_UNLINK);
                }
        }
 
@@ -1643,8 +1725,8 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
 
        init_waitqueue_head(&hid->wait);
        
-       hid->outlock = SPIN_LOCK_UNLOCKED;
-       hid->ctrllock = SPIN_LOCK_UNLOCKED;
+       spin_lock_init(&hid->outlock);
+       spin_lock_init(&hid->ctrllock);
 
        hid->version = le16_to_cpu(hdesc->bcdHID);
        hid->country = hdesc->bCountryCode;
@@ -1661,10 +1743,12 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
                strcat(hid->name, buf);
                if (usb_string(dev, dev->descriptor.iProduct, buf, 64) > 0)
                        snprintf(hid->name, 64, "%s %s", hid->name, buf);
-       } else if (usb_string(dev, dev->descriptor.iProduct, buf, 128) > 0) {
+       } else if (usb_string(dev, dev->descriptor.iProduct, buf, 64) > 0) {
                        snprintf(hid->name, 128, "%s", buf);
        } else
-               snprintf(hid->name, 128, "%04x:%04x", dev->descriptor.idVendor, dev->descriptor.idProduct);
+               snprintf(hid->name, 128, "%04x:%04x", 
+                        le16_to_cpu(dev->descriptor.idVendor),
+                        le16_to_cpu(dev->descriptor.idProduct));
 
        usb_make_path(dev, buf, 64);
        snprintf(hid->phys, 64, "%s/input%d", buf,
@@ -1682,8 +1766,7 @@ static struct hid_device *usb_hid_configure(struct usb_interface *intf)
                             hid->ctrlbuf, 1, hid_ctrl, hid);
        hid->urbctrl->setup_dma = hid->cr_dma;
        hid->urbctrl->transfer_dma = hid->ctrlbuf_dma;
-       hid->urbctrl->transfer_flags |= (URB_NO_TRANSFER_DMA_MAP
-                               | URB_NO_SETUP_DMA_MAP);
+       hid->urbctrl->transfer_flags |= (URB_NO_TRANSFER_DMA_MAP | URB_NO_SETUP_DMA_MAP | URB_ASYNC_UNLINK);
 
        return hid;
 
@@ -1709,9 +1792,9 @@ static void hid_disconnect(struct usb_interface *intf)
                return;
 
        usb_set_intfdata(intf, NULL);
-       usb_unlink_urb(hid->urbin);
-       usb_unlink_urb(hid->urbout);
-       usb_unlink_urb(hid->urbctrl);
+       usb_kill_urb(hid->urbin);
+       usb_kill_urb(hid->urbout);
+       usb_kill_urb(hid->urbctrl);
 
        if (hid->claimed & HID_CLAIMED_INPUT)
                hidinput_disconnect(hid);
@@ -1783,6 +1866,30 @@ static int hid_probe (struct usb_interface *intf, const struct usb_device_id *id
        return 0;
 }
 
+static int hid_suspend(struct usb_interface *intf, u32 state)
+{
+       struct hid_device *hid = usb_get_intfdata (intf);
+
+       usb_kill_urb(hid->urbin);
+       intf->dev.power.power_state = state;
+       dev_dbg(&intf->dev, "suspend\n");
+       return 0;
+}
+
+static int hid_resume(struct usb_interface *intf)
+{
+       struct hid_device *hid = usb_get_intfdata (intf);
+       int status;
+
+       intf->dev.power.power_state = PM_SUSPEND_ON;
+       if (hid->open)
+               status = usb_submit_urb(hid->urbin, GFP_NOIO);
+       else
+               status = 0;
+       dev_dbg(&intf->dev, "resume status %d\n", status);
+       return status;
+}
+
 static struct usb_device_id hid_usb_ids [] = {
        { .match_flags = USB_DEVICE_ID_MATCH_INT_CLASS,
            .bInterfaceClass = USB_INTERFACE_CLASS_HID },
@@ -1796,6 +1903,8 @@ static struct usb_driver hid_driver = {
        .name =         "usbhid",
        .probe =        hid_probe,
        .disconnect =   hid_disconnect,
+       .suspend =      hid_suspend,
+       .resume =       hid_resume,
        .id_table =     hid_usb_ids,
 };
 
@@ -1819,8 +1928,8 @@ hiddev_init_fail:
 
 static void __exit hid_exit(void)
 {
-       hiddev_exit();
        usb_deregister(&hid_driver);
+       hiddev_exit();
 }
 
 module_init(hid_init);