* [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:37 ` Krzysztof Kozlowski
2025-05-29 13:34 ` [PATCH 02/13] arm64: dts: ti: k3-am62-lp-sk: " Siddharth Vadapalli
` (11 subsequent siblings)
12 siblings, 1 reply; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "cpsw3g" device-tree node in the SoC file,
namely "k3-am62-main.dtsi", enable it in the board file. The motivation
for this change is that of following the existing convention of disabling
nodes in the SoC file and only enabling the required ones in the board
file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
index 72b09f9c69d8..999f5baaba1a 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
@@ -585,6 +585,7 @@ &usb1 {
};
&cpsw3g {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&rgmii1_pins_default>, <&spe_pins_default>,
<&gbe_pmx_obsclk>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file
2025-05-29 13:34 ` [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file Siddharth Vadapalli
@ 2025-05-29 13:37 ` Krzysztof Kozlowski
2025-05-30 4:52 ` Siddharth Vadapalli
0 siblings, 1 reply; 20+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-29 13:37 UTC (permalink / raw)
To: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk
On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> In preparation for disabling "cpsw3g" device-tree node in the SoC file,
> namely "k3-am62-main.dtsi", enable it in the board file. The motivation
> for this change is that of following the existing convention of disabling
> nodes in the SoC file and only enabling the required ones in the board
> file.
That's not separate commit. You disable in DTSI and enable in DTS in the
same one, to avoid any duplicated statuses like in this case (even if
one is default).
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file
2025-05-29 13:37 ` Krzysztof Kozlowski
@ 2025-05-30 4:52 ` Siddharth Vadapalli
0 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-30 4:52 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt, devicetree, linux-kernel, linux-arm-kernel, srk
On Thu, May 29, 2025 at 03:37:22PM +0200, Krzysztof Kozlowski wrote:
> On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> > In preparation for disabling "cpsw3g" device-tree node in the SoC file,
> > namely "k3-am62-main.dtsi", enable it in the board file. The motivation
> > for this change is that of following the existing convention of disabling
> > nodes in the SoC file and only enabling the required ones in the board
> > file.
>
>
> That's not separate commit. You disable in DTSI and enable in DTS in the
> same one, to avoid any duplicated statuses like in this case (even if
> one is default).
I will squash the commits. Thank you for reviewing the patch.
Regards,
Siddharth.
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 02/13] arm64: dts: ti: k3-am62-lp-sk: Enable "cpsw3g" in the board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 03/13] arm64: dts: ti: k3-am625-sk: " Siddharth Vadapalli
` (10 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "cpsw3g" device-tree node in the SoC file,
namely "k3-am62-main.dtsi", enable it in the board file. The motivation
for this change is that of following the existing convention of disabling
nodes in the SoC file and only enabling the required ones in the board
file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts b/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
index aafdb90c0eb7..cec77fba24e6 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62-lp-sk.dts
@@ -149,6 +149,10 @@ &sdhci1 {
vqmmc-supply = <&vddshv_sdio>;
};
+&cpsw3g {
+ status = "okay";
+};
+
&cpsw_port2 {
status = "disabled";
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 03/13] arm64: dts: ti: k3-am625-sk: Enable "cpsw3g" in the board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 01/13] arm64: dts: ti: k3-am625-beagleplay: Enable "cpsw3g" in the board file Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 02/13] arm64: dts: ti: k3-am62-lp-sk: " Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 04/13] arm64: dts: ti: k3-am62-main: Disable "cpsw3g" in the SoC file Siddharth Vadapalli
` (9 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "cpsw3g" device-tree node in the SoC file,
namely "k3-am62-main.dtsi", enable it in the board file. The motivation
for this change is that of following the existing convention of disabling
nodes in the SoC file and only enabling the required ones in the board
file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am625-sk.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
index 2fbfa3719345..9c2258dfd08d 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
@@ -201,6 +201,7 @@ &sdhci1 {
};
&cpsw3g {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&main_rgmii1_pins_default>, <&main_rgmii2_pins_default>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 04/13] arm64: dts: ti: k3-am62-main: Disable "cpsw3g" in the SoC file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (2 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 03/13] arm64: dts: ti: k3-am625-sk: " Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 05/13] arm64: dts: ti: k3-am654-base-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
` (8 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Following the existing convention of disabling nodes in the SoC file and
enabling only the required ones in the board file, disable "cpsw3g" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
index 9e0b6eee9ac7..3d8650e7c80b 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
@@ -724,6 +724,8 @@ cpsw3g: ethernet@8000000 {
dma-names = "tx0", "tx1", "tx2", "tx3", "tx4", "tx5", "tx6",
"tx7", "rx";
+ status = "disabled";
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 05/13] arm64: dts: ti: k3-am654-base-board: Enable "mcu_cpsw" in board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (3 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 04/13] arm64: dts: ti: k3-am62-main: Disable "cpsw3g" in the SoC file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 06/13] arm64: dts: ti: k3-am65-mcu: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
` (7 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "cpsw3g" device-tree node in the SoC file,
namely "k3-am65-mcu.dtsi", enable it in the board file. The motivation
for this change is that of following the existing convention of disabling
nodes in the SoC file and only enabling the required ones in the board
file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index c30425960398..d2632d1f8eb7 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -606,6 +606,7 @@ partition@3fe0000 {
};
&mcu_cpsw {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&mcu_cpsw_pins_default>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 06/13] arm64: dts: ti: k3-am65-mcu: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (4 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 05/13] arm64: dts: ti: k3-am654-base-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status Siddharth Vadapalli
` (6 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Following the existing convention of disabling nodes in the SoC file and
enabling only the required ones in the board file, disable "mcu_cpsw" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
index 7cf1f646500a..cd0b796c5f8f 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
@@ -354,6 +354,8 @@ mcu_cpsw: ethernet@46000000 {
"tx4", "tx5", "tx6", "tx7",
"rx";
+ status = "disabled";
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (5 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 06/13] arm64: dts: ti: k3-am65-mcu: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:38 ` Krzysztof Kozlowski
2025-05-29 13:34 ` [PATCH 08/13] arm64: dts: ti: k3-j7200-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
` (5 subsequent siblings)
12 siblings, 1 reply; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Now that the SoC file "k3-am65-mcu.dtsi" disables the "mcu_cpsw" node by
default, following the existing convention of disabling nodes in the SoC
file, remove the redundant section for disabling the "mcu_cpsw" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index e5136ed94765..48e740eb0088 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -475,10 +475,6 @@ &main_i2c3 {
#size-cells = <0>;
};
-&mcu_cpsw {
- status = "disabled";
-};
-
&sdhci1 {
status = "okay";
pinctrl-names = "default";
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status
2025-05-29 13:34 ` [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status Siddharth Vadapalli
@ 2025-05-29 13:38 ` Krzysztof Kozlowski
2025-05-30 4:54 ` Siddharth Vadapalli
0 siblings, 1 reply; 20+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-29 13:38 UTC (permalink / raw)
To: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk
On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> Now that the SoC file "k3-am65-mcu.dtsi" disables the "mcu_cpsw" node by
> default, following the existing convention of disabling nodes in the SoC
> file, remove the redundant section for disabling the "mcu_cpsw" node.
It became redundant when you disabled it, so this must be part of that
commit. Do not create redundant code with some artificial split of one
logical work.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status
2025-05-29 13:38 ` Krzysztof Kozlowski
@ 2025-05-30 4:54 ` Siddharth Vadapalli
0 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-30 4:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt, devicetree, linux-kernel, linux-arm-kernel, srk
On Thu, May 29, 2025 at 03:38:51PM +0200, Krzysztof Kozlowski wrote:
> On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> > Now that the SoC file "k3-am65-mcu.dtsi" disables the "mcu_cpsw" node by
> > default, following the existing convention of disabling nodes in the SoC
> > file, remove the redundant section for disabling the "mcu_cpsw" node.
>
> It became redundant when you disabled it, so this must be part of that
> commit. Do not create redundant code with some artificial split of one
> logical work.
Sure, I will squash the commits. I tend to split the commits since it is
easier for maintainers to squash commits (if required) than to split them.
Thank you for reviewing the patch.
Regards,
Siddharth.
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 08/13] arm64: dts: ti: k3-j7200-common-proc-board: Enable "mcu_cpsw" in board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (6 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 07/13] arm64: dts: ti: iot2050-common: Remove redundant "mcu_cpsw" node status Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 09/13] arm64: dts: ti: k3-j7200-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
` (4 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "mcu_cpsw" device-tree node in the SoC file,
namely "k3-j7200-mcu-wakeup.dtsi", enable it in the board file.
The motivation for this change is that of following the existing convention
of disabling nodes in the SoC file and only enabling the required ones in
the board file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
index f684ce6ad9ad..021cdeb84ad1 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
@@ -321,6 +321,7 @@ &wkup_gpio0 {
};
&mcu_cpsw {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&mcu_cpsw_pins_default>, <&mcu_mdio_pins_default>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 09/13] arm64: dts: ti: k3-j7200-mcu-wakeup: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (7 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 08/13] arm64: dts: ti: k3-j7200-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 10/13] arm64: dts: ti: k3-j721e-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
` (3 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Following the existing convention of disabling nodes in the SoC file and
enabling only the required ones in the board file, disable "mcu_cpsw" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 56ab144fea07..604295092d06 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -432,6 +432,8 @@ mcu_cpsw: ethernet@46000000 {
"tx4", "tx5", "tx6", "tx7",
"rx";
+ status = "disabled";
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 10/13] arm64: dts: ti: k3-j721e-common-proc-board: Enable "mcu_cpsw" in board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (8 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 09/13] arm64: dts: ti: k3-j7200-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 11/13] arm64: dts: ti: k3-j721e-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
` (2 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "mcu_cpsw" device-tree node in the SoC file,
namely "k3-j721e-mcu-wakeup.dtsi", enable it in the board file.
The motivation for this change is that of following the existing convention
of disabling nodes in the SoC file and only enabling the required ones in
the board file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
index 45311438315f..eda85dafb794 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
@@ -767,6 +767,7 @@ exp5: gpio@20 {
};
&mcu_cpsw {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&mcu_cpsw_pins_default>, <&mcu_mdio_pins_default>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 11/13] arm64: dts: ti: k3-j721e-mcu-wakeup: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (9 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 10/13] arm64: dts: ti: k3-j721e-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 12/13] arm64: dts: ti: k3-j721s2-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Following the existing convention of disabling nodes in the SoC file and
enabling only the required ones in the board file, disable "mcu_cpsw" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
index b02142b2b460..dd923540ca0a 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -551,6 +551,8 @@ mcu_cpsw: ethernet@46000000 {
"tx4", "tx5", "tx6", "tx7",
"rx";
+ status = "disabled";
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 12/13] arm64: dts: ti: k3-j721s2-common-proc-board: Enable "mcu_cpsw" in board file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (10 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 11/13] arm64: dts: ti: k3-j721e-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:34 ` [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
12 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
In preparation for disabling "mcu_cpsw" device-tree node in the SoC file,
namely "k3-j721s2-mcu-wakeup.dtsi", enable it in the board file.
The motivation for this change is that of following the existing convention
of disabling nodes in the SoC file and only enabling the required ones in
the board file.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts | 1 +
1 file changed, 1 insertion(+)
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 e2fc1288ed07..a482b3ad5095 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
@@ -409,6 +409,7 @@ &main_sdhci1 {
};
&mcu_cpsw {
+ status = "okay";
pinctrl-names = "default";
pinctrl-0 = <&mcu_cpsw_pins_default>, <&mcu_mdio_pins_default>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:34 [PATCH 00/13] TI-K3-DTS: Cleanup CPSW DT Nodes Siddharth Vadapalli
` (11 preceding siblings ...)
2025-05-29 13:34 ` [PATCH 12/13] arm64: dts: ti: k3-j721s2-common-proc-board: Enable "mcu_cpsw" in board file Siddharth Vadapalli
@ 2025-05-29 13:34 ` Siddharth Vadapalli
2025-05-29 13:38 ` Krzysztof Kozlowski
12 siblings, 1 reply; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-29 13:34 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk, s-vadapalli
Following the existing convention of disabling nodes in the SoC file and
enabling only the required ones in the board file, disable "mcu_cpsw" node.
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---
arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 2 ++
1 file changed, 2 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 bc31266126d0..b5fd012d9564 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
@@ -552,6 +552,8 @@ mcu_cpsw: ethernet@46000000 {
"tx4", "tx5", "tx6", "tx7",
"rx";
+ status = "disabled";
+
ethernet-ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:34 ` [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file Siddharth Vadapalli
@ 2025-05-29 13:38 ` Krzysztof Kozlowski
2025-05-30 4:56 ` Siddharth Vadapalli
0 siblings, 1 reply; 20+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-29 13:38 UTC (permalink / raw)
To: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt
Cc: devicetree, linux-kernel, linux-arm-kernel, srk
On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> Following the existing convention of disabling nodes in the SoC file and
> enabling only the required ones in the board file, disable "mcu_cpsw" node.
>
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
Really all of this is fake split. One line per commit.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 13/13] arm64: dts: ti: k3-j721s2-mcu-wakeup: Disable "mcu_cpsw" in the SoC file
2025-05-29 13:38 ` Krzysztof Kozlowski
@ 2025-05-30 4:56 ` Siddharth Vadapalli
0 siblings, 0 replies; 20+ messages in thread
From: Siddharth Vadapalli @ 2025-05-30 4:56 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Siddharth Vadapalli, nm, vigneshr, kristo, robh, krzk+dt,
conor+dt, devicetree, linux-kernel, linux-arm-kernel, srk
On Thu, May 29, 2025 at 03:38:10PM +0200, Krzysztof Kozlowski wrote:
> On 29/05/2025 15:34, Siddharth Vadapalli wrote:
> > Following the existing convention of disabling nodes in the SoC file and
> > enabling only the required ones in the board file, disable "mcu_cpsw" node.
> >
> > Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
> > ---
> > arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> Really all of this is fake split. One line per commit.
I split the patches into Board and SoC specific changes. I was following
the split used when we add new device-tree nodes:
- Add the node to SoC file in the first commit
- Enable it in the Board file in the next commit
Since this is a cleanup, the above split may not be required and I
understand that. I will squash the changes in the next version of this
series.
Thank you for your feedback.
Regards,
Siddharth.
^ permalink raw reply [flat|nested] 20+ messages in thread