linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hansg@kernel.org>
To: Jiri Kosina <jikos@kernel.org>,
	"Mario Limonciello (AMD)" <superm1@kernel.org>,
	Dmitry Torokhov <dtor@google.com>
Cc: mario.limonciello@amd.com, bentiss@kernel.org,
	"André Barata" <andretiagob@protonmail.com>,
	linux-input@vger.kernel.org
Subject: Re: [PATCH] HID: hid-input: Extend Elan ignore battery quirk to USB
Date: Fri, 31 Oct 2025 10:13:19 +0100	[thread overview]
Message-ID: <17f02b01-c71e-4e2e-9e91-757731f3fc2e@kernel.org> (raw)
In-Reply-To: <s0qn1098-s856-1942-48q7-n3691sn109qs@xreary.bet>

Hi Jiri,

On 31-Oct-25 10:07 AM, Jiri Kosina wrote:
> On Thu, 30 Oct 2025, Mario Limonciello (AMD) wrote:
> 
>> USB Elan devices have the same problem as the I2C ones with a fake
>> battery device showing up.
>>
>> Reviewed-by: Hans de Goede <hansg@kernel.org>
>> Reported-by: André Barata <andretiagob@protonmail.com>
>> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220722
>> Signed-off-by: Mario Limonciello (AMD) <superm1@kernel.org>
> 
> Now applied.
> 
> We'll have to come up with something more sophisticated once/if Elan ever 
> starts producing devices with real battery ...

Actually the provided HID battery is intended to be for
when a stylus is used and to report the stylus battery
values then.

There is an email thread somewhere with some of the ChromeOS folks
talking about dropping the ELAN quirk for I2C touchscreens and
indeed replacing it with something more sophisticated. IIRC
the ChromeOS folks mentioned they would work on / provide patches.

+To: Dmitry, Dmitry do you have any input on the ChromeOs issue ?

In the mean time I do believe these quirks are the lesser of
2 evils. Most people don't have a stylus (which is typically not
bundled with these devices) and even if people have a stylus
not having battery reporting for that is less of a problem then
the false positive low-battery *notifications* which e.g. GNOME
shows due to the battery reporting 0% (rather then not present)
when there is no stylus.

Regards,

Hans



  reply	other threads:[~2025-10-31  9:13 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-30 16:06 [PATCH] HID: hid-input: Extend Elan ignore battery quirk to USB Mario Limonciello (AMD)
2025-10-31  9:07 ` Jiri Kosina
2025-10-31  9:13   ` Hans de Goede [this message]
2025-10-31 14:47     ` Dmitry Torokhov
2025-11-06 19:10       ` Dmitry Torokhov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=17f02b01-c71e-4e2e-9e91-757731f3fc2e@kernel.org \
    --to=hansg@kernel.org \
    --cc=andretiagob@protonmail.com \
    --cc=bentiss@kernel.org \
    --cc=dtor@google.com \
    --cc=jikos@kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=mario.limonciello@amd.com \
    --cc=superm1@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).