Skip to content

Commit 0187c08

Browse files
dtorJiri Kosina
authored andcommitted
HID: hid-input: only ignore 0 battery events for digitizers
Commit 581c448 ("HID: input: map digitizer battery usage") added handling of battery events for digitizers (typically for batteries presented in stylii). Digitizers typically report correct battery levels only when stylus is actively touching the surface, and in other cases they may report battery level of 0. To avoid confusing consumers of the battery information the code was added to filer out reports with 0 battery levels. However there exist other kinds of devices that may legitimately report 0 battery levels. Fix this by filtering out 0-level reports only for digitizer usages, and continue reporting them for other kinds of devices (Smart Batteries, etc). Reported-by: 卢国宏 <luguohong@xiaomi.com> Fixes: 581c448 ("HID: input: map digitizer battery usage") Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> Signed-off-by: Jiri Kosina <jkosina@suse.com>
1 parent a7cdc20 commit 0187c08

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

drivers/hid/hid-input.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -635,7 +635,10 @@ static void hidinput_update_battery(struct hid_device *dev, unsigned int usage,
635635
return;
636636
}
637637

638-
if (value == 0 || value < dev->battery_min || value > dev->battery_max)
638+
if ((usage & HID_USAGE_PAGE) == HID_UP_DIGITIZER && value == 0)
639+
return;
640+
641+
if (value < dev->battery_min || value > dev->battery_max)
639642
return;
640643

641644
capacity = hidinput_scale_battery_capacity(dev, value);

0 commit comments

Comments
 (0)