arm64: dts: ti: k3-am62a7-sk: Add bootph-all property for ethernet boot

[PATCH v2] arm64: dts: ti: k3-am62a7-sk: Add bootph-all property for ethernet boot

Posted by Chintan Vankar 1 week, 2 days ago

Ethernet boot requires CPSW and PKTDMA nodes to be present starting
from R5 SPL stage. Add "bootph-all" property in CPSW MAC's eFuse node
"cpsw_mac_syscon" and "main_pktdma" to enable these nodes during SPL
stage along with later boot stages so that CPSW port will get static
MAC address and DMA transfers are functional during ethernet boot.

Signed-off-by: Chintan Vankar <c-vankar@ti.com>
---

Hello All, This patch is based on commit 'cc2b5f627e8c' of next-20260714
branch of Linux repo.

Link to v1:
https://lore.kernel.org/all/20260625113223.1711052-1-c-vankar@ti.com/

Changes from v1 to v2:
-> Added "bootph-all" property to "main_pktdma" node and updated commit
   message accordingly.
-> Collected "Ack-by" tag from Andrew, since I have modified a patch by
   adding "main_pktdma" node too, I am not keeping the "Ack-by" tag.

 arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
index 821a9705bb7d..59cf49a14f61 100644
--- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
@@ -230,6 +230,10 @@ AM62AX_MCU_IOPAD(0x0030, PIN_OUTPUT, 0) /* (C8) WKUP_UART0_RTSn */
 	};
 };
 
+&cpsw_mac_syscon {
+	bootph-all;
+};
+
 /* WKUP UART0 is used for DM firmware logs */
 &wkup_uart0 {
 	pinctrl-names = "default";
@@ -241,6 +245,10 @@ &wkup_uart0 {
 	status = "reserved";
 };
 
+&main_pktdma {
+	bootph-all;
+};
+
 &main_pmx0 {
 	main_dss0_pins_default: main-dss0-default-pins {
 		pinctrl-single,pins = <
-- 
2.34.1

Re: [PATCH v2] arm64: dts: ti: k3-am62a7-sk: Add bootph-all property for ethernet boot

Posted by Andrew Davis 1 week, 2 days ago

On 7/16/26 1:44 AM, Chintan Vankar wrote:
> Ethernet boot requires CPSW and PKTDMA nodes to be present starting
> from R5 SPL stage. Add "bootph-all" property in CPSW MAC's eFuse node
> "cpsw_mac_syscon" and "main_pktdma" to enable these nodes during SPL
> stage along with later boot stages so that CPSW port will get static
> MAC address and DMA transfers are functional during ethernet boot.
> 
> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
> ---
> 
> Hello All, This patch is based on commit 'cc2b5f627e8c' of next-20260714
> branch of Linux repo.
> 
> Link to v1:
> https://lore.kernel.org/all/20260625113223.1711052-1-c-vankar@ti.com/
> 
> Changes from v1 to v2:
> -> Added "bootph-all" property to "main_pktdma" node and updated commit
>     message accordingly.
> -> Collected "Ack-by" tag from Andrew, since I have modified a patch by
>     adding "main_pktdma" node too, I am not keeping the "Ack-by" tag.
> 

Would these tags be needed by other AM62A based boards? If the network
is enabled, I'd assume any board can network boot. Anyway, adding
main_pktdma as it is also used when doing network boot seems reasonable
here, so you can keep my ACK,

Acked-by: Andrew Davis <afd@ti.com>

>   arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> index 821a9705bb7d..59cf49a14f61 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
> @@ -230,6 +230,10 @@ AM62AX_MCU_IOPAD(0x0030, PIN_OUTPUT, 0) /* (C8) WKUP_UART0_RTSn */
>   	};
>   };
>   
> +&cpsw_mac_syscon {
> +	bootph-all;
> +};
> +
>   /* WKUP UART0 is used for DM firmware logs */
>   &wkup_uart0 {
>   	pinctrl-names = "default";
> @@ -241,6 +245,10 @@ &wkup_uart0 {
>   	status = "reserved";
>   };
>   
> +&main_pktdma {
> +	bootph-all;
> +};
> +
>   &main_pmx0 {
>   	main_dss0_pins_default: main-dss0-default-pins {
>   		pinctrl-single,pins = <