Messages in this thread | | | Date | Fri, 25 Jun 2021 16:40:21 +0200 | From | Andrew Lunn <> | Subject | Re: [RFC 2/3] net: Provide switchdev driver for NXP's More Than IP L2 switch |
| |
> I do believe that I can just extend the L2 switch driver (fec_mtip.c > file to be precise) to provide full blown L2 switch functionality > without touching the legacy FEC more than in this patch set. > > Would you consider applying this patch series then?
What is most important is the ABI. If something is merged now, we need to ensure it does not block later refactoring to a clean new driver. The DT binding is considered ABI. So the DT binding needs to be like a traditional switchdev driver. Florian already pointed out, you can use a binding very similar to DSA. ti,cpsw-switch.yaml is another good example.
So before considering merging your changes, i would like to see a usable binding.
I also don't remember seeing support for STP. Without that, your network has broadcast storm problems when there are loops. So i would like to see the code needed to put ports into blocking, listening, learning, and forwarding states.
Andrew
| |