Bug 217330

Summary:	Broken Logitech unifying battery names in hid-next tree
Product:	Drivers	Reporter:	Bastien Nocera (bugzilla)
Component:	Input Devices	Assignee:	drivers_input-devices
Status:	RESOLVED CODE_FIX
Severity:	normal	CC:	regressions
Priority:	P1
Hardware:	All
OS:	Linux
Kernel Version:		Subsystem:
Regression:	No	Bisected commit-id:

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.