* [PATCH] HID: input: avoid polling stylus battery on Chromebook Pompom
@ 2024-02-23 23:16 Dmitry Torokhov
2024-02-27 14:18 ` Jiri Kosina
0 siblings, 1 reply; 2+ messages in thread
From: Dmitry Torokhov @ 2024-02-23 23:16 UTC (permalink / raw)
To: Jiri Kosina, Benjamin Tissoires
Cc: kenalba, dianders, linux-input, linux-kernel
Internal touchscreen on Trogdor Pompom (AKA Dynabook Chromebook C1)
supports USI stylus. Unfortunately the HID descriptor for the stylus
interface does not contain "Stylus" physical collection, which makes
the kernel to try and pull battery information, resulting in errors.
Apply HID_BATTERY_QUIRK_AVOID_QUERY to the device.
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
drivers/hid/hid-ids.h | 1 +
drivers/hid/hid-input.c | 2 ++
2 files changed, 3 insertions(+)
diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
index e4d2dfd5d253..05f451390a3b 100644
--- a/drivers/hid/hid-ids.h
+++ b/drivers/hid/hid-ids.h
@@ -426,6 +426,7 @@
#define I2C_DEVICE_ID_HP_SPECTRE_X360_14T_EA100_V1 0x2BED
#define I2C_DEVICE_ID_HP_SPECTRE_X360_14T_EA100_V2 0x2BEE
#define I2C_DEVICE_ID_HP_ENVY_X360_15_EU0556NG 0x2D02
+#define I2C_DEVICE_ID_CHROMEBOOK_TROGDOR_POMPOM 0x2F81
#define USB_VENDOR_ID_ELECOM 0x056e
#define USB_DEVICE_ID_ELECOM_BM084 0x0061
diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c
index c8b20d44b147..e03d300d2bac 100644
--- a/drivers/hid/hid-input.c
+++ b/drivers/hid/hid-input.c
@@ -411,6 +411,8 @@ static const struct hid_device_id hid_battery_quirks[] = {
HID_BATTERY_QUIRK_IGNORE },
{ HID_I2C_DEVICE(USB_VENDOR_ID_ELAN, I2C_DEVICE_ID_HP_ENVY_X360_15_EU0556NG),
HID_BATTERY_QUIRK_IGNORE },
+ { HID_I2C_DEVICE(USB_VENDOR_ID_ELAN, I2C_DEVICE_ID_CHROMEBOOK_TROGDOR_POMPOM),
+ HID_BATTERY_QUIRK_AVOID_QUERY },
{}
};
--
2.44.0.rc0.258.g7320e95886-goog
--
Dmitry
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] HID: input: avoid polling stylus battery on Chromebook Pompom
2024-02-23 23:16 [PATCH] HID: input: avoid polling stylus battery on Chromebook Pompom Dmitry Torokhov
@ 2024-02-27 14:18 ` Jiri Kosina
0 siblings, 0 replies; 2+ messages in thread
From: Jiri Kosina @ 2024-02-27 14:18 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Benjamin Tissoires, kenalba, dianders, linux-input, linux-kernel
On Fri, 23 Feb 2024, Dmitry Torokhov wrote:
> Internal touchscreen on Trogdor Pompom (AKA Dynabook Chromebook C1)
> supports USI stylus. Unfortunately the HID descriptor for the stylus
> interface does not contain "Stylus" physical collection, which makes
> the kernel to try and pull battery information, resulting in errors.
>
> Apply HID_BATTERY_QUIRK_AVOID_QUERY to the device.
Applied, thanks Dmitry.
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-02-27 14:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-23 23:16 [PATCH] HID: input: avoid polling stylus battery on Chromebook Pompom Dmitry Torokhov
2024-02-27 14:18 ` Jiri Kosina
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).