| Message ID | 20241202055031.8038-17-mario.limonciello@amd.com (mailing list archive) |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Add support for binding ACPI platform profile to multiple drivers | expand |
On Sun, 1 Dec 2024, Mario Limonciello wrote: > When two profile handlers don't agree on the current profile it's ambiguous > what to show to the legacy sysfs interface. > > Add a "custom" profile string that userspace will be able to distinguish > this situation when using the legacy sysfs interface. > > Reviewed-by: Armin Wolf <W_Armin@gmx.de> > Tested-by: Mark Pearson <mpearson-lenovo@squebb.ca> > Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca> > Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> > --- > drivers/acpi/platform_profile.c | 1 + > include/linux/platform_profile.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c > index 40826876006b5..a9cd13c5fd39b 100644 > --- a/drivers/acpi/platform_profile.c > +++ b/drivers/acpi/platform_profile.c > @@ -20,6 +20,7 @@ static const char * const profile_names[] = { > [PLATFORM_PROFILE_BALANCED] = "balanced", > [PLATFORM_PROFILE_BALANCED_PERFORMANCE] = "balanced-performance", > [PLATFORM_PROFILE_PERFORMANCE] = "performance", > + [PLATFORM_PROFILE_CUSTOM] = "custom", > }; > static_assert(ARRAY_SIZE(profile_names) == PLATFORM_PROFILE_LAST); > > diff --git a/include/linux/platform_profile.h b/include/linux/platform_profile.h > index a888fd085c513..0682bb4c57e5d 100644 > --- a/include/linux/platform_profile.h > +++ b/include/linux/platform_profile.h > @@ -23,6 +23,7 @@ enum platform_profile_option { > PLATFORM_PROFILE_BALANCED, > PLATFORM_PROFILE_BALANCED_PERFORMANCE, > PLATFORM_PROFILE_PERFORMANCE, > + PLATFORM_PROFILE_CUSTOM, > PLATFORM_PROFILE_LAST, /*must always be last */ > }; > > Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c index 40826876006b5..a9cd13c5fd39b 100644 --- a/drivers/acpi/platform_profile.c +++ b/drivers/acpi/platform_profile.c @@ -20,6 +20,7 @@ static const char * const profile_names[] = { [PLATFORM_PROFILE_BALANCED] = "balanced", [PLATFORM_PROFILE_BALANCED_PERFORMANCE] = "balanced-performance", [PLATFORM_PROFILE_PERFORMANCE] = "performance", + [PLATFORM_PROFILE_CUSTOM] = "custom", }; static_assert(ARRAY_SIZE(profile_names) == PLATFORM_PROFILE_LAST); diff --git a/include/linux/platform_profile.h b/include/linux/platform_profile.h index a888fd085c513..0682bb4c57e5d 100644 --- a/include/linux/platform_profile.h +++ b/include/linux/platform_profile.h @@ -23,6 +23,7 @@ enum platform_profile_option { PLATFORM_PROFILE_BALANCED, PLATFORM_PROFILE_BALANCED_PERFORMANCE, PLATFORM_PROFILE_PERFORMANCE, + PLATFORM_PROFILE_CUSTOM, PLATFORM_PROFILE_LAST, /*must always be last */ };