On 06/06/2017 05:03 PM, Florian Fainelli wrote: > Hi all, > > This patch series prepares the ground for adding mutliple CPU port support to > DSA, and starts by removing redundant pieces of information such as > master_netdev which is cpu_dp->ethernet. Finally drivers are moved away from > directly accessing ds->dst->cpu_dp and use appropriate helper functions. > > Note that if you have Device Tree blobs/platform configurations that are > currently listing multiple CPU ports, the proposed behavior in > dsa_ds_get_cpu_dp() will be to return the last bit set in ds->cpu_port_mask. > > Future plans include: > - making dst->cpu_dp a flexible data structure (array, list, you name it) > - having the ability for drivers to return a default/preferred CPU port (if > necessary) > > Changes in v2: > > - added Reviewed-by tags > - assign port->cpu_dp earlier before ops->setup() has run
There are some hunks in patch 5 that actually belong in patch 3, I will post a v3 after getting some more feedback. > > Florian Fainelli (5): > net: dsa: Remove master_netdev and use dst->cpu_dp->netdev > net: dsa: Relocate master ethtool operations > net: dsa: Associate slave network device with CPU port > net: dsa: Introduce dsa_dst_get_cpu_dp() > net: dsa: Stop accessing ds->dst->cpu_dp in drivers > > drivers/net/dsa/b53/b53_common.c | 4 +-- > drivers/net/dsa/bcm_sf2.c | 10 +++++--- > drivers/net/dsa/mt7530.c | 6 +++-- > drivers/net/dsa/mv88e6060.c | 2 +- > drivers/net/dsa/qca8k.c | 2 +- > include/net/dsa.h | 29 +++++++++------------- > net/dsa/dsa.c | 19 ++++---------- > net/dsa/dsa2.c | 27 ++++++++++++-------- > net/dsa/dsa_priv.h | 10 ++++++++ > net/dsa/legacy.c | 23 ++++++++++------- > net/dsa/slave.c | 53 > ++++++++++++++++++++-------------------- > net/dsa/tag_brcm.c | 5 ++-- > net/dsa/tag_ksz.c | 5 ++-- > net/dsa/tag_qca.c | 3 ++- > net/dsa/tag_trailer.c | 5 ++-- > 15 files changed, 107 insertions(+), 96 deletions(-) > -- Florian
