diff mbox series

[RESEND,V11,4/4] arm64: dts: imx: add i.MX8QXP thermal support

Message ID 1554881866-26333-4-git-send-email-Anson.Huang@nxp.com (mailing list archive)
State New, archived
Headers show
Series [RESEND,V11,1/4] dt-bindings: fsl: scu: add thermal binding | expand

Commit Message

Anson Huang April 10, 2019, 7:43 a.m. UTC
Add i.MX8QXP CPU thermal zone support.

Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
Changes since V10:
	- remove property "imx,sensor-resource-id" and put HW resource ID inside each
	  thermal-sensors' phandle argument.
---
 arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 33 ++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

Comments

Daniel Lezcano April 15, 2019, 8:14 p.m. UTC | #1
On 10/04/2019 09:43, Anson Huang wrote:
> Add i.MX8QXP CPU thermal zone support.
> 
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> ---
> Changes since V10:
> 	- remove property "imx,sensor-resource-id" and put HW resource ID inside each
> 	  thermal-sensors' phandle argument.
> ---
>  arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 33 ++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> index 70ef3db..cc1d877 100644
> --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> @@ -10,6 +10,7 @@
>  #include <dt-bindings/gpio/gpio.h>
>  #include <dt-bindings/interrupt-controller/arm-gic.h>
>  #include <dt-bindings/pinctrl/pads-imx8qxp.h>
> +#include <dt-bindings/thermal/thermal.h>
>  
>  / {
>  	interrupt-parent = <&gic>;
> @@ -149,6 +150,11 @@
>  		rtc: rtc {
>  			compatible = "fsl,imx8qxp-sc-rtc";
>  		};
> +
> +		tsens: thermal-sensor {
> +			compatible = "fsl,imx8qxp-sc-thermal", "fsl,imx-sc-thermal";
> +			#thermal-sensor-cells = <1>;
> +		};
>  	};
>  
>  	timer {
> @@ -484,4 +490,31 @@
>  			power-domains = <&pd IMX_SC_R_GPIO_7>;
>  		};
>  	};
> +
> +	thermal_zones: thermal-zones {
> +		cpu-thermal0 {
> +			polling-delay-passive = <250>;
> +			polling-delay = <2000>;
> +			thermal-sensors = <&tsens IMX_SC_R_SYSTEM>;
> +			trips {
> +				cpu_alert0: trip0 {
> +					temperature = <107000>;
> +					hysteresis = <2000>;
> +					type = "passive";
> +				};
> +				cpu_crit0: trip1 {
> +					temperature = <127000>;
> +					hysteresis = <2000>;
> +					type = "critical";
> +				};
> +			};
> +			cooling-maps {
> +				map0 {
> +					trip = <&cpu_alert0>;
> +					cooling-device =
> +						<&A35_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;

	<&A35_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
	<&A35_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
	<&A35_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
	<&A35_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;


> +				};
> +			};
> +		};
> +	};
>  };
>
Anson Huang April 16, 2019, 3:24 a.m. UTC | #2
Hi, Daniel

Best Regards!
Anson Huang

