[PATCH 0/4] arm64: dts: qcom: x1e80100: Fix thermal trip points

Stephan Gerhold posted 4 patches 10 months ago
arch/arm64/boot/dts/qcom/x1e80100.dtsi | 679 ++++++++-------------------------
1 file changed, 160 insertions(+), 519 deletions(-)
[PATCH 0/4] arm64: dts: qcom: x1e80100: Fix thermal trip points
Posted by Stephan Gerhold 10 months ago
There are some mistakes in the thermal trip points currently used on
X1E80100. Several of the critical trip points are too high, so the hardware
will trigger an emergency reset before the kernel has the chance to
shutdown cleanly. We're also missing GPU cooling in the device tree, which
prevents running certain GPU-intensive applications without reaching the
critical shutdown temperatures. The CPU on the other hand throttles itself
automatically, so we can just drop the passive trip points there.

This series addresses these problems. It's quite similar to Neil's thermal
zone rework for SM8650 [1], which provides some extra background on the
mechanisms behind the throttling.

[1]: https://lore.kernel.org/linux-arm-msm/20250203-topic-sm8650-thermal-cpu-idle-v4-0-65e35f307301@linaro.org/T/

Signed-off-by: Stephan Gerhold <stephan.gerhold@linaro.org>
---
Stephan Gerhold (4):
      arm64: dts: qcom: x1e80100: Fix video thermal zone
      arm64: dts: qcom: x1e80100: Apply consistent critical thermal shutdown
      arm64: dts: qcom: x1e80100: Add GPU cooling
      arm64: dts: qcom: x1e80100: Drop unused passive thermal trip points for CPU

 arch/arm64/boot/dts/qcom/x1e80100.dtsi | 679 ++++++++-------------------------
 1 file changed, 160 insertions(+), 519 deletions(-)
---
base-commit: c177fed7617d6306541305e93e575c0c01600ff0
change-id: 20250218-x1e80100-thermal-fixes-774d08de4e3c

Best regards,
-- 
Stephan Gerhold <stephan.gerhold@linaro.org>
Re: [PATCH 0/4] arm64: dts: qcom: x1e80100: Fix thermal trip points
Posted by Bjorn Andersson 9 months, 2 weeks ago
On Wed, 19 Feb 2025 12:36:17 +0100, Stephan Gerhold wrote:
> There are some mistakes in the thermal trip points currently used on
> X1E80100. Several of the critical trip points are too high, so the hardware
> will trigger an emergency reset before the kernel has the chance to
> shutdown cleanly. We're also missing GPU cooling in the device tree, which
> prevents running certain GPU-intensive applications without reaching the
> critical shutdown temperatures. The CPU on the other hand throttles itself
> automatically, so we can just drop the passive trip points there.
> 
> [...]

Applied, thanks!

[1/4] arm64: dts: qcom: x1e80100: Fix video thermal zone
      commit: 801befff4c827aa72e3698367c5afc18987a6a3f
[2/4] arm64: dts: qcom: x1e80100: Apply consistent critical thermal shutdown
      commit: 03f2b8eed73418269a158ccebad5d8d8f2f6daa1
[3/4] arm64: dts: qcom: x1e80100: Add GPU cooling
      commit: 5ba21fa11f473c9827f378ace8c9f983de9e0287
[4/4] arm64: dts: qcom: x1e80100: Drop unused passive thermal trip points for CPU
      commit: 06eadce936971dd11279e53b6dfb151804137836

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>