From: Luke Jones <luke@ljones.dev>
To: linux-kernel@vger.kernel.org
Cc: hdegoede@redhat.com, hadess@hadess.net,
platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH] asus-wmi: Add support for platform_profile
Date: Fri, 13 Aug 2021 17:27:03 +1200 [thread overview]
Message-ID: <35JRXQ.1ZW8QHWM1YRG@ljones.dev> (raw)
In-Reply-To: <20210813024201.9518-1-luke@ljones.dev>
I'm unsure of how to update the existing code for fn+f5 (next thermal
profile) used by laptops like the TUF series that have keyboard over
i2c. I was thinking of the following:
static int throttle_thermal_policy_switch_next(struct asus_wmi *asus)
{
struct platform_profile_handler *handler =
&asus->platform_profile_handler; // added
u8 new_mode = asus->throttle_thermal_policy_mode + 1;
if (new_mode > ASUS_THROTTLE_THERMAL_POLICY_SILENT)
new_mode = ASUS_THROTTLE_THERMAL_POLICY_DEFAULT;
// asus->throttle_thermal_policy_mode = new_mode;
// return throttle_thermal_policy_write(asus);
return handler->profile_set(&asus->platform_profile_handler,
new_mode); // added
}
* two lines added, two commented
I'm not able to test this though, and there are very few active people
in my discord with TUF/i2c laptops to ask for testing also.
My concern here is to get the platform_profile correctly updated.
Simply updating asus->throttle_thermal_policy_mode isn't going to
achieve what we'll want.
Regards,
Luke.
On Fri, Aug 13 2021 at 14:42:01 +1200, Luke D. Jones <luke@ljones.dev>
wrote:
> Add initial support for platform_profile where the support is
> based on availability of ASUS_THROTTLE_THERMAL_POLICY.
>
> Signed-off-by: Luke D. Jones <luke@ljones.dev>
> ---
> drivers/platform/x86/asus-wmi.c | 112
> ++++++++++++++++++++++++++++++++
> 1 file changed, 112 insertions(+)
>
> diff --git a/drivers/platform/x86/asus-wmi.c
> b/drivers/platform/x86/asus-wmi.c
> index 90a6a0d00deb..62260043c15c 100644
> --- a/drivers/platform/x86/asus-wmi.c
> +++ b/drivers/platform/x86/asus-wmi.c
> @@ -26,6 +26,7 @@
> #include <linux/rfkill.h>
> #include <linux/pci.h>
> #include <linux/pci_hotplug.h>
> +#include <linux/platform_profile.h>
> #include <linux/power_supply.h>
> #include <linux/hwmon.h>
> #include <linux/hwmon-sysfs.h>
> @@ -219,6 +220,9 @@ struct asus_wmi {
> bool throttle_thermal_policy_available;
> u8 throttle_thermal_policy_mode;
>
> + struct platform_profile_handler platform_profile_handler;
> + bool platform_profile_support;
> +
> // The RSOC controls the maximum charging percentage.
> bool battery_rsoc_available;
>
> @@ -2144,6 +2148,106 @@ static ssize_t
> throttle_thermal_policy_store(struct device *dev,
> // Throttle thermal policy: 0 - default, 1 - overboost, 2 - silent
> static DEVICE_ATTR_RW(throttle_thermal_policy);
>
> +/* Platform profile
> ***********************************************************/
> +static int platform_profile_get(struct platform_profile_handler
> *pprof,
> + enum platform_profile_option *profile)
> +{
> + struct asus_wmi *asus;
> + int tp;
> +
> + asus = container_of(pprof, struct asus_wmi,
> platform_profile_handler);
> +
> + /* All possible toggles like throttle_thermal_policy here */
> + if (asus->throttle_thermal_policy_available) {
> + tp = asus->throttle_thermal_policy_mode;
> + } else {
> + return -1;
> + }
> +
> + if (tp < 0)
> + return tp;
> +
> + switch (tp) {
> + case ASUS_THROTTLE_THERMAL_POLICY_DEFAULT:
> + *profile = PLATFORM_PROFILE_BALANCED;
> + break;
> + case ASUS_THROTTLE_THERMAL_POLICY_OVERBOOST:
> + *profile = PLATFORM_PROFILE_PERFORMANCE;
> + break;
> + case ASUS_THROTTLE_THERMAL_POLICY_SILENT:
> + *profile = PLATFORM_PROFILE_QUIET;
> + break;
> + default:
> + return -EINVAL;
> + }
> +
> + return 0;
> +}
> +
> +static int platform_profile_set(struct platform_profile_handler
> *pprof,
> + enum platform_profile_option profile)
> +{
> + struct asus_wmi *asus;
> + int tp;
> +
> + asus = container_of(pprof, struct asus_wmi,
> platform_profile_handler);
> +
> + /* All possible toggles like throttle_thermal_policy here */
> + if (!asus->throttle_thermal_policy_available) {
> + return -1;
> + }
> +
> + switch (profile) {
> + case PLATFORM_PROFILE_PERFORMANCE:
> + tp = ASUS_THROTTLE_THERMAL_POLICY_OVERBOOST;
> + break;
> + case PLATFORM_PROFILE_BALANCED:
> + tp = ASUS_THROTTLE_THERMAL_POLICY_DEFAULT;
> + break;
> + case PLATFORM_PROFILE_QUIET:
> + tp = ASUS_THROTTLE_THERMAL_POLICY_SILENT;
> + break;
> + default:
> + return -EOPNOTSUPP;
> + }
> +
> + if (asus->throttle_thermal_policy_available) {
> + asus->throttle_thermal_policy_mode = tp;
> + return throttle_thermal_policy_write(asus);
> + }
> + return 0;
> +}
> +
> +static int platform_profile_setup(struct asus_wmi *asus)
> +{
> + int err;
> +
> + if (asus->throttle_thermal_policy_available) {
> + pr_info("Using throttle_thermal_policy for platform_profile
> support\n");
> + } else {
> + /*
> + * Not an error if a component platform_profile relies on is
> unavailable
> + * so early return, skipping the setup of platform_profile.
> + */
> + return 0;
> + }
> + asus->platform_profile_handler.profile_get = platform_profile_get;
> + asus->platform_profile_handler.profile_set = platform_profile_set;
> +
> + set_bit(PLATFORM_PROFILE_QUIET,
> asus->platform_profile_handler.choices);
> + set_bit(PLATFORM_PROFILE_BALANCED,
> + asus->platform_profile_handler.choices);
> + set_bit(PLATFORM_PROFILE_PERFORMANCE,
> + asus->platform_profile_handler.choices);
> +
> + err = platform_profile_register(&asus->platform_profile_handler);
> + if (err)
> + return err;
> +
> + asus->platform_profile_support = true;
> + return 0;
> +}
> +
> /* Backlight
> ******************************************************************/
>
> static int read_backlight_power(struct asus_wmi *asus)
> @@ -2904,6 +3008,10 @@ static int asus_wmi_add(struct platform_device
> *pdev)
> else
> throttle_thermal_policy_set_default(asus);
>
> + err = platform_profile_setup(asus);
> + if (err)
> + goto fail_platform_profile_setup;
> +
> err = panel_od_check_present(asus);
> if (err)
> goto fail_panel_od;
> @@ -2993,6 +3101,7 @@ static int asus_wmi_add(struct platform_device
> *pdev)
> asus_wmi_sysfs_exit(asus->platform_device);
> fail_sysfs:
> fail_throttle_thermal_policy:
> +fail_platform_profile_setup:
> fail_fan_boost_mode:
> fail_egpu_enable:
> fail_dgpu_disable:
> @@ -3017,6 +3126,9 @@ static int asus_wmi_remove(struct
> platform_device *device)
> asus_fan_set_auto(asus);
> asus_wmi_battery_exit(asus);
>
> + if (asus->platform_profile_support)
> + platform_profile_remove();
> +
> kfree(asus);
> return 0;
> }
> --
> 2.31.1
>
next prev parent reply other threads:[~2021-08-13 5:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-13 2:42 [PATCH] asus-wmi: Add support for platform_profile Luke D. Jones
2021-08-13 5:27 ` Luke Jones [this message]
2021-08-13 7:03 ` Hans de Goede
2021-08-13 7:13 ` Luke Jones
2021-08-13 7:40 ` Hans de Goede
2021-08-13 8:27 ` Luke Jones
2021-08-13 8:57 ` Hans de Goede
2021-08-13 8:44 ` Hans de Goede
2021-08-13 8:46 ` Hans de Goede
2021-08-13 9:42 ` Luke Jones
2021-08-13 10:05 ` Hans de Goede
2021-08-13 6:47 ` Hans de Goede
2021-08-13 7:20 ` Luke Jones
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=35JRXQ.1ZW8QHWM1YRG@ljones.dev \
--to=luke@ljones.dev \
--cc=hadess@hadess.net \
--cc=hdegoede@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).