* [PATCH v7 1/2] arm64: dts: ti: k3-j721s2-mcu-wakeup: Add support for ADC nodes
2023-03-16 9:51 [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
@ 2023-03-16 9:51 ` Bhavya Kapoor
2023-03-16 9:51 ` [PATCH v7 2/2] arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information for ADC Bhavya Kapoor
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Bhavya Kapoor @ 2023-03-16 9:51 UTC (permalink / raw)
To: devicetree, robh+dt
Cc: nm, vigneshr, kristo, krzysztof.kozlowski+dt, linux-arm-kernel,
linux-kernel, b-kapoor
J721s2 has two instances of 8 channel ADCs in MCU domain. Add support
for both ADC nodes.
Signed-off-by: Bhavya Kapoor <b-kapoor@ti.com>
---
.../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 40 +++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
index 0af242aa9816..5da5f0cf7009 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
@@ -306,4 +306,44 @@ cpts@3d000 {
ti,cpts-periodic-outputs = <2>;
};
};
+
+ tscadc0: tscadc@40200000 {
+ compatible = "ti,am3359-tscadc";
+ reg = <0x00 0x40200000 0x00 0x1000>;
+ interrupts = <GIC_SPI 860 IRQ_TYPE_LEVEL_HIGH>;
+ power-domains = <&k3_pds 0 TI_SCI_PD_EXCLUSIVE>;
+ clocks = <&k3_clks 0 0>;
+ assigned-clocks = <&k3_clks 0 2>;
+ assigned-clock-rates = <60000000>;
+ clock-names = "fck";
+ dmas = <&main_udmap 0x7400>,
+ <&main_udmap 0x7401>;
+ dma-names = "fifo0", "fifo1";
+ status = "disabled";
+
+ adc {
+ #io-channel-cells = <1>;
+ compatible = "ti,am3359-adc";
+ };
+ };
+
+ tscadc1: tscadc@40210000 {
+ compatible = "ti,am3359-tscadc";
+ reg = <0x00 0x40210000 0x00 0x1000>;
+ interrupts = <GIC_SPI 861 IRQ_TYPE_LEVEL_HIGH>;
+ power-domains = <&k3_pds 1 TI_SCI_PD_EXCLUSIVE>;
+ clocks = <&k3_clks 1 0>;
+ assigned-clocks = <&k3_clks 1 2>;
+ assigned-clock-rates = <60000000>;
+ clock-names = "fck";
+ dmas = <&main_udmap 0x7402>,
+ <&main_udmap 0x7403>;
+ dma-names = "fifo0", "fifo1";
+ status = "disabled";
+
+ adc {
+ #io-channel-cells = <1>;
+ compatible = "ti,am3359-adc";
+ };
+ };
};
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH v7 2/2] arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information for ADC
2023-03-16 9:51 [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
2023-03-16 9:51 ` [PATCH v7 1/2] arm64: dts: ti: k3-j721s2-mcu-wakeup: " Bhavya Kapoor
@ 2023-03-16 9:51 ` Bhavya Kapoor
2023-03-16 10:47 ` [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Bhavya Kapoor @ 2023-03-16 9:51 UTC (permalink / raw)
To: devicetree, robh+dt
Cc: nm, vigneshr, kristo, krzysztof.kozlowski+dt, linux-arm-kernel,
linux-kernel, b-kapoor
J721s2 has two instances of 8 channel ADCs in MCU domain. Add pinmux
information for both ADC nodes.
Signed-off-by: Bhavya Kapoor <b-kapoor@ti.com>
---
.../dts/ti/k3-j721s2-common-proc-board.dts | 44 +++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
index a7aa6cf08acd..b4b9edfe2d12 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
@@ -197,6 +197,32 @@ mcu_mcan1_gpio_pins_default: mcu-mcan1-gpio-pins-default {
J721S2_WKUP_IOPAD(0x0c8, PIN_INPUT, 7) /* (C28) WKUP_GPIO0_2 */
>;
};
+
+ mcu_adc0_pins_default: mcu-adc0-pins-default {
+ pinctrl-single,pins = <
+ J721S2_WKUP_IOPAD(0x134, PIN_INPUT, 0) /* (L25) MCU_ADC0_AIN0 */
+ J721S2_WKUP_IOPAD(0x138, PIN_INPUT, 0) /* (K25) MCU_ADC0_AIN1 */
+ J721S2_WKUP_IOPAD(0x13c, PIN_INPUT, 0) /* (M24) MCU_ADC0_AIN2 */
+ J721S2_WKUP_IOPAD(0x140, PIN_INPUT, 0) /* (L24) MCU_ADC0_AIN3 */
+ J721S2_WKUP_IOPAD(0x144, PIN_INPUT, 0) /* (L27) MCU_ADC0_AIN4 */
+ J721S2_WKUP_IOPAD(0x148, PIN_INPUT, 0) /* (K24) MCU_ADC0_AIN5 */
+ J721S2_WKUP_IOPAD(0x14c, PIN_INPUT, 0) /* (M27) MCU_ADC0_AIN6 */
+ J721S2_WKUP_IOPAD(0x150, PIN_INPUT, 0) /* (M26) MCU_ADC0_AIN7 */
+ >;
+ };
+
+ mcu_adc1_pins_default: mcu-adc1-pins-default {
+ pinctrl-single,pins = <
+ J721S2_WKUP_IOPAD(0x154, PIN_INPUT, 0) /* (P25) MCU_ADC1_AIN0 */
+ J721S2_WKUP_IOPAD(0x158, PIN_INPUT, 0) /* (R25) MCU_ADC1_AIN1 */
+ J721S2_WKUP_IOPAD(0x15c, PIN_INPUT, 0) /* (P28) MCU_ADC1_AIN2 */
+ J721S2_WKUP_IOPAD(0x160, PIN_INPUT, 0) /* (P27) MCU_ADC1_AIN3 */
+ J721S2_WKUP_IOPAD(0x164, PIN_INPUT, 0) /* (N25) MCU_ADC1_AIN4 */
+ J721S2_WKUP_IOPAD(0x168, PIN_INPUT, 0) /* (P26) MCU_ADC1_AIN5 */
+ J721S2_WKUP_IOPAD(0x16c, PIN_INPUT, 0) /* (N26) MCU_ADC1_AIN6 */
+ J721S2_WKUP_IOPAD(0x170, PIN_INPUT, 0) /* (N27) MCU_ADC1_AIN7 */
+ >;
+ };
};
&main_gpio2 {
@@ -309,3 +335,21 @@ &mcu_mcan1 {
pinctrl-0 = <&mcu_mcan1_pins_default>;
phys = <&transceiver2>;
};
+
+&tscadc0 {
+ pinctrl-0 = <&mcu_adc0_pins_default>;
+ pinctrl-names = "default";
+ status = "okay";
+ adc {
+ ti,adc-channels = <0 1 2 3 4 5 6 7>;
+ };
+};
+
+&tscadc1 {
+ pinctrl-0 = <&mcu_adc1_pins_default>;
+ pinctrl-names = "default";
+ status = "okay";
+ adc {
+ ti,adc-channels = <0 1 2 3 4 5 6 7>;
+ };
+};
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes
2023-03-16 9:51 [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
2023-03-16 9:51 ` [PATCH v7 1/2] arm64: dts: ti: k3-j721s2-mcu-wakeup: " Bhavya Kapoor
2023-03-16 9:51 ` [PATCH v7 2/2] arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information for ADC Bhavya Kapoor
@ 2023-03-16 10:47 ` Bhavya Kapoor
2023-03-17 4:13 ` Vignesh Raghavendra
2023-03-20 22:57 ` Nishanth Menon
4 siblings, 0 replies; 6+ messages in thread
From: Bhavya Kapoor @ 2023-03-16 10:47 UTC (permalink / raw)
To: devicetree, robh+dt
Cc: nm, vigneshr, kristo, krzysztof.kozlowski+dt, linux-arm-kernel,
linux-kernel
On 16/03/23 15:21, Bhavya Kapoor wrote:
> J721s2 has two instances of 8 channel ADCs in MCU domain. Add support
> for both ADC nodes in dtsi file. Add pinmux information for both
> instances of ADC in board dts file.
Link to V6 :
https://lore.kernel.org/all/20230314112939.139894-1-b-kapoor@ti.com/
> Changelog v6->v7:
> - Created 2 commits for k3-j721s2-mcu-wakeup.dtsi and
> k3-j721s2-common-proc-board.dts instead of a single commit
>
> Bhavya Kapoor (2):
> arm64: dts: ti: k3-j721s2-mcu-wakeup: Add support for ADC nodes
> arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information
> for ADC
>
> .../dts/ti/k3-j721s2-common-proc-board.dts | 44 +++++++++++++++++++
> .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 40 +++++++++++++++++
> 2 files changed, 84 insertions(+)
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes
2023-03-16 9:51 [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
` (2 preceding siblings ...)
2023-03-16 10:47 ` [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
@ 2023-03-17 4:13 ` Vignesh Raghavendra
2023-03-20 22:57 ` Nishanth Menon
4 siblings, 0 replies; 6+ messages in thread
From: Vignesh Raghavendra @ 2023-03-17 4:13 UTC (permalink / raw)
To: Bhavya Kapoor, devicetree, robh+dt
Cc: nm, kristo, krzysztof.kozlowski+dt, linux-arm-kernel, linux-kernel
On 16/03/23 15:21, Bhavya Kapoor wrote:
> J721s2 has two instances of 8 channel ADCs in MCU domain. Add support
> for both ADC nodes in dtsi file. Add pinmux information for both
> instances of ADC in board dts file.
>
> Changelog v6->v7:
> - Created 2 commits for k3-j721s2-mcu-wakeup.dtsi and
> k3-j721s2-common-proc-board.dts instead of a single commit
>
> Bhavya Kapoor (2):
> arm64: dts: ti: k3-j721s2-mcu-wakeup: Add support for ADC nodes
> arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information
> for ADC
>
> .../dts/ti/k3-j721s2-common-proc-board.dts | 44 +++++++++++++++++++
> .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 40 +++++++++++++++++
> 2 files changed, 84 insertions(+)
>
LGTM
Reviewed-by: Vignesh Raghavendra <vigneshr@ti.com>
--
Regards
Vignesh
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes
2023-03-16 9:51 [PATCH v7 0/2] arm64: dts: ti: k3-j721s2: Add support for ADC nodes Bhavya Kapoor
` (3 preceding siblings ...)
2023-03-17 4:13 ` Vignesh Raghavendra
@ 2023-03-20 22:57 ` Nishanth Menon
4 siblings, 0 replies; 6+ messages in thread
From: Nishanth Menon @ 2023-03-20 22:57 UTC (permalink / raw)
To: devicetree, robh+dt, Bhavya Kapoor
Cc: Nishanth Menon, vigneshr, kristo, krzysztof.kozlowski+dt,
linux-arm-kernel, linux-kernel
Hi Bhavya Kapoor,
On Thu, 16 Mar 2023 15:21:44 +0530, Bhavya Kapoor wrote:
> J721s2 has two instances of 8 channel ADCs in MCU domain. Add support
> for both ADC nodes in dtsi file. Add pinmux information for both
> instances of ADC in board dts file.
>
> Changelog v6->v7:
> - Created 2 commits for k3-j721s2-mcu-wakeup.dtsi and
> k3-j721s2-common-proc-board.dts instead of a single commit
>
> [...]
I have applied the following to branch ti-k3-dts-next on [1].
Thank you!
[1/2] arm64: dts: ti: k3-j721s2-mcu-wakeup: Add support for ADC nodes
commit: 4beba5cf9ecc099d59e6fe8a03254cd995b80123
[2/2] arm64: dts: ti: k3-j721s2-common-proc-board: Add pinmux information for ADC
commit: cf2aacfe5f48b8c91d148dd7a3d3814b8ae04463
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
[1] git://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 6+ messages in thread