[PATCH v4 3/4] arm64: dts: qcom: sm8650: harmonize all unregulated thermal trip points

From: Neil Armstrong
Date: Mon Feb 03 2025 - 08:24:22 EST

Next message: Neil Armstrong: "[PATCH v4 4/4] arm64: dts: qcom: sm8650: drop remaining polling-delay-passive properties"
Previous message: Neil Armstrong: "[PATCH v4 2/4] arm64: dts: qcom: sm8650: setup gpu thermal with higher temperatures"
In reply to: Konrad Dybcio: "Re: [PATCH v4 2/4] arm64: dts: qcom: sm8650: setup gpu thermal with higher temperatures"
Next in thread: Neil Armstrong: "[PATCH v4 4/4] arm64: dts: qcom: sm8650: drop remaining polling-delay-passive properties"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

While the CPUs thermal is handled by the LMH, and GPU has a passive
cooldowm via the HLOS DCVS, all the other thermal blocks only have
hot and critical and no passive/active trip points.

Passive or active thermal management for those blocks should
be either defined if somehow we can express those in DT or
in the board definition if there's an active cooling device
available.

The tsens MAX_THRESHOLD is set to 120C on those platforms, so set
the hot to 110C to leave a chance to HLOS to react and critical to
115C to avoid the monitor thermal shutdown.

In the case a passive or active cooling device would be
available, the downstream reference implementation uses
the 95C "tj" trip point, as we already use for the
gpuss thermal blocks.

Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
---
arch/arm64/boot/dts/qcom/sm8650.dtsi | 170 +++++++++++++++++------------------
1 file changed, 85 insertions(+), 85 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi
index 798265e0300e7c7934e04cd971e3b9d3d55e3bec..24070d99b6f097f38e618330414b5a04734e7e32 100644
--- a/arch/arm64/boot/dts/qcom/sm8650.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi
@@ -5661,14 +5661,14 @@ aoss0-thermal {
thermal-sensors = <&tsens0 0>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ aoss0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

aoss0-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5679,14 +5679,14 @@ cpuss0-thermal {
thermal-sensors = <&tsens0 1>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ cpuss0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

cpuss0-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5697,14 +5697,14 @@ cpuss1-thermal {
thermal-sensors = <&tsens0 2>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ cpuss1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

cpuss1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5715,14 +5715,14 @@ cpuss2-thermal {
thermal-sensors = <&tsens0 3>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ cpuss2-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

cpuss2-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5733,14 +5733,14 @@ cpuss3-thermal {
thermal-sensors = <&tsens0 4>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ cpuss3-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

cpuss3-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5871,14 +5871,14 @@ aoss1-thermal {
thermal-sensors = <&tsens1 0>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ aoss1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

aoss1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5951,14 +5951,14 @@ nsphvx0-thermal {
thermal-sensors = <&tsens2 6>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphvx0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

- nsphvx1-critical {
- temperature = <110000>;
+ nsphvx0-critical {
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5971,14 +5971,14 @@ nsphvx1-thermal {
thermal-sensors = <&tsens2 7>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphvx1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

nsphvx1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -5991,14 +5991,14 @@ nsphmx0-thermal {
thermal-sensors = <&tsens2 8>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphmx0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

nsphmx0-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6011,14 +6011,14 @@ nsphmx1-thermal {
thermal-sensors = <&tsens2 9>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphmx1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

nsphmx1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6031,14 +6031,14 @@ nsphmx2-thermal {
thermal-sensors = <&tsens2 10>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphmx2-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

nsphmx2-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6051,14 +6051,14 @@ nsphmx3-thermal {
thermal-sensors = <&tsens2 11>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ nsphmx3-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

nsphmx3-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6071,14 +6071,14 @@ video-thermal {
thermal-sensors = <&tsens1 12>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ video-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

video-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6091,14 +6091,14 @@ ddr-thermal {
thermal-sensors = <&tsens1 13>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ ddr-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

ddr-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6109,14 +6109,14 @@ camera0-thermal {
thermal-sensors = <&tsens1 14>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ camera0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

camera0-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6127,14 +6127,14 @@ camera1-thermal {
thermal-sensors = <&tsens1 15>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ camera1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

camera1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6145,14 +6145,14 @@ aoss2-thermal {
thermal-sensors = <&tsens2 0>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ aoss2-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

aoss2-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6427,14 +6427,14 @@ modem0-thermal {
thermal-sensors = <&tsens2 9>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ modem0-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

modem0-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6445,14 +6445,14 @@ modem1-thermal {
thermal-sensors = <&tsens2 10>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ modem1-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

modem1-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6463,14 +6463,14 @@ modem2-thermal {
thermal-sensors = <&tsens2 11>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ modem2-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

modem2-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};
@@ -6481,14 +6481,14 @@ modem3-thermal {
thermal-sensors = <&tsens2 12>;

trips {
- trip-point0 {
- temperature = <90000>;
- hysteresis = <2000>;
+ modem3-hot {
+ temperature = <110000>;
+ hysteresis = <1000>;
type = "hot";
};

modem3-critical {
- temperature = <110000>;
+ temperature = <115000>;
hysteresis = <0>;
type = "critical";
};

--
2.34.1

Next message: Neil Armstrong: "[PATCH v4 4/4] arm64: dts: qcom: sm8650: drop remaining polling-delay-passive properties"
Previous message: Neil Armstrong: "[PATCH v4 2/4] arm64: dts: qcom: sm8650: setup gpu thermal with higher temperatures"
In reply to: Konrad Dybcio: "Re: [PATCH v4 2/4] arm64: dts: qcom: sm8650: setup gpu thermal with higher temperatures"
Next in thread: Neil Armstrong: "[PATCH v4 4/4] arm64: dts: qcom: sm8650: drop remaining polling-delay-passive properties"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]