| Message ID | 20200331075356.19171-5-j-keerthy@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Daniel Lezcano |
| Headers | show |
| Series | thermal: k3: Add support for bandgap sensors | expand |
On Tue, Mar 31, 2020 at 1:24 PM Keerthy <j-keerthy@ti.com> wrote: > > VTM stands for voltage and thermal management. Add the vtm node and > the associated thermal zones on the SoC. > > Signed-off-by: Keerthy <j-keerthy@ti.com> > --- > arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > index f4227e2743f2..54a133fa1bf2 100644 > --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > @@ -89,4 +89,15 @@ > clocks = <&k3_clks 59 0>; > clock-names = "gpio"; > }; > + > + wkup_vtm0: thermal@42050000 { > + compatible = "ti,am654-vtm"; > + reg = <0x42050000 0x25c>; > + power-domains = <&k3_pds 80 TI_SCI_PD_EXCLUSIVE>; > + #thermal-sensor-cells = <1>; > + }; > + Make this patch 3 > + thermal_zones: thermal-zones { > + #include "k3-am654-industrial-thermal.dtsi" > + }; > }; Move this with what is currently patch 3. > -- > 2.17.1 >
Hi Keerthy, Should the subject refer to am65x instead of am6 since your dtsi file is called k3-am65-wakeup.dtsi? Regards, Amit On Tue, Mar 31, 2020 at 1:24 PM Keerthy <j-keerthy@ti.com> wrote: > > VTM stands for voltage and thermal management. Add the vtm node and > the associated thermal zones on the SoC. > > Signed-off-by: Keerthy <j-keerthy@ti.com> > --- > arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > index f4227e2743f2..54a133fa1bf2 100644 > --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi > @@ -89,4 +89,15 @@ > clocks = <&k3_clks 59 0>; > clock-names = "gpio"; > }; > + > + wkup_vtm0: thermal@42050000 { > + compatible = "ti,am654-vtm"; > + reg = <0x42050000 0x25c>; > + power-domains = <&k3_pds 80 TI_SCI_PD_EXCLUSIVE>; > + #thermal-sensor-cells = <1>; > + }; > + > + thermal_zones: thermal-zones { > + #include "k3-am654-industrial-thermal.dtsi" > + }; > }; > -- > 2.17.1 >
On 4/7/2020 12:35 AM, Amit Kucheria wrote: > On Tue, Mar 31, 2020 at 1:24 PM Keerthy <j-keerthy@ti.com> wrote: >> >> VTM stands for voltage and thermal management. Add the vtm node and >> the associated thermal zones on the SoC. >> >> Signed-off-by: Keerthy <j-keerthy@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 11 +++++++++++ >> 1 file changed, 11 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi >> index f4227e2743f2..54a133fa1bf2 100644 >> --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi >> @@ -89,4 +89,15 @@ >> clocks = <&k3_clks 59 0>; >> clock-names = "gpio"; >> }; >> + >> + wkup_vtm0: thermal@42050000 { >> + compatible = "ti,am654-vtm"; >> + reg = <0x42050000 0x25c>; >> + power-domains = <&k3_pds 80 TI_SCI_PD_EXCLUSIVE>; >> + #thermal-sensor-cells = <1>; >> + }; >> + > > Make this patch 3 > >> + thermal_zones: thermal-zones { >> + #include "k3-am654-industrial-thermal.dtsi" >> + }; >> }; > > Move this with what is currently patch 3. Agreed on all the comments on patch 3 & 4. I will incorporate them. - Keerthy > >> -- >> 2.17.1 >>
diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi index f4227e2743f2..54a133fa1bf2 100644 --- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi @@ -89,4 +89,15 @@ clocks = <&k3_clks 59 0>; clock-names = "gpio"; }; + + wkup_vtm0: thermal@42050000 { + compatible = "ti,am654-vtm"; + reg = <0x42050000 0x25c>; + power-domains = <&k3_pds 80 TI_SCI_PD_EXCLUSIVE>; + #thermal-sensor-cells = <1>; + }; + + thermal_zones: thermal-zones { + #include "k3-am654-industrial-thermal.dtsi" + }; };
VTM stands for voltage and thermal management. Add the vtm node and the associated thermal zones on the SoC. Signed-off-by: Keerthy <j-keerthy@ti.com> --- arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 11 +++++++++++ 1 file changed, 11 insertions(+)