On Tue, Nov 19, 2024, at 12:17 PM, Mario Limonciello wrote:
> Currently there are a number of ASUS products on the market that happen to
> have ACPI objects for amd-pmf to bind to as well as an ACPI platform
> profile provided by asus-wmi.
>
> The ACPI platform profile support created by amd-pmf on these ASUS
> products is "Function 9" which is specifically for "BIOS or EC
> notification" of power slider position. This feature is actively used
> by some designs such as Framework 13 and Framework 16.
>
> On these ASUS designs we keep on quirking more and more of them to turn
> off this notification so that asus-wmi can bind.
>
> This however isn't how Windows works. "Multiple" things are notified for
> the power slider position. This series adjusts Linux to behave similarly.
>
> Multiple drivers can now register an ACPI platform profile and will react
> to set requests.
>
> To avoid chaos, only positions that are common to both drivers are
> accepted when the legacy /sys/firmware/acpi/platform_profile interface
> is used.
>
> This series also adds a new concept of a "custom" profile. This allows
> userspace to discover that there are multiple driver handlers that are
> configured differently.
>
> This series also allows dropping all of the PMF quirks from amd-pmf.
> ---
> v7:
> * Add tags
> * Rebase on pdx86/for-next
> * Armin's various code feedback (see individual patches)
>
> Mario Limonciello (22):
> ACPI: platform-profile: Add a name member to handlers
> platform/x86/dell: dell-pc: Create platform device
> ACPI: platform_profile: Add device pointer into platform profile
> handler
> ACPI: platform_profile: Add platform handler argument to
> platform_profile_remove()
> ACPI: platform_profile: Pass the profile handler into
> platform_profile_notify()
> ACPI: platform_profile: Move sanity check out of the mutex
> ACPI: platform_profile: Move matching string for new profile out of
> mutex
> ACPI: platform_profile: Use guard(mutex) for register/unregister
> ACPI: platform_profile: Use `scoped_cond_guard`
> ACPI: platform_profile: Create class for ACPI platform profile
> ACPI: platform_profile: Add name attribute to class interface
> ACPI: platform_profile: Add choices attribute for class interface
> ACPI: platform_profile: Add profile attribute for class interface
> ACPI: platform_profile: Notify change events on register and
> unregister
> ACPI: platform_profile: Only show profiles common for all handlers
> ACPI: platform_profile: Add concept of a "custom" profile
> ACPI: platform_profile: Make sure all profile handlers agree on
> profile
> ACPI: platform_profile: Check all profile handler to calculate next
> ACPI: platform_profile: Notify class device from
> platform_profile_notify()
> ACPI: platform_profile: Allow multiple handlers
> platform/x86/amd: pmf: Drop all quirks
> Documentation: Add documentation about class interface for platform
> profiles
>
> .../ABI/testing/sysfs-platform_profile | 5 +
> .../userspace-api/sysfs-platform_profile.rst | 31 ++
> drivers/acpi/platform_profile.c | 525 ++++++++++++++----
> .../surface/surface_platform_profile.c | 8 +-
> drivers/platform/x86/acer-wmi.c | 12 +-
> drivers/platform/x86/amd/pmf/Makefile | 2 +-
> drivers/platform/x86/amd/pmf/core.c | 1 -
> drivers/platform/x86/amd/pmf/pmf-quirks.c | 66 ---
> drivers/platform/x86/amd/pmf/pmf.h | 3 -
> drivers/platform/x86/amd/pmf/sps.c | 4 +-
> drivers/platform/x86/asus-wmi.c | 8 +-
> drivers/platform/x86/dell/alienware-wmi.c | 8 +-
> drivers/platform/x86/dell/dell-pc.c | 36 +-
> drivers/platform/x86/hp/hp-wmi.c | 8 +-
> drivers/platform/x86/ideapad-laptop.c | 6 +-
> .../platform/x86/inspur_platform_profile.c | 7 +-
> drivers/platform/x86/thinkpad_acpi.c | 16 +-
> include/linux/platform_profile.h | 9 +-
> 18 files changed, 542 insertions(+), 213 deletions(-)
> delete mode 100644 drivers/platform/x86/amd/pmf/pmf-quirks.c
>
>
> base-commit: c6a2b4fcec5f2d80b0183fae1117f06127584c28
> --
> 2.43.0
For the series
Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
Let me know if it would be useful to run tests on the latest version.
Mark