On Mon, Apr 8, 2024 at 6:41 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
>
> Hi Gergo,
>
> On 4/6/24 2:01 AM, Gergo Koteles wrote:
> > Some laptops have a key to switch platform profiles.
> >
> > Add a platform_profile_cycle() function to cycle between the enabled
> > profiles.
> >
> > Signed-off-by: Gergo Koteles <soyer@xxxxxx>
>
> Thank you for your patch, 1 small remark below,
> otherwise this looks good to me.
>
> Rafael, may I have your Ack for merging this through the pdx86 tree
> together with the rest of the series once my remark has been addressed ?
Sure, please feel free to add
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
to this patch. Thanks!
> > ---
> > drivers/acpi/platform_profile.c | 39 ++++++++++++++++++++++++++++++++
> > include/linux/platform_profile.h | 1 +
> > 2 files changed, 40 insertions(+)
> >
> > diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
> > index d418462ab791..acc606af812a 100644
> > --- a/drivers/acpi/platform_profile.c
> > +++ b/drivers/acpi/platform_profile.c
> > @@ -136,6 +136,45 @@ void platform_profile_notify(void)
> > }
> > EXPORT_SYMBOL_GPL(platform_profile_notify);
> >
> > +int platform_profile_cycle(void)
> > +{
> > + enum platform_profile_option profile;
> > + enum platform_profile_option next;
> > + int err;
> > +
> > + err = mutex_lock_interruptible(&profile_lock);
> > + if (err)
> > + return err;
> > +
> > + if (!cur_profile) {
> > + mutex_unlock(&profile_lock);
> > + return -ENODEV;
> > + }
> > +
> > + err = cur_profile->profile_get(cur_profile, &profile);
> > + if (err) {
> > + mutex_unlock(&profile_lock);
> > + return err;
> > + }
> > +
> > + next = find_next_bit_wrap(cur_profile->choices,
> > + ARRAY_SIZE(profile_names), profile + 1);
> > +
> > + if (WARN_ON(next == ARRAY_SIZE(profile_names))) {
>
> Other code in drivers/acpi/platform_profile.c uses PLATFORM_PROFILE_LAST
> instead of ARRAY_SIZE(profile_names) (these are guaranteed to be equal)
> please switch to using PLATFORM_PROFILE_LAST for consistency.
>
> Regards,
>
> Hans
>
>
>
>
>
> > + mutex_unlock(&profile_lock);
> > + return -EINVAL;
> > + }
> > +
> > + err = cur_profile->profile_set(cur_profile, next);
> > + mutex_unlock(&profile_lock);
> > +
> > + if (!err)
> > + sysfs_notify(acpi_kobj, NULL, "platform_profile");
> > +
> > + return err;
> > +}
> > +EXPORT_SYMBOL_GPL(platform_profile_cycle);
> > +
> > int platform_profile_register(struct platform_profile_handler *pprof)
> > {
> > int err;
> > diff --git a/include/linux/platform_profile.h b/include/linux/platform_profile.h
> > index e5cbb6841f3a..f5492ed413f3 100644
> > --- a/include/linux/platform_profile.h
> > +++ b/include/linux/platform_profile.h
> > @@ -36,6 +36,7 @@ struct platform_profile_handler {
> >
> > int platform_profile_register(struct platform_profile_handler *pprof);
> > int platform_profile_remove(void);
> > +int platform_profile_cycle(void);
> > void platform_profile_notify(void);
> >
> > #endif /*_PLATFORM_PROFILE_H_*/
>
