]> www.infradead.org Git - users/jedix/linux-maple.git/commit
arm64: dts: qcom: sm8650: drop cpu thermal passive trip points
authorNeil Armstrong <neil.armstrong@linaro.org>
Mon, 3 Feb 2025 13:23:17 +0000 (14:23 +0100)
committerBjorn Andersson <andersson@kernel.org>
Wed, 26 Feb 2025 02:57:04 +0000 (20:57 -0600)
commit7f9a670396029116424a803d3971ff0e552ff0b3
treeda369a21e7670a358870056f9718f5ef45f15a7c
parentf08edb5299166b7c6d4eae439b1d3f81c31ba50e
arm64: dts: qcom: sm8650: drop cpu thermal passive trip points

On the SM8650, the dynamic clock and voltage scaling (DCVS) is done in an
hardware controlled loop using the LMH and EPSS blocks with constraints and
OPPs programmed in the board firmware.

Since the Hardware does a better job at maintaining the CPUs temperature
in an acceptable range by taking in account more parameters like the die
characteristics or other factory fused values, it makes no sense to try
and reproduce a similar set of constraints with the Linux cpufreq thermal
core.

In addition, the tsens IP is responsible for monitoring the temperature
across the SoC and the current settings will heavily trigger the tsens
UP/LOW interrupts if the CPU temperatures reaches the hardware thermal
constraints which are currently defined in the DT. And since the CPUs
are not hooked in the thermal trip points, the potential interrupts and
calculations are a waste of system resources.

Drop the current passive trip points and only leave the critical trip
point that will trigger a software system reboot before an hardware
thermal shutdown in the allmost impossible case the hardware DCVS cannot
handle the temperature surge.

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Link: https://lore.kernel.org/r/20250203-topic-sm8650-thermal-cpu-idle-v4-1-65e35f307301@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
arch/arm64/boot/dts/qcom/sm8650.dtsi