Heads-up: one change merged for -rc8 that might be good to have in the next 6.1.y release
From: Linux regression tracking (Thorsten Leemhuis)
Date: Sat Feb 11 2023 - 07:42:31 EST
Hi Greg. Would be great if you could pick up 690eb7dec72a ("HID:
logitech: Disable hi-res scrolling on USB") for the next 6.1.y release,
as it's fixing a regression I saw multiple people report.
The commit (see below) that was recently merged to mainline and has a
proper stable "Cc: <stable@...>" tag, so I guess you scripts will at
some point pick it up automatically. But I noticed you updated the
stable queue and hour ago and this patch afaics is not in it yet
(despite some other patches being in it that were merged later), so I
thought: just to be sure send a quick heads up.
Ciao, Thorsten
On 09.02.23 19:10, Linux Kernel Mailing List wrote:
> Commit: 690eb7dec72ae52d1d710d14a451844b4d0f4f19
> Parent: ea427a222d8bdf2bc1a8a6da3ebe247f7dced70c
> Refname: refs/heads/master
> Web: https://git.kernel.org/torvalds/c/690eb7dec72ae52d1d710d14a451844b4d0f4f19
> Author: Bastien Nocera <hadess@xxxxxxxxxx>
> AuthorDate: Fri Feb 3 11:18:00 2023 +0100
> Committer: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
> CommitDate: Mon Feb 6 10:58:15 2023 +0100
>
> HID: logitech: Disable hi-res scrolling on USB
>
> On some Logitech mice, such as the G903, and possibly the G403, the HID
> events are generated on a different interface to the HID++ one.
>
> If we enable hi-res through the HID++ interface, the HID interface
> wouldn't know anything about it, and handle the events as if they were
> regular scroll events, making the mouse unusable.
>
> Disable hi-res scrolling on those devices until we implement scroll
> events through HID++.
>
> Signed-off-by: Bastien Nocera <hadess@xxxxxxxxxx>
> Tested-by: Tobias Klausmann <klausman@xxxxxxxxxxxxxxx>
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=216885
> Fixes: 908d325e1665 ("HID: logitech-hidpp: Detect hi-res scrolling support")
> Cc: stable@xxxxxxxxxxxxxxx
> Link: https://lore.kernel.org/r/20230203101800.139380-1-hadess@xxxxxxxxxx
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
> ---
> drivers/hid/hid-logitech-hidpp.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c
> index abf2c95e4d0b0..9c1ee8e91e0ca 100644
> --- a/drivers/hid/hid-logitech-hidpp.c
> +++ b/drivers/hid/hid-logitech-hidpp.c
> @@ -3978,7 +3978,8 @@ static void hidpp_connect_event(struct hidpp_device *hidpp)
> }
>
> hidpp_initialize_battery(hidpp);
> - hidpp_initialize_hires_scroll(hidpp);
> + if (!hid_is_usb(hidpp->hid_dev))
> + hidpp_initialize_hires_scroll(hidpp);
>
> /* forward current battery state */
> if (hidpp->capabilities & HIDPP_CAPABILITY_HIDPP10_BATTERY) {
>