Commit f496c09c authored Apr 28, 2017 by Jason Gerecke Committed by Jiri Kosina May 05, 2017

HID: wacom: generic: Ignore HID_DG_BATTERYSTRENTH == 0



AES sensors use the value 0 to indicate "not available" rather than
"completely dead". Such values are often sent for dozens of reports
while the pen is being brought into proximity and can cause userspace
to get the wrong impression about the actual battery state.

Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>
Reviewed-by: Ping Cheng <ping.cheng@wacom.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>

parent 37d16019

drivers/hid/wacom_wac.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -2036,6 +2036,8 @@ static void wacom_wac_pen_event(struct hid_device hdev, struct hid_field field
		wacom_wac->hid_data.sense_state = value;
		return;
		case HID_DG_BATTERYSTRENGTH:
		if (value == 0) /* "not available" */
		break;
		value = value * 100 / (field->logical_maximum - field->logical_minimum);
		wacom_wac->hid_data.battery_capacity = value;
		wacom_wac->hid_data.bat_connected = 1;