Re: [PATCH 2/4] arm64: dts: ti: k3-j7200-common-proc-board: Add ethernet alias

[Siddharth Vadapalli]

On 06/09/26 20:58, Hari Prasath G E wrote:
> Hello Udit,
>
> On 9/6/2026 7:34 PM, Kumar, Udit wrote:
> > On 9/6/2026 5:48 PM, Hari Prasath G E wrote:
> > > Hello Udit,
> > >
> > > On 9/6/2026 11:13 AM, Kumar, Udit wrote:
> > > > On 8/11/2026 11:43 AM, Siddharth Vadapalli wrote:
> > > > > Add the 'ethernet0' alias for the Ethernet interface corresponding to
> > > > > the
> > > > > MCU CPSW2G instance of the CPSW Ethernet Switch on the J7200 Common
> > > > > Processor Board.
> > > > >
> > > > > Signed-off-by: Siddharth Vadapalli <s-vadapalli@xxxxxx>
> > > > > ---
> > > > >    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 54dea36b153b..ac1c1ffe40b7 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
> > > > > @@ -24,6 +24,7 @@ aliases {
> > > > >            serial5 = &main_uart3;
> > > > >            mmc0 = &main_sdhci0;
> > > > >            mmc1 = &main_sdhci1;
> > > > > +        ethernet0 = &cpsw_port1;
> > > >
> > > > Hello Siddharth,
> > > > As default only 1 cpsw port is enabled for all boards in this series.
> > > > So ethernet0 will be cpsw_port1 always, In case you want to enable more
> > > > port in default config then having aliasing will help.
> > > >
> > > > otherwise, you can consider to add aliasing in overlays where you enable
> > > > more than 1 port.
> > >
> > > The first patch in the series for k3-am62a7-sk has two ports enabled by
> > > default, so there it's fine to add alias. Also, there are many other TI
> >
> > May be I am missing something, but i don't see two port are enabled for
> > am62a7
> > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/ 
> > arch/arm64/boot/dts/ti/k3-am62a7-sk.dts#n796
>
> Apologies, I mistakenly looked at the TI vendor kernel where it's enabled 
> on both ports. I think still the below reasoning is valid to take this 
> patch series in. Let's wait for Siddharth to come back with this comments.

Despite the board having a single Ethernet port enabled, there are two 
reasons for this patch/series:
1. For consistency - keeping these boards on-par with the rest of the K3 
boards (some of which also have a single Ethernet port)
2. As Francesco pointed out at:
https://lore.kernel.org/r/20260906153114.GA3576@francesco-nb/
if PCIe-Ethernet Adapters OR even USB-Ethernet Adapters are used (both of 
which are possible on J7200-Common-Processor-Board which has PCIe and USB 
connectors), we need to specify which of them is 'ethernet0'. For NFS 
(Network File System), if 'ethernet0' isn't specified in the device-tree, 
it is possible that the wrong network interface is used by Linux (despite 
eth0 passed in the Kernel Command Line for NFS) with PCIe/USB Ethernet 
Adapters connected to the board.

Regards,
Siddharth.