Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HID: wacom: Remove AES power_supply after extended inactivity #405

Conversation

flying-elephant
Copy link
Contributor

Even if a user does not use their AES pen for an extended period, the battery power supply attributes continue to exist. This results in the desktop showing battery status for a pen that is no longer in use and which may in fact be in a different state (e.g. the user may be charging the pen).
To avoid confusion and ensure userspace has an accurate view of the battery state, this patch automatically removes the power_supply after 30 minutes of inactivity.

Reviewed-by: Jason Gerecke [email protected]
Reviewed-by: Aaron Skomra [email protected]
Reviewed-by: Josh Dickens [email protected]
Link: https://lore.kernel.org/r/[email protected]
[[email protected]: Imported into input-wacom (fd2a9b29dc9c)]
Signed-off-by: Tatsunosuke Tobita [email protected]

Even if a user does not use their AES pen for an extended period,
the battery power supply attributes continue to exist.
This results in the desktop showing battery status for a pen
that is no longer in use and which may in fact be in a different
state (e.g. the user may be charging the pen).
To avoid confusion and ensure userspace has an accurate view
of the battery state, this patch automatically removes
the power_supply after 30 minutes of inactivity.

Signed-off-by: Tatsunosuke Tobita <[email protected]>
Reviewed-by: Jason Gerecke <[email protected]>
Reviewed-by: Aaron Skomra <[email protected]>
Reviewed-by: Josh Dickens <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Benjamin Tissoires <[email protected]>
[[email protected]: Imported into input-wacom (fd2a9b29dc9c)]
Signed-off-by: Tatsunosuke Tobita <[email protected]>
@Pinglinux Pinglinux merged commit 4d7449a into linuxwacom:master Dec 22, 2023
12 of 13 checks passed
@Pinglinux
Copy link
Member

Patch merged. Thank you, Tats!

@flying-elephant flying-elephant deleted the import_changes-for-AES-battery-handler branch January 4, 2024 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants