Hi, Eduardo Best Regards! Anson Huang > -----Original Message----- > From: Eduardo Valentin [mailto:edubezval@xxxxxxxxx] > Sent: 2018年11月30日 0:51 > To: Anson Huang <anson.huang@xxxxxxx> > Cc: rui.zhang@xxxxxxxxx; daniel.lezcano@xxxxxxxxxx; robh+dt@xxxxxxxxxx; > mark.rutland@xxxxxxx; catalin.marinas@xxxxxxx; will.deacon@xxxxxxx; > Aisheng DONG <aisheng.dong@xxxxxxx>; Andy Gross > <andy.gross@xxxxxxxxxx>; heiko@xxxxxxxxx; horms+renesas@xxxxxxxxxxxx; > arnd@xxxxxxxx; amit.kucheria@xxxxxxxxxx; enric.balletbo@xxxxxxxxxxxxx; > ezequiel@xxxxxxxxxxxxx; olof@xxxxxxxxx; bjorn.andersson@xxxxxxxxxx; > shawnguo@xxxxxxxxxx; LW@xxxxxxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx> > 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 <Anson.Huang@xxxxxxx> > > --- > > 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 > >
