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

[Hari Prasath G E]

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.

Regards,
Hari

> > K3 platforms which have only one cpsw port enabled but still define
> > aliases. So I guess to keep it consistent, we can decide to accept this.
> > If future overlays are added, this will just mean reserving eth0 for
> > cpsw port1 which is also fine. I am fine to move with accepting this
> > series.
> >
> > Acked-by: Hari Prasath Gujulan Elango <gehariprasath@xxxxxx>
>
>
> > Regards,
> > Hari
> >
> > > >        };
> > > >          chosen {