RE: [EXT] Re: [PATCH RESEND V11 4/4] arm64: dts: imx: add i.MX8QXP thermal support

[Anson Huang <anson.huang@xxxxxxx>]

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@xxxxxxx>
> > ---
> > 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