Messages in this thread | | | Date | Wed, 23 Jun 2021 15:17:15 +0200 | From | Andrew Lunn <> | Subject | Re: [RFC 1/3] ARM: dts: imx28: Add description for L2 switch on XEA board |
| |
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
| |