Re: [PATCH] platform/x86: asus-armoury: add support for FX517ZR

Denis Benato <[email protected]>
Newsgroups org.kernel.vger.platform-driver-x86,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 7/29/26 13:09, AbdElRahman Soliman wrote:
> Add DMI match and power-limit table entry for the ASUS TUF Dash F15
> (2022), board FX517ZR, an Alder Lake + RTX 3070 laptop.
>
> AC and DC min/max values for ppt_pl1_spl, ppt_pl2_sppt,
> nv_dynamic_boost and nv_temp_target were read from the sliders in
> ASUS Armoury Crate's manual performance-tuning mode on Windows for
> this exact model, since ASUS does not publish these limits on the Web.
>
> Tested locally: values now report correctly via 'asusctl armoury
> list' instead of unavailable, and Dynamic Boost via nvidia-powerd
> measurably raises GPU power limit under load, consistent with the
> 80W base + 25W boost = 105W max TGP in ASUS's own spec sheet for
> this configuration.
>
> Assisted-by: Claude:claude-sonnet-5
Hi,

Thanks for this.

I would say remove the "Tested locally" paragraph: guidelines says
that commit messages are to be written in the second person as if
you were telling git what to do.

Also I personally never used a commit message so long and
a shorter one will be absolutely be fine too.

Next revision use the -v2 flag and don't forget to email me too.

If you need further help just tell me, thanks!
> Signed-off-by: AbdElRahman Soliman <[email protected]>
> ---
>  drivers/platform/x86/asus-armoury.h | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>
> diff --git a/drivers/platform/x86/asus-armoury.h b/drivers/platform/x86/asus-armoury.h
> index bc899bec7..46c308ba8 100644
> --- a/drivers/platform/x86/asus-armoury.h
> +++ b/drivers/platform/x86/asus-armoury.h
> @@ -975,6 +975,32 @@ static const struct dmi_system_id power_limits[] = {
>  			.requires_fan_curve = true,
>  		},
>  	},
> +	{
> +		.matches = {
> +			DMI_MATCH(DMI_BOARD_NAME, "FX517ZR"),
> +		},
> +		.driver_data = &(struct power_data) {
> +			.ac_data = &(struct power_limits) {
> +				.ppt_pl1_spl_min = 28,
> +				.ppt_pl1_spl_max = 85,
> +				.ppt_pl2_sppt_min = 28,
> +				.ppt_pl2_sppt_max = 135,
> +				.nv_dynamic_boost_min = 5,
> +				.nv_dynamic_boost_max = 25,
> +				.nv_temp_target_min = 75,
> +				.nv_temp_target_max = 87,
> +			},
> +			.dc_data = &(struct power_limits) {
> +				.ppt_pl1_spl_min = 25,
> +				.ppt_pl1_spl_max = 45,
> +				.ppt_pl2_sppt_min = 35,
> +				.ppt_pl2_sppt_max = 60,
> +				.nv_temp_target_min = 75,
> +				.nv_temp_target_max = 87,
> +			},
> +			.requires_fan_curve = true,
> +		},
> +	},
>  	{
>  		.matches = {
>  			DMI_MATCH(DMI_BOARD_NAME, "FX607VU"),
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.