Re: [PATCH v5 4/4] arm64: dts: ti: am6: Add VTM node

From: J, KEERTHY
Date: Mon Apr 06 2020 - 21:36:19 EST




On 4/7/2020 12:35 AM, Amit Kucheria wrote:
On Tue, Mar 31, 2020 at 1:24 PM Keerthy <j-keerthy@xxxxxx> 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@xxxxxx>
---
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