Subject: [PATCH 0/1] Changed all sensor values Thermal-zones node

Changes:
* changed threshold values to 110C
* changed trip point to critical

Rajeshwari (1):
arm64: dts: qcom: sc7180: Changed all sensor values Thermal-zones node

arch/arm64/boot/dts/qcom/sc7180.dtsi | 52 ++++++++++++++++++------------------
1 file changed, 26 insertions(+), 26 deletions(-)

--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation


Subject: [PATCH 1/1] arm64: dts: qcom: sc7180: Changed all sensor values Thermal-zones node

To enable kernel critical shutdown feature all sensors threshold values
should be 110C to perform shutdown in orderly manner and changed trip
point from hot to critical.

Signed-off-by: Rajeshwari <[email protected]>
---
arch/arm64/boot/dts/qcom/sc7180.dtsi | 52 ++++++++++++++++++------------------
1 file changed, 26 insertions(+), 26 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index d068584..55fd156 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -1952,9 +1952,9 @@

trips {
aoss0_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2007,9 +2007,9 @@

trips {
gpuss0_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2022,9 +2022,9 @@

trips {
gpuss1_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2037,9 +2037,9 @@

trips {
aoss1_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2052,9 +2052,9 @@

trips {
cwlan_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2067,9 +2067,9 @@

trips {
audio_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2082,9 +2082,9 @@

trips {
ddr_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2097,9 +2097,9 @@

trips {
q6_hvx_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2112,9 +2112,9 @@

trips {
camera_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2127,9 +2127,9 @@

trips {
mdm_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2142,9 +2142,9 @@

trips {
mdm_dsp_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2157,9 +2157,9 @@

trips {
npu_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
@@ -2172,9 +2172,9 @@

trips {
video_alert0: trip-point0 {
- temperature = <90000>;
+ temperature = <110000>;
hysteresis = <2000>;
- type = "hot";
+ type = "critical";
};
};
};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

2020-03-03 11:42:46

by Amit Kucheria

[permalink] [raw]
Subject: Re: [PATCH 1/1] arm64: dts: qcom: sc7180: Changed all sensor values Thermal-zones node

On Tue, Mar 3, 2020 at 2:47 PM Rajeshwari <[email protected]> wrote:
>
> To enable kernel critical shutdown feature all sensors threshold values
> should be 110C to perform shutdown in orderly manner and changed trip
> point from hot to critical.

IMO, we should keep the hot trip at 90 so we can potentially use it
for notifications via the ops->notify callback in the future.

Just add a new critical trip section to all these non-CPU thermal
zones if you want to trigger the orderly shutdown when one of them
reaches the threshold.

> Signed-off-by: Rajeshwari <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 52 ++++++++++++++++++------------------
> 1 file changed, 26 insertions(+), 26 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index d068584..55fd156 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -1952,9 +1952,9 @@
>
> trips {
> aoss0_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2007,9 +2007,9 @@
>
> trips {
> gpuss0_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2022,9 +2022,9 @@
>
> trips {
> gpuss1_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2037,9 +2037,9 @@
>
> trips {
> aoss1_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2052,9 +2052,9 @@
>
> trips {
> cwlan_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2067,9 +2067,9 @@
>
> trips {
> audio_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2082,9 +2082,9 @@
>
> trips {
> ddr_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2097,9 +2097,9 @@
>
> trips {
> q6_hvx_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2112,9 +2112,9 @@
>
> trips {
> camera_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2127,9 +2127,9 @@
>
> trips {
> mdm_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2142,9 +2142,9 @@
>
> trips {
> mdm_dsp_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2157,9 +2157,9 @@
>
> trips {
> npu_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> @@ -2172,9 +2172,9 @@
>
> trips {
> video_alert0: trip-point0 {
> - temperature = <90000>;
> + temperature = <110000>;
> hysteresis = <2000>;
> - type = "hot";
> + type = "critical";
> };
> };
> };
> --
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>