Re: [PATCH] arm64: dts: rockchip: Add idle cooling devices
From: Heiko Stuebner <heiko@sntech.de>
Date: 2021-10-17 10:13:36
Also in:
linux-devicetree, linux-rockchip, lkml
Hi Daniel, Am Freitag, 1. Oktober 2021, 18:17:28 CEST schrieb Daniel Lezcano:
quoted hunk ↗ jump to hunk
The thermal framework accepts now the cpu idle cooling device as an alternative when the cpufreq cooling device fails. Add the node in the DT so the cooling devices will be present and the platforms can extend the thermal zone definition to add them. Signed-off-by: Daniel Lezcano <redacted> --- arch/arm64/boot/dts/rockchip/rk3399.dtsi | 10 ++++++++++ 1 file changed, 10 insertions(+)diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi index 3871c7fd83b0..9ac232ffd284 100644 --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi@@ -124,6 +124,11 @@ cpu_b0: cpu@100 { #cooling-cells = <2>; /* min followed by max */ dynamic-power-coefficient = <436>; cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP>; + thermal-idle { + #cooling-cells = <2>; + duration-us = <10000>; + exit-latency-us = <500>; + };
I guess the basic question would be where do the duration and exit-latency values come from. And also what happened to cpu_l0-l3 (aka the little cores)? Heiko
quoted hunk ↗ jump to hunk
}; cpu_b1: cpu@101 {@@ -136,6 +141,11 @@ cpu_b1: cpu@101 { #cooling-cells = <2>; /* min followed by max */ dynamic-power-coefficient = <436>; cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP>; + thermal-idle { + #cooling-cells = <2>; + duration-us = <10000>; + exit-latency-us = <500>; + }; }; idle-states {
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel