Re: [PATCH 1/1] arm64: dts: qcom: sc7180: Changed all sensor values Thermal-zones node
From: Amit Kucheria
Date: Tue Mar 03 2020 - 06:33:03 EST
On Tue, Mar 3, 2020 at 2:47 PM Rajeshwari <rkambl@xxxxxxxxxxxxxx> 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 <rkambl@xxxxxxxxxxxxxx>
> ---
> 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
>