From: Armin Wolf <W_Armin@gmx.de>
To: "Kurt Borja" <kuurtb@gmail.com>,
"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
Cc: platform-driver-x86@vger.kernel.org,
Hans de Goede <hdegoede@redhat.com>,
Dell.Client.Kernel@dell.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 01/10] platform/x86: alienware-wmi-wmax: Rename thermal related symbols
Date: Sun, 16 Feb 2025 06:40:15 +0100 [thread overview]
Message-ID: <dd35f587-20fd-4ff0-8876-f1ea83346e1f@gmx.de> (raw)
In-Reply-To: <20250208051614.10644-2-kuurtb@gmail.com>
Am 08.02.25 um 06:16 schrieb Kurt Borja:
> The "thermal" features of the WMAX WMI device are only present on the
> host device if the ACPI _UID is "AWCC". Replace WMAX prefixes with
> "AWCC" to reflect this relationship.
>
> Thermal profiles with WMAX_PROFILE_BASIC prefix are also renamed to
> WMAX_PROFILE_LEGACY because they are only supported in older versions
> of this WMI device.
Reviewed-by: Armin Wolf <W_Armin@gmx.de>
>
> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
> ---
> .../platform/x86/dell/alienware-wmi-wmax.c | 173 +++++++++---------
> 1 file changed, 87 insertions(+), 86 deletions(-)
>
> diff --git a/drivers/platform/x86/dell/alienware-wmi-wmax.c b/drivers/platform/x86/dell/alienware-wmi-wmax.c
> index 3d3014b5adf0..ed70e12d73d7 100644
> --- a/drivers/platform/x86/dell/alienware-wmi-wmax.c
> +++ b/drivers/platform/x86/dell/alienware-wmi-wmax.c
> @@ -24,16 +24,17 @@
> #define WMAX_METHOD_DEEP_SLEEP_STATUS 0x0C
> #define WMAX_METHOD_BRIGHTNESS 0x3
> #define WMAX_METHOD_ZONE_CONTROL 0x4
> -#define WMAX_METHOD_THERMAL_INFORMATION 0x14
> -#define WMAX_METHOD_THERMAL_CONTROL 0x15
> -#define WMAX_METHOD_GAME_SHIFT_STATUS 0x25
>
> -#define WMAX_THERMAL_MODE_GMODE 0xAB
> +#define AWCC_METHOD_THERMAL_INFORMATION 0x14
> +#define AWCC_METHOD_THERMAL_CONTROL 0x15
> +#define AWCC_METHOD_GAME_SHIFT_STATUS 0x25
>
> -#define WMAX_FAILURE_CODE 0xFFFFFFFF
> -#define WMAX_THERMAL_TABLE_MASK GENMASK(7, 4)
> -#define WMAX_THERMAL_MODE_MASK GENMASK(3, 0)
> -#define WMAX_SENSOR_ID_MASK BIT(8)
> +#define AWCC_THERMAL_MODE_GMODE 0xAB
> +
> +#define AWCC_FAILURE_CODE 0xFFFFFFFF
> +#define AWCC_THERMAL_TABLE_MASK GENMASK(7, 4)
> +#define AWCC_THERMAL_MODE_MASK GENMASK(3, 0)
> +#define AWCC_SENSOR_ID_MASK BIT(8)
>
> static bool force_platform_profile;
> module_param_unsafe(force_platform_profile, bool, 0);
> @@ -151,38 +152,38 @@ static const struct dmi_system_id awcc_dmi_table[] __initconst = {
> },
> };
>
> -enum WMAX_THERMAL_INFORMATION_OPERATIONS {
> - WMAX_OPERATION_SYS_DESCRIPTION = 0x02,
> - WMAX_OPERATION_LIST_IDS = 0x03,
> - WMAX_OPERATION_CURRENT_PROFILE = 0x0B,
> +enum AWCC_THERMAL_INFORMATION_OPERATIONS {
> + AWCC_OP_GET_SYSTEM_DESCRIPTION = 0x02,
> + AWCC_OP_GET_RESOURCE_ID = 0x03,
> + AWCC_OP_GET_CURRENT_PROFILE = 0x0B,
> };
>
> -enum WMAX_THERMAL_CONTROL_OPERATIONS {
> - WMAX_OPERATION_ACTIVATE_PROFILE = 0x01,
> +enum AWCC_THERMAL_CONTROL_OPERATIONS {
> + AWCC_OP_ACTIVATE_PROFILE = 0x01,
> };
>
> -enum WMAX_GAME_SHIFT_STATUS_OPERATIONS {
> - WMAX_OPERATION_TOGGLE_GAME_SHIFT = 0x01,
> - WMAX_OPERATION_GET_GAME_SHIFT_STATUS = 0x02,
> +enum AWCC_GAME_SHIFT_STATUS_OPERATIONS {
> + AWCC_OP_TOGGLE_GAME_SHIFT = 0x01,
> + AWCC_OP_GET_GAME_SHIFT_STATUS = 0x02,
> };
>
> -enum WMAX_THERMAL_TABLES {
> - WMAX_THERMAL_TABLE_BASIC = 0x90,
> - WMAX_THERMAL_TABLE_USTT = 0xA0,
> +enum AWCC_THERMAL_TABLES {
> + AWCC_THERMAL_TABLE_LEGACY = 0x90,
> + AWCC_THERMAL_TABLE_USTT = 0xA0,
> };
>
> -enum wmax_thermal_mode {
> - THERMAL_MODE_USTT_BALANCED,
> - THERMAL_MODE_USTT_BALANCED_PERFORMANCE,
> - THERMAL_MODE_USTT_COOL,
> - THERMAL_MODE_USTT_QUIET,
> - THERMAL_MODE_USTT_PERFORMANCE,
> - THERMAL_MODE_USTT_LOW_POWER,
> - THERMAL_MODE_BASIC_QUIET,
> - THERMAL_MODE_BASIC_BALANCED,
> - THERMAL_MODE_BASIC_BALANCED_PERFORMANCE,
> - THERMAL_MODE_BASIC_PERFORMANCE,
> - THERMAL_MODE_LAST,
> +enum awcc_thermal_profile {
> + AWCC_PROFILE_USTT_BALANCED,
> + AWCC_PROFILE_USTT_BALANCED_PERFORMANCE,
> + AWCC_PROFILE_USTT_COOL,
> + AWCC_PROFILE_USTT_QUIET,
> + AWCC_PROFILE_USTT_PERFORMANCE,
> + AWCC_PROFILE_USTT_LOW_POWER,
> + AWCC_PROFILE_LEGACY_QUIET,
> + AWCC_PROFILE_LEGACY_BALANCED,
> + AWCC_PROFILE_LEGACY_BALANCED_PERFORMANCE,
> + AWCC_PROFILE_LEGACY_PERFORMANCE,
> + AWCC_PROFILE_LAST,
> };
>
> struct wmax_led_args {
> @@ -210,20 +211,20 @@ struct wmax_u32_args {
> struct awcc_priv {
> struct wmi_device *wdev;
> struct device *ppdev;
> - enum wmax_thermal_mode supported_thermal_profiles[PLATFORM_PROFILE_LAST];
> + enum awcc_thermal_profile supported_thermal_profiles[PLATFORM_PROFILE_LAST];
> };
>
> -static const enum platform_profile_option wmax_mode_to_platform_profile[THERMAL_MODE_LAST] = {
> - [THERMAL_MODE_USTT_BALANCED] = PLATFORM_PROFILE_BALANCED,
> - [THERMAL_MODE_USTT_BALANCED_PERFORMANCE] = PLATFORM_PROFILE_BALANCED_PERFORMANCE,
> - [THERMAL_MODE_USTT_COOL] = PLATFORM_PROFILE_COOL,
> - [THERMAL_MODE_USTT_QUIET] = PLATFORM_PROFILE_QUIET,
> - [THERMAL_MODE_USTT_PERFORMANCE] = PLATFORM_PROFILE_PERFORMANCE,
> - [THERMAL_MODE_USTT_LOW_POWER] = PLATFORM_PROFILE_LOW_POWER,
> - [THERMAL_MODE_BASIC_QUIET] = PLATFORM_PROFILE_QUIET,
> - [THERMAL_MODE_BASIC_BALANCED] = PLATFORM_PROFILE_BALANCED,
> - [THERMAL_MODE_BASIC_BALANCED_PERFORMANCE] = PLATFORM_PROFILE_BALANCED_PERFORMANCE,
> - [THERMAL_MODE_BASIC_PERFORMANCE] = PLATFORM_PROFILE_PERFORMANCE,
> +static const enum platform_profile_option awcc_mode_to_platform_profile[AWCC_PROFILE_LAST] = {
> + [AWCC_PROFILE_USTT_BALANCED] = PLATFORM_PROFILE_BALANCED,
> + [AWCC_PROFILE_USTT_BALANCED_PERFORMANCE] = PLATFORM_PROFILE_BALANCED_PERFORMANCE,
> + [AWCC_PROFILE_USTT_COOL] = PLATFORM_PROFILE_COOL,
> + [AWCC_PROFILE_USTT_QUIET] = PLATFORM_PROFILE_QUIET,
> + [AWCC_PROFILE_USTT_PERFORMANCE] = PLATFORM_PROFILE_PERFORMANCE,
> + [AWCC_PROFILE_USTT_LOW_POWER] = PLATFORM_PROFILE_LOW_POWER,
> + [AWCC_PROFILE_LEGACY_QUIET] = PLATFORM_PROFILE_QUIET,
> + [AWCC_PROFILE_LEGACY_BALANCED] = PLATFORM_PROFILE_BALANCED,
> + [AWCC_PROFILE_LEGACY_BALANCED_PERFORMANCE] = PLATFORM_PROFILE_BALANCED_PERFORMANCE,
> + [AWCC_PROFILE_LEGACY_PERFORMANCE] = PLATFORM_PROFILE_PERFORMANCE,
> };
>
> static struct awcc_quirks *awcc;
> @@ -444,26 +445,26 @@ const struct attribute_group wmax_deepsleep_attribute_group = {
> * Thermal Profile control
> * - Provides thermal profile control through the Platform Profile API
> */
> -static bool is_wmax_thermal_code(u32 code)
> +static bool is_awcc_thermal_mode(u32 code)
> {
> - if (code & WMAX_SENSOR_ID_MASK)
> + if (code & AWCC_SENSOR_ID_MASK)
> return false;
>
> - if ((code & WMAX_THERMAL_MODE_MASK) >= THERMAL_MODE_LAST)
> + if ((code & AWCC_THERMAL_MODE_MASK) >= AWCC_PROFILE_LAST)
> return false;
>
> - if ((code & WMAX_THERMAL_TABLE_MASK) == WMAX_THERMAL_TABLE_BASIC &&
> - (code & WMAX_THERMAL_MODE_MASK) >= THERMAL_MODE_BASIC_QUIET)
> + if ((code & AWCC_THERMAL_TABLE_MASK) == AWCC_THERMAL_TABLE_LEGACY &&
> + (code & AWCC_THERMAL_MODE_MASK) >= AWCC_PROFILE_LEGACY_QUIET)
> return true;
>
> - if ((code & WMAX_THERMAL_TABLE_MASK) == WMAX_THERMAL_TABLE_USTT &&
> - (code & WMAX_THERMAL_MODE_MASK) <= THERMAL_MODE_USTT_LOW_POWER)
> + if ((code & AWCC_THERMAL_TABLE_MASK) == AWCC_THERMAL_TABLE_USTT &&
> + (code & AWCC_THERMAL_MODE_MASK) <= AWCC_PROFILE_USTT_LOW_POWER)
> return true;
>
> return false;
> }
>
> -static int wmax_thermal_information(struct wmi_device *wdev, u8 operation,
> +static int awcc_thermal_information(struct wmi_device *wdev, u8 operation,
> u8 arg, u32 *out_data)
> {
> struct wmax_u32_args in_args = {
> @@ -474,21 +475,21 @@ static int wmax_thermal_information(struct wmi_device *wdev, u8 operation,
> };
> int ret;
>
> - ret = alienware_wmi_command(wdev, WMAX_METHOD_THERMAL_INFORMATION,
> + ret = alienware_wmi_command(wdev, AWCC_METHOD_THERMAL_INFORMATION,
> &in_args, sizeof(in_args), out_data);
> if (ret < 0)
> return ret;
>
> - if (*out_data == WMAX_FAILURE_CODE)
> + if (*out_data == AWCC_FAILURE_CODE)
> return -EBADRQC;
>
> return 0;
> }
>
> -static int wmax_thermal_control(struct wmi_device *wdev, u8 profile)
> +static int awcc_thermal_control(struct wmi_device *wdev, u8 profile)
> {
> struct wmax_u32_args in_args = {
> - .operation = WMAX_OPERATION_ACTIVATE_PROFILE,
> + .operation = AWCC_OP_ACTIVATE_PROFILE,
> .arg1 = profile,
> .arg2 = 0,
> .arg3 = 0,
> @@ -496,18 +497,18 @@ static int wmax_thermal_control(struct wmi_device *wdev, u8 profile)
> u32 out_data;
> int ret;
>
> - ret = alienware_wmi_command(wdev, WMAX_METHOD_THERMAL_CONTROL,
> + ret = alienware_wmi_command(wdev, AWCC_METHOD_THERMAL_CONTROL,
> &in_args, sizeof(in_args), &out_data);
> if (ret)
> return ret;
>
> - if (out_data == WMAX_FAILURE_CODE)
> + if (out_data == AWCC_FAILURE_CODE)
> return -EBADRQC;
>
> return 0;
> }
>
> -static int wmax_game_shift_status(struct wmi_device *wdev, u8 operation,
> +static int awcc_game_shift_status(struct wmi_device *wdev, u8 operation,
> u32 *out_data)
> {
> struct wmax_u32_args in_args = {
> @@ -518,46 +519,46 @@ static int wmax_game_shift_status(struct wmi_device *wdev, u8 operation,
> };
> int ret;
>
> - ret = alienware_wmi_command(wdev, WMAX_METHOD_GAME_SHIFT_STATUS,
> + ret = alienware_wmi_command(wdev, AWCC_METHOD_GAME_SHIFT_STATUS,
> &in_args, sizeof(in_args), out_data);
> if (ret < 0)
> return ret;
>
> - if (*out_data == WMAX_FAILURE_CODE)
> + if (*out_data == AWCC_FAILURE_CODE)
> return -EOPNOTSUPP;
>
> return 0;
> }
>
> -static int thermal_profile_get(struct device *dev,
> - enum platform_profile_option *profile)
> +static int awcc_platform_profile_get(struct device *dev,
> + enum platform_profile_option *profile)
> {
> struct awcc_priv *priv = dev_get_drvdata(dev);
> u32 out_data;
> int ret;
>
> - ret = wmax_thermal_information(priv->wdev, WMAX_OPERATION_CURRENT_PROFILE,
> + ret = awcc_thermal_information(priv->wdev, AWCC_OP_GET_CURRENT_PROFILE,
> 0, &out_data);
>
> if (ret < 0)
> return ret;
>
> - if (out_data == WMAX_THERMAL_MODE_GMODE) {
> + if (out_data == AWCC_THERMAL_MODE_GMODE) {
> *profile = PLATFORM_PROFILE_PERFORMANCE;
> return 0;
> }
>
> - if (!is_wmax_thermal_code(out_data))
> + if (!is_awcc_thermal_mode(out_data))
> return -ENODATA;
>
> - out_data &= WMAX_THERMAL_MODE_MASK;
> - *profile = wmax_mode_to_platform_profile[out_data];
> + out_data &= AWCC_THERMAL_MODE_MASK;
> + *profile = awcc_mode_to_platform_profile[out_data];
>
> return 0;
> }
>
> -static int thermal_profile_set(struct device *dev,
> - enum platform_profile_option profile)
> +static int awcc_platform_profile_set(struct device *dev,
> + enum platform_profile_option profile)
> {
> struct awcc_priv *priv = dev_get_drvdata(dev);
>
> @@ -565,8 +566,8 @@ static int thermal_profile_set(struct device *dev,
> u32 gmode_status;
> int ret;
>
> - ret = wmax_game_shift_status(priv->wdev,
> - WMAX_OPERATION_GET_GAME_SHIFT_STATUS,
> + ret = awcc_game_shift_status(priv->wdev,
> + AWCC_OP_GET_GAME_SHIFT_STATUS,
> &gmode_status);
>
> if (ret < 0)
> @@ -574,8 +575,8 @@ static int thermal_profile_set(struct device *dev,
>
> if ((profile == PLATFORM_PROFILE_PERFORMANCE && !gmode_status) ||
> (profile != PLATFORM_PROFILE_PERFORMANCE && gmode_status)) {
> - ret = wmax_game_shift_status(priv->wdev,
> - WMAX_OPERATION_TOGGLE_GAME_SHIFT,
> + ret = awcc_game_shift_status(priv->wdev,
> + AWCC_OP_TOGGLE_GAME_SHIFT,
> &gmode_status);
>
> if (ret < 0)
> @@ -583,21 +584,21 @@ static int thermal_profile_set(struct device *dev,
> }
> }
>
> - return wmax_thermal_control(priv->wdev,
> + return awcc_thermal_control(priv->wdev,
> priv->supported_thermal_profiles[profile]);
> }
>
> -static int thermal_profile_probe(void *drvdata, unsigned long *choices)
> +static int awcc_platform_profile_probe(void *drvdata, unsigned long *choices)
> {
> enum platform_profile_option profile;
> struct awcc_priv *priv = drvdata;
> - enum wmax_thermal_mode mode;
> + enum awcc_thermal_profile mode;
> u8 sys_desc[4];
> u32 first_mode;
> u32 out_data;
> int ret;
>
> - ret = wmax_thermal_information(priv->wdev, WMAX_OPERATION_SYS_DESCRIPTION,
> + ret = awcc_thermal_information(priv->wdev, AWCC_OP_GET_SYSTEM_DESCRIPTION,
> 0, (u32 *) &sys_desc);
> if (ret < 0)
> return ret;
> @@ -605,7 +606,7 @@ static int thermal_profile_probe(void *drvdata, unsigned long *choices)
> first_mode = sys_desc[0] + sys_desc[1];
>
> for (u32 i = 0; i < sys_desc[3]; i++) {
> - ret = wmax_thermal_information(priv->wdev, WMAX_OPERATION_LIST_IDS,
> + ret = awcc_thermal_information(priv->wdev, AWCC_OP_GET_RESOURCE_ID,
> i + first_mode, &out_data);
>
> if (ret == -EIO)
> @@ -614,11 +615,11 @@ static int thermal_profile_probe(void *drvdata, unsigned long *choices)
> if (ret == -EBADRQC)
> break;
>
> - if (!is_wmax_thermal_code(out_data))
> + if (!is_awcc_thermal_mode(out_data))
> continue;
>
> - mode = out_data & WMAX_THERMAL_MODE_MASK;
> - profile = wmax_mode_to_platform_profile[mode];
> + mode = out_data & AWCC_THERMAL_MODE_MASK;
> + profile = awcc_mode_to_platform_profile[mode];
> priv->supported_thermal_profiles[profile] = out_data;
>
> set_bit(profile, choices);
> @@ -629,7 +630,7 @@ static int thermal_profile_probe(void *drvdata, unsigned long *choices)
>
> if (awcc->gmode) {
> priv->supported_thermal_profiles[PLATFORM_PROFILE_PERFORMANCE] =
> - WMAX_THERMAL_MODE_GMODE;
> + AWCC_THERMAL_MODE_GMODE;
>
> set_bit(PLATFORM_PROFILE_PERFORMANCE, choices);
> }
> @@ -638,9 +639,9 @@ static int thermal_profile_probe(void *drvdata, unsigned long *choices)
> }
>
> static const struct platform_profile_ops awcc_platform_profile_ops = {
> - .probe = thermal_profile_probe,
> - .profile_get = thermal_profile_get,
> - .profile_set = thermal_profile_set,
> + .probe = awcc_platform_profile_probe,
> + .profile_get = awcc_platform_profile_get,
> + .profile_set = awcc_platform_profile_set,
> };
>
> static int awcc_platform_profile_init(struct wmi_device *wdev)
next prev parent reply other threads:[~2025-02-16 5:40 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-08 5:16 [PATCH 00/10] HWMON support + DebugFS + Improvements Kurt Borja
2025-02-08 5:16 ` [PATCH 01/10] platform/x86: alienware-wmi-wmax: Rename thermal related symbols Kurt Borja
2025-02-16 5:40 ` Armin Wolf [this message]
2025-02-08 5:16 ` [PATCH 02/10] platform/x86: alienware-wmi-wmax: Refactor is_awcc_thermal_mode() Kurt Borja
2025-02-16 5:44 ` Armin Wolf
2025-02-08 5:16 ` [PATCH 03/10] platform/x86: alienware-wmi-wmax: Improve internal AWCC API Kurt Borja
2025-02-16 5:48 ` Armin Wolf
2025-02-08 5:16 ` [PATCH 04/10] platform/x86: alienware-wmi-wmax: Modify supported_thermal_profiles[] Kurt Borja
2025-02-16 5:49 ` Armin Wolf
2025-02-08 5:16 ` [PATCH 05/10] platform/x86: alienware-wmi-wmax: Improve platform profile probe Kurt Borja
2025-02-16 5:57 ` Armin Wolf
2025-02-08 5:16 ` [PATCH 06/10] platform/x86: alienware-wmi-wmax: Add support for the "custom" thermal profile Kurt Borja
2025-02-16 5:59 ` Armin Wolf
2025-02-08 5:16 ` [PATCH 07/10] platform/x86: alienware-wmi-wmax: Add HWMON support Kurt Borja
2025-02-16 6:06 ` Armin Wolf
2025-02-16 21:27 ` Kurt Borja
2025-02-08 5:16 ` [PATCH 08/10] platform/x86: alienware-wmi-wmax: Add support for manual fan control Kurt Borja
2025-02-16 6:12 ` Armin Wolf
2025-02-16 6:14 ` Armin Wolf
2025-02-16 21:25 ` Kurt Borja
2025-02-08 5:16 ` [PATCH 09/10] platform/x86: alienware-wmi-wmax: Add a DebugFS interface Kurt Borja
2025-02-08 5:16 ` [PATCH 10/10] platform/x86: alienware-wmi: Improve and update documentation Kurt Borja
2025-02-16 6:22 ` Armin Wolf
2025-02-16 20:51 ` Kurt Borja
[not found] ` <198a578f-99d6-4d98-8186-eab141a2550d@gmx.de>
2025-02-16 21:38 ` Kurt Borja
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=dd35f587-20fd-4ff0-8876-f1ea83346e1f@gmx.de \
--to=w_armin@gmx.de \
--cc=Dell.Client.Kernel@dell.com \
--cc=hdegoede@redhat.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=kuurtb@gmail.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
all inboxes | Powered by JetHome®