arm64: dts: rockchip: Allow Turing RK1 cooling fan to spin down
Subsystems (2)
PATCH v2 1/1 arm64: dts: rockchip: Allow Turing RK1 cooling fan to spin down
The RK3588 thermal sensor driver only receives interrupts when a higher-temperature threshold is crossed; it cannot notify when the sensor cools back off. As a result, the driver must poll for temperature changes to detect when the conditions for a thermal trip are no longer met. However, it only does so if the DT enables polling.
Before this patch, the RK1 DT did not enable polling, causing the fan to continue running at the speed corresponding to the highest temperature reached.
Follow suit with similar RK3588 boards by setting a polling-delay of 1000ms, enabling the driver to detect when the sensor cools back off, allowing the fan speed to decrease as appropriate.
---
Changes v1->v2:
- Added Cc: stable per Dragan's suggestion
- Applied Dragan's Reviewed-by tag
- No changes to patch content or main body of commit message
--- arch/arm64/boot/dts/rockchip/rk3588-turing-rk1.dtsi | 2 ++ 1 file changed, 2 insertions(+)
arch/arm64/boot/dts/rockchip/rk3588-turing-rk1.dtsi +2 -0
@@ -214,6 +214,8 @@ rgmii_phy: ethernet-phy@1 { }; &package_thermal { + polling-delay = <1000>; + trips { package_active1: trip-active1 { temperature = <45000>;
Signature · 1 line
2.45.3
Quoted 11 lines · Sam Edwards
The RK3588 thermal sensor driver only receives interrupts when a higher-temperature threshold is crossed; it cannot notify when the sensor cools back off. As a result, the driver must poll for temperature changes to detect when the conditions for a thermal trip are no longer met. However, it only does so if the DT enables polling.
Before this patch, the RK1 DT did not enable polling, causing the fan to continue running at the speed corresponding to the highest temperature reached.
[...]
Applied, thanks!
[1/1] arm64: dts: rockchip: Allow Turing RK1 cooling fan to spin down
commit: fdc7bd909a5f38793468e9cf9b6a9063d96c6234
Best regards,
Signature · 1 line
Heiko Stuebner <[email protected]>