Message ID | 20211117210451.26415-11-ansuelsmth@gmail.com (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | Multiple cleanup and feature for qca8k | expand |
On Wed, Nov 17, 2021 at 10:04:42PM +0100, Ansuel Smith wrote: > The switch doesn't support fast aging but it does support the flush of > the ARL table for a specific port. Add this function to simulate > fast aging and proprely support stp state set. ~~~~~~~~ ~~~ properly verb? noun? what are you saying here? What difference do you see between fast ageing and ARL table flushing for a specific port? > > Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com> > --- > drivers/net/dsa/qca8k.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/net/dsa/qca8k.c b/drivers/net/dsa/qca8k.c > index cf4f69b36b47..d73886b36e6a 100644 > --- a/drivers/net/dsa/qca8k.c > +++ b/drivers/net/dsa/qca8k.c > @@ -1823,6 +1823,16 @@ qca8k_port_bridge_leave(struct dsa_switch *ds, int port, struct net_device *br) > QCA8K_PORT_LOOKUP_MEMBER, BIT(cpu_port)); > } > > +static void > +qca8k_port_fast_age(struct dsa_switch *ds, int port) > +{ > + struct qca8k_priv *priv = ds->priv; > + > + mutex_lock(&priv->reg_mutex); > + qca8k_fdb_access(priv, QCA8K_FDB_FLUSH_PORT, port); > + mutex_unlock(&priv->reg_mutex); > +} > + > static int > qca8k_port_enable(struct dsa_switch *ds, int port, > struct phy_device *phy) > @@ -2031,6 +2041,7 @@ static const struct dsa_switch_ops qca8k_switch_ops = { > .port_stp_state_set = qca8k_port_stp_state_set, > .port_bridge_join = qca8k_port_bridge_join, > .port_bridge_leave = qca8k_port_bridge_leave, > + .port_fast_age = qca8k_port_fast_age, > .port_fdb_add = qca8k_port_fdb_add, > .port_fdb_del = qca8k_port_fdb_del, > .port_fdb_dump = qca8k_port_fdb_dump, > -- > 2.32.0 >
diff --git a/drivers/net/dsa/qca8k.c b/drivers/net/dsa/qca8k.c index cf4f69b36b47..d73886b36e6a 100644 --- a/drivers/net/dsa/qca8k.c +++ b/drivers/net/dsa/qca8k.c @@ -1823,6 +1823,16 @@ qca8k_port_bridge_leave(struct dsa_switch *ds, int port, struct net_device *br) QCA8K_PORT_LOOKUP_MEMBER, BIT(cpu_port)); } +static void +qca8k_port_fast_age(struct dsa_switch *ds, int port) +{ + struct qca8k_priv *priv = ds->priv; + + mutex_lock(&priv->reg_mutex); + qca8k_fdb_access(priv, QCA8K_FDB_FLUSH_PORT, port); + mutex_unlock(&priv->reg_mutex); +} + static int qca8k_port_enable(struct dsa_switch *ds, int port, struct phy_device *phy) @@ -2031,6 +2041,7 @@ static const struct dsa_switch_ops qca8k_switch_ops = { .port_stp_state_set = qca8k_port_stp_state_set, .port_bridge_join = qca8k_port_bridge_join, .port_bridge_leave = qca8k_port_bridge_leave, + .port_fast_age = qca8k_port_fast_age, .port_fdb_add = qca8k_port_fdb_add, .port_fdb_del = qca8k_port_fdb_del, .port_fdb_dump = qca8k_port_fdb_dump,
The switch doesn't support fast aging but it does support the flush of the ARL table for a specific port. Add this function to simulate fast aging and proprely support stp state set. Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com> --- drivers/net/dsa/qca8k.c | 11 +++++++++++ 1 file changed, 11 insertions(+)