[PATCH 0/4] hwmon: (dell-smm) Add support for automatic fan mode

Armin Wolf posted 4 patches 2 weeks ago
Documentation/hwmon/dell-smm-hwmon.rst | 56 +++++++++------
drivers/hwmon/dell-smm-hwmon.c         | 99 ++++++++++++++++++--------
include/uapi/linux/i8k.h               |  2 +
3 files changed, 108 insertions(+), 49 deletions(-)
[PATCH 0/4] hwmon: (dell-smm) Add support for automatic fan mode
Posted by Armin Wolf 2 weeks ago
This patch series adds support for individually enabling and disabling
automatic fan control for each fan on most Dell machines. This works
by using fan state 3 that does not act like an ordinary fan state on
most devices. Instead of increasing the speed, said fan state usually
enables automatic fan control for the associated fan.

The first patch removes the custom config data for the Precision 490
that turned out to be incorrect, see the link inside the patch
description for details.

The second patch enables i8k_set_fan() to accept fan states larger
than i8k_fan_max to prepare for the following patch.

The third patch finally adds support for this special fan state. It
should be noted that after applying this patch, the "pwmX" and
"fanX_target" sysfs attributes will return -ENODATA when automatic fan
control has been enabled for fan X using the special fan state.

The last patch adds the OptiPlex 7040 to the device whitelist so that
the driver automatically loads on those machine.

All patches have been tested on a Dell Inspiron 3505 and appear to
work.

Armin Wolf (4):
  hwmon: (dell-smm) Remove Dell Precision 490 custom config data
  hwmon: (dell-smm) Move clamping of fan speed out of i8k_set_fan()
  hwmon: (dell-smm) Add support for automatic fan mode
  hwmon: (dell-smm) Add support for Dell OptiPlex 7040

 Documentation/hwmon/dell-smm-hwmon.rst | 56 +++++++++------
 drivers/hwmon/dell-smm-hwmon.c         | 99 ++++++++++++++++++--------
 include/uapi/linux/i8k.h               |  2 +
 3 files changed, 108 insertions(+), 49 deletions(-)

-- 
2.39.5
Re: [PATCH 0/4] hwmon: (dell-smm) Add support for automatic fan mode
Posted by Pali Rohár 2 weeks ago
On Wednesday 17 September 2025 20:10:32 Armin Wolf wrote:
> This patch series adds support for individually enabling and disabling
> automatic fan control for each fan on most Dell machines. This works
> by using fan state 3 that does not act like an ordinary fan state on
> most devices. Instead of increasing the speed, said fan state usually
> enables automatic fan control for the associated fan.
> 
> The first patch removes the custom config data for the Precision 490
> that turned out to be incorrect, see the link inside the patch
> description for details.
> 
> The second patch enables i8k_set_fan() to accept fan states larger
> than i8k_fan_max to prepare for the following patch.
> 
> The third patch finally adds support for this special fan state. It
> should be noted that after applying this patch, the "pwmX" and
> "fanX_target" sysfs attributes will return -ENODATA when automatic fan
> control has been enabled for fan X using the special fan state.
> 
> The last patch adds the OptiPlex 7040 to the device whitelist so that
> the driver automatically loads on those machine.
> 
> All patches have been tested on a Dell Inspiron 3505 and appear to
> work.
> 
> Armin Wolf (4):
>   hwmon: (dell-smm) Remove Dell Precision 490 custom config data
>   hwmon: (dell-smm) Move clamping of fan speed out of i8k_set_fan()
>   hwmon: (dell-smm) Add support for automatic fan mode
>   hwmon: (dell-smm) Add support for Dell OptiPlex 7040
> 
>  Documentation/hwmon/dell-smm-hwmon.rst | 56 +++++++++------
>  drivers/hwmon/dell-smm-hwmon.c         | 99 ++++++++++++++++++--------
>  include/uapi/linux/i8k.h               |  2 +
>  3 files changed, 108 insertions(+), 49 deletions(-)
> 
> -- 
> 2.39.5
> 

This patch series looks good to me.
Acked-by: Pali Rohár <pali@kernel.org>