Hi Andrew, On Tue, 23 Apr 2019 02:22:43 +0200, Andrew Lunn <and...@lunn.ch> wrote: > istorically, DSA tag drivers have been compiled into the kernel as > part of the DSA core. With the growing number of tag drivers, it makes > sense to allow this driver code to be compiled as a module, and loaded > on demand. > > v2 > -- > Move name to end of structure, keeping the hot entries at the beginning. > More tag protocol to end of structure to keep hot members at the beginning. > Fix indent of #endif > Rewrite to move list pointer into a new structure > void functions, since there cannot be errors > Fix fall-through comment > Reorder patch for unused symbols to before tag drivers can be modules > tab/space cleanup > Help text wording > NET_DSA_TAG_BRCM_COMMON and NET_DSA_TAG_KZS_COMMON hidden > > > Andrew Lunn (14): > dsa: Add SPDX header to tag drivers. > dsa: Move tagger name into its ops structure > dsa: Add MODULE_ALIAS to taggers in preparation to become modules > dsa: Add MODULE_LICENSE to tag drivers > dsa: Add TAG protocol to tag ops > dsa: Add boilerplate helper to register DSA tag driver modules > dsa: Keep link list of tag drivers > dsa: Register the none tagger ops > dsa: Rename dsa_resolve_tag_protocol() to _get ready for locking > dsa: Add stub tag driver put method > dsa: Make use of the list of tag drivers > dsa: Cleanup unneeded table and make tag structures static > dsa: tag_brcm: Avoid unused symbols > dsa: Allow tag drivers to be built as modules > > include/net/dsa.h | 110 ++++++++++++++++++++++++---- > net/dsa/Kconfig | 83 ++++++++++++++++----- > net/dsa/Makefile | 19 +++-- > net/dsa/dsa.c | 165 ++++++++++++++++++++++-------------------- > net/dsa/dsa2.c | 4 +- > net/dsa/dsa_priv.h | 34 +-------- > net/dsa/legacy.c | 4 +- > net/dsa/tag_brcm.c | 41 ++++++++--- > net/dsa/tag_dsa.c | 15 ++-- > net/dsa/tag_edsa.c | 15 ++-- > net/dsa/tag_gswip.c | 9 ++- > net/dsa/tag_ksz.c | 29 ++++++-- > net/dsa/tag_lan9303.c | 20 +++-- > net/dsa/tag_mtk.c | 18 ++--- > net/dsa/tag_qca.c | 19 +++-- > net/dsa/tag_trailer.c | 15 ++-- > 16 files changed, 380 insertions(+), 220 deletions(-)
Nice cleanup at the same time. Tested-by: Vivien Didelot <vivien.dide...@gmail.com>
