Module: wine Branch: master Commit: ee4e33ed9acf8a0dc4318d6cb0b5ba6c5cf19c0c URL: https://source.winehq.org/git/wine.git/?a=commit;h=ee4e33ed9acf8a0dc4318d6cb...
Author: Arkadiusz Hiler ahiler@codeweavers.com Date: Tue Feb 23 15:54:07 2021 +0200
winebus.sys: Remove incorrect Report ID from the bus_sdl HID report descriptor.
According to "USB Device Class Definition for Human Interface Devices Firmware Specification Version 1.11" Report ID 0 is reserved and should not be used.
On top of that if Report ID is used anywhere in the descriptor the reports are expected to be prepended with an ID and this is not the case for bus_sdl devices.
It worked only because our current HID parser implementation treats the Report ID 0 as there would be none.
Signed-off-by: Arkadiusz Hiler ahiler@codeweavers.com Signed-off-by: Alexandre Julliard julliard@winehq.org
---
dlls/winebus.sys/bus_sdl.c | 1 - 1 file changed, 1 deletion(-)
diff --git a/dlls/winebus.sys/bus_sdl.c b/dlls/winebus.sys/bus_sdl.c index 8f48b44eb95..bf62f448976 100644 --- a/dlls/winebus.sys/bus_sdl.c +++ b/dlls/winebus.sys/bus_sdl.c @@ -199,7 +199,6 @@ static const BYTE CONTROLLER_TRIGGERS [] = { static const BYTE HAPTIC_RUMBLE[] = { 0x06, 0x00, 0xff, /* USAGE PAGE (vendor-defined) */ 0x09, 0x01, /* USAGE (1) */ - 0x85, 0x00, /* REPORT_ID (0) */ /* padding */ 0x95, 0x02, /* REPORT_COUNT (2) */ 0x75, 0x08, /* REPORT_SIZE (8) */