Re: Re: [PATCH v2] HID: input: fix a4tech horizontal wheel custom usage

From: Heorhi Valakhanovich
Date: Sun Aug 25 2019 - 17:05:04 EST


On 8/5/19 3:38 PM, Jiri Kosina wrote:
On Thu, 1 Aug 2019, Nicolas Saenz Julienne wrote:

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 <nsaenzjulienne@xxxxxxx>
---

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.* ?