> -----Original Message-----
> 
> 
> On 10/04/2019 09:43, Anson Huang wrote:
> > Add i.MX8QXP CPU thermal zone support.
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
> > ---
> > Changes since V10:
> >       - remove property "imx,sensor-resource-id" and put HW resource ID
> inside each
> >         thermal-sensors' phandle argument.
> > ---
> >  arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 33
> > ++++++++++++++++++++++++++++++
> >  1 file changed, 33 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > index 70ef3db..cc1d877 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > @@ -10,6 +10,7 @@
> >  #include <dt-bindings/gpio/gpio.h>
> >  #include <dt-bindings/interrupt-controller/arm-gic.h>
> >  #include <dt-bindings/pinctrl/pads-imx8qxp.h>
> > +#include <dt-bindings/thermal/thermal.h>
> >
> >  / {
> >       interrupt-parent = <&gic>;
> > @@ -149,6 +150,11 @@
> >               rtc: rtc {
> >                       compatible = "fsl,imx8qxp-sc-rtc";
> >               };
> > +
> > +             tsens: thermal-sensor {
> > +                     compatible = "fsl,imx8qxp-sc-thermal", "fsl,imx-sc-thermal";
> > +                     #thermal-sensor-cells = <1>;
> > +             };
> >       };
> >
> >       timer {
> > @@ -484,4 +490,31 @@
> >                       power-domains = <&pd IMX_SC_R_GPIO_7>;
> >               };
> >       };
> > +
> > +     thermal_zones: thermal-zones {
> > +             cpu-thermal0 {
> > +                     polling-delay-passive = <250>;
> > +                     polling-delay = <2000>;
> > +                     thermal-sensors = <&tsens IMX_SC_R_SYSTEM>;
> > +                     trips {
> > +                             cpu_alert0: trip0 {
> > +                                     temperature = <107000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "passive";
> > +                             };
> > +                             cpu_crit0: trip1 {
> > +                                     temperature = <127000>;
> > +                                     hysteresis = <2000>;
> > +                                     type = "critical";
> > +                             };
> > +                     };
> > +                     cooling-maps {
> > +                             map0 {
> > +                                     trip = <&cpu_alert0>;
> > +                                     cooling-device =
> > +                                             <&A35_0 THERMAL_NO_LIMIT
> > + THERMAL_NO_LIMIT>;
> 
>         <&A35_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
>         <&A35_1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
>         <&A35_2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
>         <&A35_3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> 

Fix it in V12, thanks.

Anson.

> 
> > +                             };
> > +                     };
> > +             };
> > +     };
> >  };
> >
> 
> 
> --
> 
> <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.
> linaro.org%2F&amp;data=02%7C01%7Canson.huang%40nxp.com%7Cfca87d
> aa526949586d3608d6c1def467%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C
> 0%7C0%7C636909560658582260&amp;sdata=zJ9pZFVN94TN3tci4sut9A%2Bz
> bnDklZjxy9dwEXCssHk%3D&amp;reserved=0> Linaro.org │ Open source
> software for ARM SoCs
> 
> Follow Linaro:
> <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.
> facebook.com%2Fpages%2FLinaro&amp;data=02%7C01%7Canson.huang%4
> 0nxp.com%7Cfca87daa526949586d3608d6c1def467%7C686ea1d3bc2b4c6fa9
> 2cd99c5c301635%7C0%7C0%7C636909560658582260&amp;sdata=8U9ZV3uL
> dMfzRitq1cl4tvxpFakxFNGW0TDZ42wDuEc%3D&amp;reserved=0> Facebook
> |
> <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Ftwitte
> r.com%2F%23!%2Flinaroorg&amp;data=02%7C01%7Canson.huang%40nxp.c
> om%7Cfca87daa526949586d3608d6c1def467%7C686ea1d3bc2b4c6fa92cd99
> c5c301635%7C0%7C0%7C636909560658582260&amp;sdata=lJ5yngMoW3ELI
> xYQaASAtvGXdNVDxH5BWoKSLwRL0mM%3D&amp;reserved=0> Twitter |
> <https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.
> linaro.org%2Flinaro-
> blog%2F&amp;data=02%7C01%7Canson.huang%40nxp.com%7Cfca87daa526
> 949586d3608d6c1def467%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C
> 0%7C636909560658582260&amp;sdata=hBTpeBSg7JGk0qHbC7DEp2JgN7sVq
> yhC2uzBAG%2BP01s%3D&amp;reserved=0> Blog
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
index 70ef3db..cc1d877 100644
--- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
@@ -10,6 +10,7 @@ 
 #include <dt-bindings/gpio/gpio.h>
 #include <dt-bindings/interrupt-controller/arm-gic.h>
 #include <dt-bindings/pinctrl/pads-imx8qxp.h>
+#include <dt-bindings/thermal/thermal.h>
 
 / {
 	interrupt-parent = <&gic>;
@@ -149,6 +150,11 @@ 
 		rtc: rtc {
 			compatible = "fsl,imx8qxp-sc-rtc";
 		};
+
+		tsens: thermal-sensor {
+			compatible = "fsl,imx8qxp-sc-thermal", "fsl,imx-sc-thermal";
+			#thermal-sensor-cells = <1>;
+		};
 	};
 
 	timer {
@@ -484,4 +490,31 @@ 
 			power-domains = <&pd IMX_SC_R_GPIO_7>;
 		};
 	};
+
+	thermal_zones: thermal-zones {
+		cpu-thermal0 {
+			polling-delay-passive = <250>;
+			polling-delay = <2000>;
+			thermal-sensors = <&tsens IMX_SC_R_SYSTEM>;
+			trips {
+				cpu_alert0: trip0 {
+					temperature = <107000>;
+					hysteresis = <2000>;
+					type = "passive";
+				};
+				cpu_crit0: trip1 {
+					temperature = <127000>;
+					hysteresis = <2000>;
+					type = "critical";
+				};
+			};
+			cooling-maps {
+				map0 {
+					trip = <&cpu_alert0>;
+					cooling-device =
+						<&A35_0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
+				};
+			};
+		};
+	};
 };