HID: Send Report ID when numbered reports are sent over the control endpoint.
authorAlan Ott <alan@signal11.us>
Wed, 30 Jun 2010 13:50:36 +0000 (09:50 -0400)
committerJiri Kosina <jkosina@suse.cz>
Sun, 11 Jul 2010 21:13:15 +0000 (23:13 +0200)
The Report ID wasn't sent as part of the payload for reports which were sent
over the control endpoint. This is required by section 8.1 of the HID spec.

Signed-off-by: Alan Ott <alan@signal11.us>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/usbhid/hid-core.c

index 1ebd3244eb857694e905de2162fd1248eac8c1ad..b729c02866798c00ae4d2c77eadba097a79d7f73 100644 (file)
@@ -827,14 +827,21 @@ static int usbhid_output_raw_report(struct hid_device *hid, __u8 *buf, size_t co
                                ret++;
                }
        } else {
+               int skipped_report_id = 0;
+               if (buf[0] == 0x0) {
+                       /* Don't send the Report ID */
+                       buf++;
+                       count--;
+                       skipped_report_id = 1;
+               }
                ret = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
                        HID_REQ_SET_REPORT,
                        USB_DIR_OUT | USB_TYPE_CLASS | USB_RECIP_INTERFACE,
                        ((report_type + 1) << 8) | *buf,
-                       interface->desc.bInterfaceNumber, buf + 1, count - 1,
+                       interface->desc.bInterfaceNumber, buf, count,
                        USB_CTRL_SET_TIMEOUT);
-               /* count also the report id */
-               if (ret > 0)
+               /* count also the report id, if this was a numbered report. */
+               if (ret > 0 && skipped_report_id)
                        ret++;
        }