2021-03-09 13:09:19

by Vignesh Raghavendra

[permalink] [raw]
Subject: [PATCH] arm64: dts: ti: k3-am64-main: Add ADC nodes

AM64 SoC has a single ADC IP with 8 channels. Add DT node for the same.

Default usecase is to control ADC from non Linux core on the system on
AM642 GP EVM, therefore mark the node as reserved in k3-am642-evm.dts
file. ADC lines are not pinned out on AM642 SK board, therefore disable
the node in k3-am642-sk.dts file.

Signed-off-by: Vignesh Raghavendra <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 17 +++++++++++++++++
arch/arm64/boot/dts/ti/k3-am642-evm.dts | 5 +++++
arch/arm64/boot/dts/ti/k3-am642-sk.dts | 4 ++++
3 files changed, 26 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index bcec4fa444b5..1bb1e292547d 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -427,4 +427,21 @@ ospi0: spi@fc40000 {
power-domains = <&k3_pds 75 TI_SCI_PD_EXCLUSIVE>;
};
};
+
+ tscadc0: tscadc@28001000 {
+ compatible = "ti,am654-tscadc", "ti,am3359-tscadc";
+ reg = <0x00 0x28001000 0x00 0x1000>;
+ interrupts = <GIC_SPI 96 IRQ_TYPE_LEVEL_HIGH>;
+ power-domains = <&k3_pds 0 TI_SCI_PD_EXCLUSIVE>;
+ clocks = <&k3_clks 0 0>;
+ assigned-clocks = <&k3_clks 0 0>;
+ assigned-clock-parents = <&k3_clks 0 3>;
+ assigned-clock-rates = <60000000>;
+ clock-names = "adc_tsc_fck";
+
+ adc {
+ #io-channel-cells = <1>;
+ compatible = "ti,am654-adc", "ti,am3359-adc";
+ };
+ };
};
diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts b/arch/arm64/boot/dts/ti/k3-am642-evm.dts
index 7dd8e94b108d..7154d34faf4c 100644
--- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts
@@ -280,3 +280,8 @@ flash@0{
#size-cells = <1>;
};
};
+
+&tscadc0 {
+ /* ADC is reserved for R5 usage */
+ status = "reserved";
+};
diff --git a/arch/arm64/boot/dts/ti/k3-am642-sk.dts b/arch/arm64/boot/dts/ti/k3-am642-sk.dts
index fc27470fc083..28059f07bef1 100644
--- a/arch/arm64/boot/dts/ti/k3-am642-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am642-sk.dts
@@ -207,3 +207,7 @@ flash@0{
#size-cells = <1>;
};
};
+
+&tscadc0 {
+ status = "disabled";
+};
--
2.30.1


2021-03-11 11:38:50

by Lokesh Vutla

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: Add ADC nodes



On 09/03/21 6:37 pm, Vignesh Raghavendra wrote:
> AM64 SoC has a single ADC IP with 8 channels. Add DT node for the same.
>
> Default usecase is to control ADC from non Linux core on the system on
> AM642 GP EVM, therefore mark the node as reserved in k3-am642-evm.dts
> file. ADC lines are not pinned out on AM642 SK board, therefore disable
> the node in k3-am642-sk.dts file.
>
> Signed-off-by: Vignesh Raghavendra <[email protected]>

Reviewed-by: Lokesh Vutla <[email protected]>

Thanks and regards,
Lokesh

2021-03-16 15:40:35

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: Add ADC nodes

On 17:05-20210311, Lokesh Vutla wrote:
>
>
> On 09/03/21 6:37 pm, Vignesh Raghavendra wrote:
> > AM64 SoC has a single ADC IP with 8 channels. Add DT node for the same.
> >
> > Default usecase is to control ADC from non Linux core on the system on
> > AM642 GP EVM, therefore mark the node as reserved in k3-am642-evm.dts
> > file. ADC lines are not pinned out on AM642 SK board, therefore disable
> > the node in k3-am642-sk.dts file.
> >
> > Signed-off-by: Vignesh Raghavendra <[email protected]>
>
> Reviewed-by: Lokesh Vutla <[email protected]>


Looks like the patch wont apply anymore after all the cpsw nodes got in.
could you rebase and repost?

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D