Re: Re: [PATCH v2] HID: input: fix a4tech horizontal wheel custom usage
From: Heorhi Valakhanovich <hidden>
Date: 2019-08-25 20:57:39
Also in:
lkml
From: Heorhi Valakhanovich <hidden>
Date: 2019-08-25 20:57:39
Also in:
lkml
On 8/5/19 3:38 PM, Jiri Kosina wrote:
On Thu, 1 Aug 2019, Nicolas Saenz Julienne wrote:quoted
quoted
Some a4tech mice use the 'GenericDesktop.00b8' usage to inform whether the previous wheel report was horizontal or vertical. Before c01908a14bf73 ("HID: input: add mapping for "Toggle Display" key") this usage was being mapped to 'Relative.Misc'. After the patch it's simply ignored (usage->type == 0 & usage->code == 0). Which ultimately makes hid-a4tech ignore the WHEEL/HWHEEL selection event, as it has no usage->type. We shouldn't rely on a mapping for that usage as it's nonstandard and doesn't really map to an input event. So we bypass the mapping and make sure the custom event handling properly handles both reports. Fixes: c01908a14bf73 ("HID: input: add mapping for "Toggle Display" key") Signed-off-by: Nicolas Saenz Julienne <redacted> ---It would be nice for this patch not to get lost. It fixes issues both repoted on opensuse and fedora.Sorry for the delay. I've now queued the patch. Thanks for fixing this,
Any plans to apply this patch on top of 5.2.* and 5.1.* ?