From c0605ae3962cca877030d3d6c9924ddb72af0fa0 Mon Sep 17 00:00:00 2001
From: Samtinel <nospam>
Date: Tue, 31 Oct 2023 13:36:53 +0000
Subject: [PATCH] pine64-rockpro64: add fancontrol
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

rk3399's operating temps are -20 to 80°C. It has sensors for CPU and GPU.
The MINSTART/-STOP values are a result of empirical testing.
---
 pine64/rockpro64/default.nix | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/pine64/rockpro64/default.nix b/pine64/rockpro64/default.nix
index 03f600143..09617ccee 100644
--- a/pine64/rockpro64/default.nix
+++ b/pine64/rockpro64/default.nix
@@ -6,4 +6,24 @@
     "pcie_rockchip_host"
     "phy_rockchip_pcie"
   ];
+  # control the fan on the rockpro64 like in the NAS case
+  hardware.fancontrol = {
+    enable = lib.mkDefault true;
+    config = lib.mkDefault ''
+      INTERVAL=3
+      DEVPATH=hwmon0=devices/virtual/thermal/thermal_zone0 hwmon1=devices/virtual/thermal/thermal_zone1 hwmon3=devices/platform/pwm-fan
+      DEVNAME=hwmon0=cpu_thermal hwmon1=gpu_thermal hwmon3=pwmfan
+      # there can only be one sensor mapped to one pwm
+      # therefore you'll have to decide if you want to check cpu temps
+      # just replace hwmon0 with hwmon1 below
+      # https://github.com/lm-sensors/lm-sensors/issues/228
+      FCTEMPS=hwmon3/pwm1=hwmon0/temp1_input
+      MINTEMP=hwmon3/pwm1=40
+      MAXTEMP=hwmon3/pwm1=80
+      MINSTART=hwmon3/pwm1=35
+      MINSTOP=hwmon3/pwm1=30
+      MINPWM=hwmon3/pwm1=0
+      MAXPWM=hwmon3/pwm1=255
+    '';
+  };
 }