Re: [PATCH AUTOSEL 4.9 03/10] hid: topre: Add driver fixing report descriptor

[Pavel Machek]

Hi!

> From: Harry Stern <harry@xxxxxxxxxxxxxx>
> 
> [ Upstream commit a109d5c45b3d6728b9430716b915afbe16eef27c ]
> 
> The Topre REALFORCE R2 firmware incorrectly reports that interface
> descriptor number 1, input report descriptor 2's events are array events
> rather than variable events. That particular report descriptor is used
> to report keypresses when there are more than 6 keys held at a time.
> This bug prevents events from this interface from being registered
> properly, so only 6 keypresses (from a different interface) can be
> registered at once, rather than full n-key rollover.
> 
> This commit fixes the bug by setting the correct value in a report_fixup
> function.
> 
> The original bug report can be found here:
> Link: https://gitlab.freedesktop.org/libinput/libinput/-/issues/804
> 
> Thanks to Benjamin Tissoires for diagnosing the issue with the report
> descriptor.

Come on, whole new driver to work around hw problem that is not really
serious. Plus this won't really do anything unless people enable it in
config.

This should not be in stable.

Best regards,
								Pavel

> Signed-off-by: Harry Stern <harry@xxxxxxxxxxxxxx>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
> Link: https://lore.kernel.org/r/20220911003614.297613-1-harry@xxxxxxxxxxxxxx
> Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
> ---
>  drivers/hid/Kconfig     |  6 +++++
>  drivers/hid/Makefile    |  1 +
>  drivers/hid/hid-ids.h   |  3 +++
>  drivers/hid/hid-topre.c | 49 +++++++++++++++++++++++++++++++++++++++++

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Attachment: signature.asc
Description: PGP signature