Hello, I have tested your patches. It seems like the driver is now a lot more usable, thanks. What seems to be still unusable for me, is RPM mode. The chip creates an effort to set the RPM, but it almost never stabilizes. I think the reason is mainly because the driver doesn't work with registers like "window" and "pwm rate of change". PWM mode seems to work fine. I think some of the patches in my patch series on the chip are now obsolete. However, I do think at least fanX_div should be implemented. When testing with Sunon PF36281BX-000U-S99 (its max RPM is 23000), the default divisor is not enough (one bit of change equals to about 500 RPM). The only way to change the divisor right now, is to set the target RPM or min RPM. There is also the regmap patch, but I've implemented that one mainly because it made the driver a bit easier to debug. Václav st 26. 5. 2021 v 17:40 odesílatel Guenter Roeck <linux@xxxxxxxxxxxx> napsal: > > The following series provides various fixes and improvements for the > max31790 driver. > > ---------------------------------------------------------------- > Guenter Roeck (7): > hwmon: (max31790) Fix fan speed reporting for fan7..12 > hwmon: (max31790) Report correct current pwm duty cycles > hwmon: (max31790) Fix pwmX_enable attributes > hwmon: (max31790) Add support for fanX_enable attributes > hwmon: (max31790) Clear fan fault after reporting it > hwmon: (max31790) Detect and report zero fan speed > hwmon: (max31790) Add support for fanX_min attributes > > Documentation/hwmon/max31790.rst | 17 +++- > drivers/hwmon/max31790.c | 171 ++++++++++++++++++++++++++++++--------- > 2 files changed, 147 insertions(+), 41 deletions(-)
