2023-06-07 19:25:17

by Robert Marko

Subject: [PATCH] arm64: dts: qcom: ipq8074: add critical thermal trips

According to bindings, thermal zones must have associated trips as well.
Since we currently dont have CPUFreq support and thus no passive cooling
lets start by defining critical trips to protect the devices against
severe overheating.

Signed-off-by: Robert Marko <[email protected]>
---
arch/arm64/boot/dts/qcom/ipq8074.dtsi | 96 +++++++++++++++++++++++++++
1 file changed, 96 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/ipq8074.dtsi b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
index 996a9b9ff8bb..74551f51f097 100644
--- a/arch/arm64/boot/dts/qcom/ipq8074.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
@@ -1318,6 +1318,14 @@ nss-top-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 4>;
+
+ trips {
+ nss-top-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

nss0-thermal {
@@ -1325,6 +1333,14 @@ nss0-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 5>;
+
+ trips {
+ nss-0-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

nss1-thermal {
@@ -1332,6 +1348,14 @@ nss1-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 6>;
+
+ trips {
+ nss-1-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

wcss-phya0-thermal {
@@ -1339,6 +1363,14 @@ wcss-phya0-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 7>;
+
+ trips {
+ wcss-phya0-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

wcss-phya1-thermal {
@@ -1346,6 +1378,14 @@ wcss-phya1-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 8>;
+
+ trips {
+ wcss-phya1-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

cpu0_thermal: cpu0-thermal {
@@ -1353,6 +1393,14 @@ cpu0_thermal: cpu0-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 9>;
+
+ trips {
+ cpu0-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

cpu1_thermal: cpu1-thermal {
@@ -1360,6 +1408,14 @@ cpu1_thermal: cpu1-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 10>;
+
+ trips {
+ cpu1-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

cpu2_thermal: cpu2-thermal {
@@ -1367,6 +1423,14 @@ cpu2_thermal: cpu2-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 11>;
+
+ trips {
+ cpu2-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

cpu3_thermal: cpu3-thermal {
@@ -1374,6 +1438,14 @@ cpu3_thermal: cpu3-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 12>;
+
+ trips {
+ cpu3-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

cluster_thermal: cluster-thermal {
@@ -1381,6 +1453,14 @@ cluster_thermal: cluster-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 13>;
+
+ trips {
+ cluster-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

wcss-phyb0-thermal {
@@ -1388,6 +1468,14 @@ wcss-phyb0-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 14>;
+
+ trips {
+ wcss-phyb0-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};

wcss-phyb1-thermal {
@@ -1395,6 +1483,14 @@ wcss-phyb1-thermal {
polling-delay = <1000>;

thermal-sensors = <&tsens 15>;
+
+ trips {
+ wcss-phyb1-crit {
+ temperature = <110000>;
+ hysteresis = <1000>;
+ type = "critical";
+ };
+ };
};
};
};
--
2.40.1

2023-06-14 00:02:28

by Bjorn Andersson

Subject: Re: [PATCH] arm64: dts: qcom: ipq8074: add critical thermal trips

On Wed, 7 Jun 2023 20:44:48 +0200, Robert Marko wrote:
> According to bindings, thermal zones must have associated trips as well.
> Since we currently dont have CPUFreq support and thus no passive cooling
> lets start by defining critical trips to protect the devices against
> severe overheating.
>
>

Applied, thanks!

[1/1] arm64: dts: qcom: ipq8074: add critical thermal trips
commit: 56d3067cb694ba60d654e7f5ef231b6fabc4697f

Best regards,
--
Bjorn Andersson <[email protected]>