@@ -315,6 +315,16 @@ static int dsa_port_setup(struct dsa_port *dp)
break;
}
+ switch (dp->type) {
+ case DSA_PORT_TYPE_CPU:
+ case DSA_PORT_TYPE_DSA:
+ if (!err)
+ err = dsa_port_enable(dp, NULL);
+ break;
+ default:
+ break;
+ }
+
if (err)
devlink_port_unregister(&dp->devlink_port);
@@ -323,8 +333,17 @@ static int dsa_port_setup(struct dsa_port *dp)
static void dsa_port_teardown(struct dsa_port *dp)
{
- if (dp->type != DSA_PORT_TYPE_UNUSED)
+ switch (dp->type) {
+ case DSA_PORT_TYPE_UNUSED:
+ break;
+ case DSA_PORT_TYPE_CPU:
+ case DSA_PORT_TYPE_DSA:
+ dsa_port_disable(dp);
+ /* fall-through */
+ case DSA_PORT_TYPE_USER:
devlink_port_unregister(&dp->devlink_port);
+ break;
+ }
switch (dp->type) {
case DSA_PORT_TYPE_UNUSED:
@@ -75,7 +75,7 @@ int dsa_port_enable(struct dsa_port *dp, struct phy_device *phy)
return err;
}
- if (!dp->bridge_dev)
+ if (dp->type == DSA_PORT_TYPE_USER && !dp->bridge_dev)
dsa_port_set_state_now(dp, BR_STATE_FORWARDING);
return 0;
@@ -86,7 +86,7 @@ void dsa_port_disable(struct dsa_port *dp)
struct dsa_switch *ds = dp->ds;
int port = dp->index;
- if (!dp->bridge_dev)
+ if (dp->type == DSA_PORT_TYPE_USER && !dp->bridge_dev)
dsa_port_set_state_now(dp, BR_STATE_DISABLED);
if (ds->ops->port_disable)
Call dsa_port_enable for CPU/DSA ports in dsa_port_setup, and dsa_port_disable for CPU/DSA ports in dsa_port_teardown. This requires changing all DSA drivers, since they expect the port_enable/port_disable methods to be called only for user ports. Signed-off-by: Marek BehĂșn <marek.behun@nic.cz> Cc: Andrew Lunn <andrew@lunn.ch> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: Vladimir Oltean <olteanv@gmail.com> Cc: Vivien Didelot <vivien.didelot@gmail.com> --- net/dsa/dsa2.c | 21 ++++++++++++++++++++- net/dsa/port.c | 4 ++-- 2 files changed, 22 insertions(+), 3 deletions(-)