liquidctl / liquidtux

Linux kernel hwmon drivers for AIO liquid coolers and other devices
Other
80 stars 15 forks source link

nzxt-kraken3: PWM setting fixes; status reporting optimization #34

Closed aleksamagicka closed 1 year ago

aleksamagicka commented 1 year ago

This should resolve the issues noted in #7 and optimize Z series status reporting.


Seems to work when using fan2go, here's my config: fan2go.txt. I had to set maxRpmDiffForSettledFan: 20 because X53 varied between 14-20 as far as I've seen, and the default value is 10. Didn't yet test with pwmconfig and fancontrol.

stalkerg commented 1 year ago

Thanks a lot! I will check today or tomorrow!

stalkerg commented 1 year ago

It seems like working as expected, I also have no any significant issues with fan2go. Thanks again!

amezin commented 1 year ago

@aleksamagicka @stalkerg By the way, have you tested the driver with lm-sensors (i. e. pwmconfig+fancontrol)? These utilities sometimes fail in surprising ways. As far as I remember, they treat almost any sysfs read error as "oh the device is completely broken let's die".

stalkerg commented 1 year ago

@amezin I didn't test it in fancontrol but fan2go has totally the same behavior for read error (and it's ok).