tuxd3v / ats

91 stars 12 forks source link

Fans always run at full speed #25

Closed nicolasbadia closed 3 years ago

nicolasbadia commented 3 years ago

I have purchased this https://pine64.com/product/rockpro64-metal-desktop-nas-casing and connected the fan on +FAN-. Because the fan was always running at full speed, I've installed ats (with the command luarocks build https://raw.githubusercontent.com/tuxd3v/ats/master/ats-master-0.rockspec) on my rockpro64 (Linux rockpro64 5.9.14-rockchip64 #20.11.4 SMP PREEMPT Tue Dec 15 08:52:20 CET 2020 aarch64 aarch64 aarch64 GNU/Linux) but, even if the current CPU temp is 35°C, the fan still run at full speed. I believe the fan should have stopped. I did not change the ats default configuration. ats service is active (running). Any idea on with it is not working as expected?

chippey5 commented 3 years ago

@nicolasbadia I had to add more paths for the fan control on my 2 rockpro64 NAS. Try using this config in your ats.conf instead. I also created a small Python script that waits for these files to be created, but I don't know if this is needed anymore. Let me know if you're still having the issue.

chippey5 commented 3 years ago

#26

nicolasbadia commented 3 years ago

Awesome, as soon as I added /sys/devices/platform/pwm-fan/hwmon/hwmon3/pwm1 and restarted ats, it started working.

chippey5 commented 3 years ago

Nice, just beware that I've seen all of these paths used with my rockpro64's:

PWM_CTL         = {
                        "/sys/class/hwmon/hwmon0/pwm1",
                        "/sys/devices/platform/pwm-fan/hwmon/hwmon0/pwm1",
                        "/sys/devices/platform/pwm-fan/hwmon/hwmon1/pwm1",
                        "/sys/devices/platform/pwm-fan/hwmon/hwmon2/pwm1",
                        "/sys/devices/platform/pwm-fan/hwmon/hwmon3/pwm1"
},
nicolasbadia commented 3 years ago

Ok, good to know. Thanks.

bavay commented 3 years ago

One comment for those stumbling on this thread when looking for a solution to the pwm path often changing after reboots: the solution given by @chippey5 works great, but this does not work with ats version 0.2.0, I had to build ats from master (luarocks build https://raw.githubusercontent.com/tuxd3v/ats/master/ats-master-0.rockspec)