Re: [RFC 1/3] ARM: dts: imx28: Add description for L2 switch on XEA board

From: Andrew Lunn
Date: Wed Jun 23 2021 - 09:17:33 EST


On Tue, Jun 22, 2021 at 10:51:34PM +0200, Lukasz Majewski wrote:
> Hi Andrew,
>
> > On Tue, Jun 22, 2021 at 04:41:09PM +0200, Lukasz Majewski wrote:
> > > The 'eth_switch' node is now extendfed to enable support for L2
> > > switch.
> > >
> > > Moreover, the mac[01] nodes are defined as well and linked to the
> > > former with 'phy-handle' property.
> >
> > A phy-handle points to a phy, not a MAC! Don't abuse a well known DT
> > property like this.
>
> Ach.... You are right. I will change it.
>
> Probably 'ethernet' property or 'link' will fit better?

You should first work on the overall architecture. I suspect you will
end up with something more like the DSA binding, and not have the FEC
nodes at all. Maybe the MDIO busses will appear under the switch?

Please don't put minimal changes to the FEC driver has your first
goal. We want an architecture which is similar to other switchdev
drivers. Maybe look at drivers/net/ethernet/ti/cpsw_new.c. The cpsw
driver has an interesting past, it did things the wrong way for a long
time, but the new switchdev driver has an architecture similar to what
the FEC driver could be like.

Andrew