From: Vivien Didelot <vivien.didelot@gmail.com>
To: "David S. Miller" <davem@davemloft.net>
Cc: linux-kernel@vger.kernel.org,
Vivien Didelot <vivien.didelot@gmail.com>,
Florian Fainelli <f.fainelli@gmail.com>,
Andrew Lunn <andrew@lunn.ch>,
netdev@vger.kernel.org
Subject: [PATCH net-next v2 00/16] net: dsa: turn arrays of ports into a list
Date: Mon, 21 Oct 2019 16:51:14 -0400 [thread overview]
Message-ID: <20191021205130.304149-1-vivien.didelot@gmail.com> (raw)
The dsa_switch structure represents the physical switch device itself,
and is allocated by the driver. The dsa_switch_tree and dsa_port structures
represent the logical switch fabric (eventually composed of multiple switch
devices) and its ports, and are allocated by the DSA core.
This branch lists the logical ports directly in the fabric which simplifies
the iteration over all ports when assigning the default CPU port or configuring
the D in DSA in drivers like mv88e6xxx.
This also removes the unique dst->cpu_dp pointer and is a first step towards
supporting multiple CPU ports and dropping the DSA_MAX_PORTS limitation.
Because the dsa_port structures are not tight to the dsa_switch structure
anymore, we do not need to provide an helper for the drivers to allocate a
switch structure. Like in many other subsystems, drivers can now embed their
dsa_switch structure as they wish into their private structure. This will
be particularly interesting for the Broadcom drivers which were currently
limited by the dynamically allocated array of DSA ports.
The series implements the list of dsa_port structures, makes use of it,
then drops dst->cpu_dp and the dsa_switch_alloc helper.
Changes in v2:
- use list_add_tail instead of list_add to respect ports order
- use a single return statement for dsa_to_port
- remove pr_info messages
- put comments under appropriate branches
- add Git tags from reviewers
Vivien Didelot (16):
net: dsa: use dsa_to_port helper everywhere
net: dsa: add ports list in the switch fabric
net: dsa: use ports list in dsa_to_port
net: dsa: use ports list to find slave
net: dsa: use ports list to setup switches
net: dsa: use ports list for routing table setup
net: dsa: use ports list to find a port by node
net: dsa: use ports list to setup multiple master devices
net: dsa: use ports list to find first CPU port
net: dsa: use ports list to setup default CPU port
net: dsa: mv88e6xxx: silently skip PVT ops
net: dsa: mv88e6xxx: use ports list to map port VLAN
net: dsa: mv88e6xxx: use ports list to map bridge
net: dsa: sja1105: register switch before assigning port private data
net: dsa: allocate ports on touch
net: dsa: remove dsa_switch_alloc helper
drivers/net/dsa/b53/b53_common.c | 11 +-
drivers/net/dsa/bcm_sf2.c | 8 +-
drivers/net/dsa/bcm_sf2_cfp.c | 6 +-
drivers/net/dsa/dsa_loop.c | 5 +-
drivers/net/dsa/lan9303-core.c | 4 +-
drivers/net/dsa/lantiq_gswip.c | 4 +-
drivers/net/dsa/microchip/ksz_common.c | 5 +-
drivers/net/dsa/mt7530.c | 17 +-
drivers/net/dsa/mv88e6060.c | 4 +-
drivers/net/dsa/mv88e6xxx/chip.c | 90 ++++----
drivers/net/dsa/qca8k.c | 7 +-
drivers/net/dsa/realtek-smi-core.c | 5 +-
drivers/net/dsa/sja1105/sja1105_main.c | 37 ++--
drivers/net/dsa/vitesse-vsc73xx-core.c | 5 +-
include/net/dsa.h | 26 ++-
net/dsa/dsa.c | 8 +-
net/dsa/dsa2.c | 274 +++++++++++++------------
net/dsa/dsa_priv.h | 23 +--
net/dsa/switch.c | 4 +-
net/dsa/tag_8021q.c | 6 +-
20 files changed, 292 insertions(+), 257 deletions(-)
--
2.23.0
next reply other threads:[~2019-10-21 20:51 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-21 20:51 Vivien Didelot [this message]
2019-10-21 20:51 ` [PATCH net-next v2 01/16] net: dsa: use dsa_to_port helper everywhere Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 02/16] net: dsa: add ports list in the switch fabric Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 03/16] net: dsa: use ports list in dsa_to_port Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 04/16] net: dsa: use ports list to find slave Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 05/16] net: dsa: use ports list to setup switches Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 06/16] net: dsa: use ports list for routing table setup Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 07/16] net: dsa: use ports list to find a port by node Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 08/16] net: dsa: use ports list to setup multiple master devices Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 09/16] net: dsa: use ports list to find first CPU port Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 10/16] net: dsa: use ports list to setup default " Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 11/16] net: dsa: mv88e6xxx: silently skip PVT ops Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 12/16] net: dsa: mv88e6xxx: use ports list to map port VLAN Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 13/16] net: dsa: mv88e6xxx: use ports list to map bridge Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 14/16] net: dsa: sja1105: register switch before assigning port private data Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 15/16] net: dsa: allocate ports on touch Vivien Didelot
2019-10-21 20:51 ` [PATCH net-next v2 16/16] net: dsa: remove dsa_switch_alloc helper Vivien Didelot
2019-10-21 22:34 ` [PATCH net-next v2 00/16] net: dsa: turn arrays of ports into a list Florian Fainelli
2019-10-22 11:43 ` Michal Vokáč
2019-10-22 19:41 ` Jakub Kicinski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191021205130.304149-1-vivien.didelot@gmail.com \
--to=vivien.didelot@gmail.com \
--cc=andrew@lunn.ch \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®