Bug 217330 - Broken Logitech unifying battery names in hid-next tree
Summary: Broken Logitech unifying battery names in hid-next tree
Status: RESOLVED CODE_FIX
Alias: None
Product: Drivers
Classification: Unclassified
Component: Input Devices (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: drivers_input-devices
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-12 13:42 UTC by Bastien Nocera
Modified: 2023-06-25 08:34 UTC (History)
1 user (show)

See Also:
Kernel Version:
Subsystem:
Regression: No
Bisected commit-id:


Attachments

Description Bastien Nocera 2023-04-12 13:42:29 UTC
Unifying devices with long names have the wrong battery names.

For example, my K750 keyboard should read "Wireless Solar Keyboard K750", but instead in udev:
POWER_SUPPLY_MODEL_NAME=Wireless Solar KWireless Sol

Wireless Solar Keyboard K750
Wireless Solar KWireless Sol

Note how the strings are the same length but loops around at offset 16.

Also happens with a M705 mouse:
Marathon Mouse/PMarathon Mouse/PMara
Marathon Mouse/Performance Plus M705

Doesn't happen with 6.2, so might be a regression. Needs bisecting.
Comment 1 The Linux kernel's regression tracker (Thorsten Leemhuis) 2023-05-16 12:38:50 UTC
(In reply to Bastien Nocera from comment #0)

> Doesn't happen with 6.2, so might be a regression. Needs bisecting.

Does that still happen? Did you ever get down to bisecting? There are two other issues and I wonder if they are related [you got a mail about one of them a few minutes ago, but wanted to query the status of this report as well while at it]

https://lore.kernel.org/all/eeb19342-3499-a1fb-388f-d4670472b16c@gmail.com/
https://bugzilla.kernel.org/show_bug.cgi?id=217412
Comment 2 Bastien Nocera 2023-06-25 08:34:40 UTC
The bug was reported as fixed in 6.3.7, most likely by the logic fixes in the command sending.

Note You need to log in before you can comment on or make changes to this bug.