Page MenuHomeFreeBSD

Add EVDEV_SUPPORT to acpi_button and acpi_lid
ClosedPublic

Authored by hselasky on Feb 27 2020, 10:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 3, 8:32 PM
Unknown Object (File)
Oct 23 2024, 1:13 PM
Unknown Object (File)
Oct 20 2024, 9:14 AM
Unknown Object (File)
Oct 20 2024, 8:07 AM
Unknown Object (File)
Sep 21 2024, 8:39 PM
Unknown Object (File)
Sep 16 2024, 9:57 PM
Unknown Object (File)
Sep 16 2024, 9:57 PM
Unknown Object (File)
Sep 16 2024, 9:57 PM
Subscribers

Details

Summary

On Linux, these ACPI devices are exposed as evdev. This is useful for a bunch of things — libinput disables touchpads when the lid is closed, various DEs can show power-off dialogs when the power is pressed, etc. (While the latter should be doable with devd, a cross-platform solution is nicer.)

Test Plan

Tested libinput's touchpad disablement on my Pixelbook.

Couldn't test acpi_button: the Pixelbook power button is part of the *keyboard*, while on my desktop the button is not getting to that code for some reason.. (no devd event either)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/dev/acpica/acpi_lid.c
119 ↗(On Diff #68919)

Can you test this w/ suspend and resume.

hselasky updated this revision to Diff 69058.
hselasky edited reviewers, added: val_packett.cool; removed: hselasky.

Mostly style nits and an init order fix in case of error.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 2 2020, 9:45 AM
This revision was automatically updated to reflect the committed changes.