* [PATCH v3] hwmon: (dell-smm) Add Dell Pro Precision 7 16 PW716260 to fan control whitelist
[not found] <20261111172707.84072-1-shady.abh@gmail.com>
@ 2026-11-12 16:04 ` Shady Abu-Hussein
2026-11-14 16:54 ` Guenter Roeck
0 siblings, 1 reply; 2+ messages in thread
From: Shady Abu-Hussein @ 2026-11-12 16:04 UTC (permalink / raw)
To: Pali Rohár, Guenter Roeck
Cc: linux-hwmon, linux-kernel, Shady Abu-Hussein
On the Dell Pro Precision 7 16 PW716260 the firmware fan control
cannot be overridden using the default SMM method, writes to pwmX
are ignored and pwmX_enable returns -EINVAL. The firmware AC fan
policy also requests speeds below the minimum sustainable speed of
the fans, so one fan repeatedly stops and restarts every few
seconds at low load.
Add the machine to the fan control whitelist using the
I8K_FAN_30A3_31A3 SMM codes. With this, writing 1 to pwm1_enable
disables automatic fan control, the fans follow the requested
state (0: off, 1: ~2500 RPM, 2: ~3950 RPM), and writing 2 to
pwm1_enable restores automatic fan control.
Tested on BIOS 1.5.0 with kernel 7.2.15.
Signed-off-by: Shady Abu-Hussein <shady.abh@gmail.com>
---
v2: Use real name in Signed-off-by instead of a handle, per review
v3: No code change; adding this changelog, which v2 was missing
drivers/hwmon/dell-smm-hwmon.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
index 5ec487843..250d8c1e6 100644
--- a/drivers/hwmon/dell-smm-hwmon.c
+++ b/drivers/hwmon/dell-smm-hwmon.c
@@ -1687,6 +1687,14 @@ static const struct dmi_system_id i8k_whitelist_fan_control[] __initconst = {
},
.driver_data = (void *)&i8k_fan_control_data[I8K_FAN_30A3_31A3],
},
+ {
+ .ident = "Dell Pro Precision 7 16 PW716260",
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+ DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Dell Pro Precision 7 16 PW716260"),
+ },
+ .driver_data = (void *)&i8k_fan_control_data[I8K_FAN_30A3_31A3],
+ },
{ }
};
--
2.56.0
^ permalink raw reply [flat|nested] 2+ messages in thread