| Message ID | 20210107012403.1521114-1-olteanv@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Reduce coupling between DSA and Broadcom SYSTEMPORT driver | expand |
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Thu, 7 Jan 2021 03:23:59 +0200 you wrote: > From: Vladimir Oltean <vladimir.oltean@nxp.com> > > Upon a quick inspection, it seems that there is some code in the generic > DSA layer that is somehow specific to the Broadcom SYSTEMPORT driver. > The challenge there is that the hardware integration is very tight between > the switch and the DSA master interface. However this does not mean that > the drivers must also be as integrated as the hardware is. We can avoid > creating a DSA notifier just for the Broadcom SYSTEMPORT, and we can > move some Broadcom-specific queue mapping helpers outside of the common > include/net/dsa.h. > > [...] Here is the summary with links: - [v2,net-next,1/4] net: dsa: move the Broadcom tag information in a separate header file https://git.kernel.org/netdev/net-next/c/f46b9b8ee89b - [v2,net-next,2/4] net: dsa: export dsa_slave_dev_check https://git.kernel.org/netdev/net-next/c/a5e3c9ba9258 - [v2,net-next,3/4] net: systemport: use standard netdevice notifier to detect DSA presence https://git.kernel.org/netdev/net-next/c/1593cd40d785 - [v2,net-next,4/4] net: dsa: remove the DSA specific notifiers https://git.kernel.org/netdev/net-next/c/1dbb130281c4 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
From: Vladimir Oltean <vladimir.oltean@nxp.com> Upon a quick inspection, it seems that there is some code in the generic DSA layer that is somehow specific to the Broadcom SYSTEMPORT driver. The challenge there is that the hardware integration is very tight between the switch and the DSA master interface. However this does not mean that the drivers must also be as integrated as the hardware is. We can avoid creating a DSA notifier just for the Broadcom SYSTEMPORT, and we can move some Broadcom-specific queue mapping helpers outside of the common include/net/dsa.h. Vladimir Oltean (4): net: dsa: move the Broadcom tag information in a separate header file net: dsa: export dsa_slave_dev_check net: systemport: use standard netdevice notifier to detect DSA presence net: dsa: remove the DSA specific notifiers MAINTAINERS | 1 + drivers/net/ethernet/broadcom/bcmsysport.c | 82 ++++++++++------------ drivers/net/ethernet/broadcom/bcmsysport.h | 2 +- include/linux/dsa/brcm.h | 16 +++++ include/net/dsa.h | 48 +------------ net/dsa/dsa.c | 22 ------ net/dsa/dsa_priv.h | 1 - net/dsa/slave.c | 18 +---- net/dsa/tag_brcm.c | 1 + 9 files changed, 60 insertions(+), 131 deletions(-) create mode 100644 include/linux/dsa/brcm.h