Hi, Eduardo Best Regards! Anson Huang
> -----Original Message----- > From: Eduardo Valentin [mailto:[email protected]] > Sent: 2018年11月30日 0:51 > To: Anson Huang <[email protected]> > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > Aisheng DONG <[email protected]>; Andy Gross > <[email protected]>; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; dl-linux-imx <[email protected]> > Subject: Re: [PATCH V4 4/4] ARM64: dts: imx: add i.MX8QXP thermal support > > On Thu, Nov 29, 2018 at 02:37:47AM +0000, Anson Huang wrote: > > Add i.MX8QXP CPU thermal zone support. > > > > Signed-off-by: Anson Huang <[email protected]> > > --- > > ChangeLog: > > V3->V4: > > *move the tsens node to inside of SCU node as it is belonging to SCU. > > --- > > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 27 > > +++++++++++++++++++++++++++ > > 1 file changed, 27 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > index 9b08688..48f622c 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > > @@ -9,6 +9,7 @@ > > #include <dt-bindings/firmware/imx/rsrc.h> #include > > <dt-bindings/gpio/gpio.h> #include > > <dt-bindings/pinctrl/pads-imx8qxp.h> > > +#include <dt-bindings/thermal/thermal.h> > > > > #include "imx8-ca35.dtsi" > > > > @@ -87,6 +88,12 @@ > > rtc: rtc { > > compatible = "fsl,imx8qxp-sc-rtc"; > > }; > > + > > + tsens: thermal-sensor { > > + compatible = "fsl,imx8qxp-sc-thermal"; > > + tsens-num = <1>; > > + #thermal-sensor-cells = <1>; > > + }; > > }; > > > > adma_subsys: bus@59000000 { > > @@ -410,4 +417,24 @@ > > #size-cells = <1>; > > ranges = <0x5f000000 0x0 0x5f000000 0x1000000>; > > }; > > + > > + thermal_zones: thermal-zones { > > + cpu-thermal0 { > > + polling-delay-passive = <250>; > > + polling-delay = <2000>; > > + thermal-sensors = <&tsens 355>; > > + trips { > > + cpu_alert0: trip0 { > > + temperature = <107000>; > > + hysteresis = <2000>; > > + type = "passive"; > > Passive, but yet no cooling maps? The cpu-freq driver for i.MX8QXP is NOT ready yet, and NOT sure when it will be ready, so I can NOT test the cooling function now, that is why I did NOT add the cooling maps, plan to add the CPU cooling map after cpu-freq is ready. Anson. > > > + }; > > + cpu_crit0: trip1 { > > + temperature = <127000>; > > + hysteresis = <2000>; > > + type = "critical"; > > + }; > > + }; > > + }; > > + }; > > }; > > -- > > 2.7.4 > >
