lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <474a861e-284e-fa15-5338-31d1c32ec37b@linux.intel.com>
Date: Fri, 28 Mar 2025 16:02:37 +0200 (EET)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Kurt Borja <kuurtb@...il.com>
cc: Armin Wolf <W_Armin@....de>, Hans de Goede <hdegoede@...hat.com>, 
    platform-driver-x86@...r.kernel.org, Dell.Client.Kernel@...l.com, 
    LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 01/12] platform/x86: alienware-wmi-wmax: Rename thermal
 related symbols

On Thu, 13 Mar 2025, Kurt Borja wrote:

> 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@....de>
> Signed-off-by: Kurt Borja <kuurtb@...il.com>

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>

I note though that this also renames OPERATION -> OP, which is fine, it 
just wasn't mentioned.

-- 
 i.


> ---
>  drivers/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 3d3014b5adf046c94c1ebf39a0e28a92622b40d6..ed70e12d73d7fe5d89f3364c5367820bf47e3c1e 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)
> 
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