| Message ID | 20210802131037.32326-7-o.rempel@pengutronix.de (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | ar9331: mainline some parts of switch functionality | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/subject_prefix | success | Link |
| netdev/cc_maintainers | warning | 1 maintainers not CCed: linux@rempel-privat.de |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 0 this patch: 0 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 271 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 0 this patch: 0 |
| netdev/header_inline | success | Link |
On Mon, Aug 02, 2021 at 03:10:37PM +0200, Oleksij Rempel wrote: > This switch provides simple VLAN resolution database for 16 entries. > With this database we can cover typical functionalities as port based > VLANs, untagged and tagged egress. Port based ingress filtering. > > The VLAN database is working on top of forwarding database. So, > potentially, we can have multiple VLANs on top of multiple bridges. > Hawing one VLAN on top of multiple bridges will fail on different > levels, most probably DSA framework should warn if some one won't to make > something likes this. Please define the levels in which it will fail for ar9331. I opened the AR9331 manual and I see that the only forwarding isolation mechanism is the PORT_VID_MEMBER, the "port base VLAN", which defines which other ports can each port forward to. Otherwise, it seems that when two ports are in 802.1Q mode (VLAN-aware) and in the same VLAN, they can forward packets in that VLAN regardless of PORT_VID_MEMBER. If correct, it means you only support a single VLAN-aware bridge, I recommend you copy what sja1105_prechangeupper does to block a second one. The manual also says "The AR9331 only supports shared VLAN learning (SVL)." So there isn't in fact all that much that the DSA core can save the day. Even with a single bridge, the standalone ports will still attempt to look up the FDB, and even though packets from standalone ports should always reach the CPU port, that might not happen if that MAC DA is in the bridge FDB of a bridged port. Example: you have a software LAG interface on top of your standalone ports, and this is in a software bridge with other ar9331 ports that directly offload the bridge: https://patchwork.kernel.org/project/netdevbpf/patch/20210731191023.1329446-3-dqfext@gmail.com/ DSA cannot (easily) help you in that case either, but I agree that tracking all possible constellations of unoffloaded LAG interfaces on top of ar9331 switches that also have ports in a hardware bridge is tricky too. I will give this some thought. > > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> > --- > drivers/net/dsa/qca/ar9331.c | 235 ++++++++++++++++++++++++++++++++++- > 1 file changed, 233 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/dsa/qca/ar9331.c b/drivers/net/dsa/qca/ar9331.c > index a0324fed2136..0865ffbc2c74 100644 > --- a/drivers/net/dsa/qca/ar9331.c > +++ b/drivers/net/dsa/qca/ar9331.c > @@ -67,6 +67,27 @@ > #define AR9331_SW_REG_GLOBAL_CTRL 0x30 > #define AR9331_SW_GLOBAL_CTRL_MFS_M GENMASK(13, 0) > > +#define AR9331_SW_NUM_VLAN_RECORDS 16 > + > +#define AR9331_SW_REG_VLAN_TABLE_FUNCTION0 0x40 > +#define AR9331_SW_VT0_PRI_EN BIT(31) > +#define AR9331_SW_VT0_PRI GENMASK(30, 28) > +#define AR9331_SW_VT0_VID GENMASK(27, 16) > +#define AR9331_SW_VT0_PORT_NUM GENMASK(11, 8) > +#define AR9331_SW_VT0_FULL_VIO BIT(4) > +#define AR9331_SW_VT0_BUSY BIT(3) > +#define AR9331_SW_VT0_FUNC GENMASK(2, 0) > +#define AR9331_SW_VT0_FUNC_NOP 0 > +#define AR9331_SW_VT0_FUNC_FLUSH_ALL 1 > +#define AR9331_SW_VT0_FUNC_LOAD_ENTRY 2 > +#define AR9331_SW_VT0_FUNC_PURGE_ENTRY 3 > +#define AR9331_SW_VT0_FUNC_DEL_PORT 4 > +#define AR9331_SW_VT0_FUNC_GET_NEXT 5 > + > +#define AR9331_SW_REG_VLAN_TABLE_FUNCTION1 0x44 > +#define AR9331_SW_VT1_VALID BIT(11) > +#define AR9331_SW_VT1_VID_MEM GENMASK(9, 0) > + > /* Size of the address resolution table (ARL) */ > #define AR9331_SW_NUM_ARL_RECORDS 1024 > > @@ -309,6 +330,11 @@ struct ar9331_sw_port { > struct spinlock stats_lock; > }; > > +struct ar9331_sw_vlan_db { > + u16 vid; > + u8 port_mask; > +}; > + > struct ar9331_sw_fdb { > u8 port_mask; > u8 aging; > @@ -327,6 +353,7 @@ struct ar9331_sw_priv { > struct regmap *regmap; > struct reset_control *sw_reset; > struct ar9331_sw_port port[AR9331_SW_PORTS]; > + struct ar9331_sw_vlan_db vdb[AR9331_SW_NUM_VLAN_RECORDS]; > }; > > static struct ar9331_sw_priv *ar9331_sw_port_to_priv(struct ar9331_sw_port *port) > @@ -557,8 +584,6 @@ static int ar9331_sw_setup(struct dsa_switch *ds) > goto error; > } > > - ds->configure_vlan_while_not_filtering = false; > - > return 0; > error: > dev_err_ratelimited(priv->dev, "%s: %i\n", __func__, ret); > @@ -1144,6 +1169,209 @@ static void ar9331_sw_port_bridge_leave(struct dsa_switch *ds, int port, > ar9331_sw_port_bridge_mod(ds, port, br, false); > } > > +static int ar9331_port_vlan_filtering(struct dsa_switch *ds, int port, > + bool vlan_filtering, > + struct netlink_ext_ack *extack) > +{ > + struct ar9331_sw_priv *priv = (struct ar9331_sw_priv *)ds->priv; > + struct regmap *regmap = priv->regmap; > + u32 mode; > + int ret; > + > + if (vlan_filtering) > + mode = AR9331_SW_8021Q_MODE_SECURE; > + else > + mode = AR9331_SW_8021Q_MODE_NONE; > + > + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), > + AR9331_SW_PORT_VLAN_8021Q_MODE, > + FIELD_PREP(AR9331_SW_PORT_VLAN_8021Q_MODE, > + mode)); > + if (ret) > + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); > + > + return ret; > +} > + > +static int ar9331_sw_vt_wait(struct ar9331_sw_priv *priv, u32 *f0) > +{ > + struct regmap *regmap = priv->regmap; > + > + return regmap_read_poll_timeout(regmap, > + AR9331_SW_REG_VLAN_TABLE_FUNCTION0, > + *f0, !(*f0 & AR9331_SW_VT0_BUSY), > + 100, 2000); > +} > + > +static int ar9331_sw_port_vt_rmw(struct ar9331_sw_priv *priv, u16 vid, > + u8 port_mask_set, u8 port_mask_clr) > +{ > + struct regmap *regmap = priv->regmap; > + u32 f0, f1, port_mask = 0, port_mask_new, func; Reverse Christmas tree ordering? > + struct ar9331_sw_vlan_db *vdb = NULL; > + int ret, i; > + > + if (!vid) > + return 0; Explanation? > + > + ret = ar9331_sw_vt_wait(priv, &f0); > + if (ret) > + return ret; > + > + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, 0); > + if (ret) > + goto error; > + > + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, 0); > + if (ret) > + goto error; > + > + for (i = 0; i < ARRAY_SIZE(priv->vdb); i++) { > + if (priv->vdb[i].vid == vid) { > + vdb = &priv->vdb[i]; > + break; > + } > + } > + > + ret = regmap_read(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, &f1); > + if (ret) > + return ret; > + > + if (vdb) > + port_mask = vdb->port_mask; > + > + port_mask_new = port_mask & ~port_mask_clr; > + port_mask_new |= port_mask_set; > + > + if (port_mask_new && port_mask_new == port_mask) > + return 0; > + > + if (port_mask_new) { > + func = AR9331_SW_VT0_FUNC_LOAD_ENTRY; > + } else { > + func = AR9331_SW_VT0_FUNC_PURGE_ENTRY; > + port_mask_new = port_mask; > + } > + > + if (vdb) { > + vdb->port_mask = port_mask_new; > + > + if (!port_mask_new) > + vdb->vid = 0; > + } else { > + for (i = 0; i < ARRAY_SIZE(priv->vdb); i++) { > + if (!priv->vdb[i].vid) { > + vdb = &priv->vdb[i]; > + break; > + } > + } > + > + if (!vdb) > + return -ENOMEM; > + > + vdb->vid = vid; > + vdb->port_mask = port_mask_new; > + } > + > + f0 = FIELD_PREP(AR9331_SW_VT0_VID, vid) | > + FIELD_PREP(AR9331_SW_VT0_FUNC, func) | > + AR9331_SW_VT0_BUSY; > + f1 = FIELD_PREP(AR9331_SW_VT1_VID_MEM, port_mask_new) | > + AR9331_SW_VT1_VALID; > + > + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, f1); > + if (ret) > + return ret; > + > + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, f0); > + if (ret) > + return ret; > + > + ret = ar9331_sw_vt_wait(priv, &f0); > + if (ret) > + return ret; > + > + if (f0 & AR9331_SW_VT0_FULL_VIO) { > + /* cleanup error status */ > + regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, 0); > + return -ENOMEM; s/ENOMEM/ENOSPC/ since ENOMEM is for kernel memory allocation AFAIK Also, what about "ret = -ENOSPC; goto error;" to get the print for this too and be consistent? > + } > + > + return 0; > + > +error: > + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); > + > + return ret; > +} > + > +static int ar9331_port_vlan_set_pvid(struct ar9331_sw_priv *priv, int port, > + u16 pvid) > +{ > + struct regmap *regmap = priv->regmap; > + int ret; > + u32 mask, val; > + > + mask = AR9331_SW_PORT_VLAN_8021Q_MODE | > + AR9331_SW_PORT_VLAN_FORCE_DEFALUT_VID_EN | > + AR9331_SW_PORT_VLAN_FORCE_PORT_VLAN_EN; > + val = AR9331_SW_PORT_VLAN_FORCE_DEFALUT_VID_EN | > + AR9331_SW_PORT_VLAN_FORCE_PORT_VLAN_EN | > + FIELD_PREP(AR9331_SW_PORT_VLAN_8021Q_MODE, > + AR9331_SW_8021Q_MODE_FALLBACK); > + > + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), > + mask, val); > + if (ret) > + return ret; > + > + return regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), > + AR9331_SW_PORT_VLAN_PORT_VID, > + FIELD_PREP(AR9331_SW_PORT_VLAN_PORT_VID, > + pvid)); > +} > + > +static int ar9331_port_vlan_add(struct dsa_switch *ds, int port, > + const struct switchdev_obj_port_vlan *vlan, > + struct netlink_ext_ack *extack) > +{ > + struct ar9331_sw_priv *priv = ds->priv; > + struct regmap *regmap = priv->regmap; > + int ret, mode; > + > + ret = ar9331_sw_port_vt_rmw(priv, vlan->vid, BIT(port), 0); > + if (ret) > + goto error; > + > + if (vlan->flags & BRIDGE_VLAN_INFO_PVID) > + ret = ar9331_port_vlan_set_pvid(priv, port, vlan->vid); > + > + if (ret) > + goto error; > + > + if (vlan->flags & BRIDGE_VLAN_INFO_UNTAGGED) > + mode = AR9331_SW_PORT_CTRL_EG_VLAN_MODE_STRIP; > + else > + mode = AR9331_SW_PORT_CTRL_EG_VLAN_MODE_ADD; > + > + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_CTRL(port), > + AR9331_SW_PORT_CTRL_EG_VLAN_MODE, mode); > + if (ret) > + goto error; > + > + return 0; > +error: > + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); > + > + return ret; > +} > + > +static int ar9331_port_vlan_del(struct dsa_switch *ds, int port, > + const struct switchdev_obj_port_vlan *vlan) > +{ > + return ar9331_sw_port_vt_rmw(ds->priv, vlan->vid, 0, BIT(port)); > +} > + > static const struct dsa_switch_ops ar9331_sw_ops = { > .get_tag_protocol = ar9331_sw_get_tag_protocol, > .setup = ar9331_sw_setup, > @@ -1162,6 +1390,9 @@ static const struct dsa_switch_ops ar9331_sw_ops = { > .set_ageing_time = ar9331_sw_set_ageing_time, > .port_bridge_join = ar9331_sw_port_bridge_join, > .port_bridge_leave = ar9331_sw_port_bridge_leave, > + .port_vlan_filtering = ar9331_port_vlan_filtering, > + .port_vlan_add = ar9331_port_vlan_add, > + .port_vlan_del = ar9331_port_vlan_del, > }; > > static irqreturn_t ar9331_sw_irq(int irq, void *data) > -- > 2.30.2 >
diff --git a/drivers/net/dsa/qca/ar9331.c b/drivers/net/dsa/qca/ar9331.c index a0324fed2136..0865ffbc2c74 100644 --- a/drivers/net/dsa/qca/ar9331.c +++ b/drivers/net/dsa/qca/ar9331.c @@ -67,6 +67,27 @@ #define AR9331_SW_REG_GLOBAL_CTRL 0x30 #define AR9331_SW_GLOBAL_CTRL_MFS_M GENMASK(13, 0) +#define AR9331_SW_NUM_VLAN_RECORDS 16 + +#define AR9331_SW_REG_VLAN_TABLE_FUNCTION0 0x40 +#define AR9331_SW_VT0_PRI_EN BIT(31) +#define AR9331_SW_VT0_PRI GENMASK(30, 28) +#define AR9331_SW_VT0_VID GENMASK(27, 16) +#define AR9331_SW_VT0_PORT_NUM GENMASK(11, 8) +#define AR9331_SW_VT0_FULL_VIO BIT(4) +#define AR9331_SW_VT0_BUSY BIT(3) +#define AR9331_SW_VT0_FUNC GENMASK(2, 0) +#define AR9331_SW_VT0_FUNC_NOP 0 +#define AR9331_SW_VT0_FUNC_FLUSH_ALL 1 +#define AR9331_SW_VT0_FUNC_LOAD_ENTRY 2 +#define AR9331_SW_VT0_FUNC_PURGE_ENTRY 3 +#define AR9331_SW_VT0_FUNC_DEL_PORT 4 +#define AR9331_SW_VT0_FUNC_GET_NEXT 5 + +#define AR9331_SW_REG_VLAN_TABLE_FUNCTION1 0x44 +#define AR9331_SW_VT1_VALID BIT(11) +#define AR9331_SW_VT1_VID_MEM GENMASK(9, 0) + /* Size of the address resolution table (ARL) */ #define AR9331_SW_NUM_ARL_RECORDS 1024 @@ -309,6 +330,11 @@ struct ar9331_sw_port { struct spinlock stats_lock; }; +struct ar9331_sw_vlan_db { + u16 vid; + u8 port_mask; +}; + struct ar9331_sw_fdb { u8 port_mask; u8 aging; @@ -327,6 +353,7 @@ struct ar9331_sw_priv { struct regmap *regmap; struct reset_control *sw_reset; struct ar9331_sw_port port[AR9331_SW_PORTS]; + struct ar9331_sw_vlan_db vdb[AR9331_SW_NUM_VLAN_RECORDS]; }; static struct ar9331_sw_priv *ar9331_sw_port_to_priv(struct ar9331_sw_port *port) @@ -557,8 +584,6 @@ static int ar9331_sw_setup(struct dsa_switch *ds) goto error; } - ds->configure_vlan_while_not_filtering = false; - return 0; error: dev_err_ratelimited(priv->dev, "%s: %i\n", __func__, ret); @@ -1144,6 +1169,209 @@ static void ar9331_sw_port_bridge_leave(struct dsa_switch *ds, int port, ar9331_sw_port_bridge_mod(ds, port, br, false); } +static int ar9331_port_vlan_filtering(struct dsa_switch *ds, int port, + bool vlan_filtering, + struct netlink_ext_ack *extack) +{ + struct ar9331_sw_priv *priv = (struct ar9331_sw_priv *)ds->priv; + struct regmap *regmap = priv->regmap; + u32 mode; + int ret; + + if (vlan_filtering) + mode = AR9331_SW_8021Q_MODE_SECURE; + else + mode = AR9331_SW_8021Q_MODE_NONE; + + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), + AR9331_SW_PORT_VLAN_8021Q_MODE, + FIELD_PREP(AR9331_SW_PORT_VLAN_8021Q_MODE, + mode)); + if (ret) + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); + + return ret; +} + +static int ar9331_sw_vt_wait(struct ar9331_sw_priv *priv, u32 *f0) +{ + struct regmap *regmap = priv->regmap; + + return regmap_read_poll_timeout(regmap, + AR9331_SW_REG_VLAN_TABLE_FUNCTION0, + *f0, !(*f0 & AR9331_SW_VT0_BUSY), + 100, 2000); +} + +static int ar9331_sw_port_vt_rmw(struct ar9331_sw_priv *priv, u16 vid, + u8 port_mask_set, u8 port_mask_clr) +{ + struct regmap *regmap = priv->regmap; + u32 f0, f1, port_mask = 0, port_mask_new, func; + struct ar9331_sw_vlan_db *vdb = NULL; + int ret, i; + + if (!vid) + return 0; + + ret = ar9331_sw_vt_wait(priv, &f0); + if (ret) + return ret; + + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, 0); + if (ret) + goto error; + + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, 0); + if (ret) + goto error; + + for (i = 0; i < ARRAY_SIZE(priv->vdb); i++) { + if (priv->vdb[i].vid == vid) { + vdb = &priv->vdb[i]; + break; + } + } + + ret = regmap_read(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, &f1); + if (ret) + return ret; + + if (vdb) + port_mask = vdb->port_mask; + + port_mask_new = port_mask & ~port_mask_clr; + port_mask_new |= port_mask_set; + + if (port_mask_new && port_mask_new == port_mask) + return 0; + + if (port_mask_new) { + func = AR9331_SW_VT0_FUNC_LOAD_ENTRY; + } else { + func = AR9331_SW_VT0_FUNC_PURGE_ENTRY; + port_mask_new = port_mask; + } + + if (vdb) { + vdb->port_mask = port_mask_new; + + if (!port_mask_new) + vdb->vid = 0; + } else { + for (i = 0; i < ARRAY_SIZE(priv->vdb); i++) { + if (!priv->vdb[i].vid) { + vdb = &priv->vdb[i]; + break; + } + } + + if (!vdb) + return -ENOMEM; + + vdb->vid = vid; + vdb->port_mask = port_mask_new; + } + + f0 = FIELD_PREP(AR9331_SW_VT0_VID, vid) | + FIELD_PREP(AR9331_SW_VT0_FUNC, func) | + AR9331_SW_VT0_BUSY; + f1 = FIELD_PREP(AR9331_SW_VT1_VID_MEM, port_mask_new) | + AR9331_SW_VT1_VALID; + + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION1, f1); + if (ret) + return ret; + + ret = regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, f0); + if (ret) + return ret; + + ret = ar9331_sw_vt_wait(priv, &f0); + if (ret) + return ret; + + if (f0 & AR9331_SW_VT0_FULL_VIO) { + /* cleanup error status */ + regmap_write(regmap, AR9331_SW_REG_VLAN_TABLE_FUNCTION0, 0); + return -ENOMEM; + } + + return 0; + +error: + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); + + return ret; +} + +static int ar9331_port_vlan_set_pvid(struct ar9331_sw_priv *priv, int port, + u16 pvid) +{ + struct regmap *regmap = priv->regmap; + int ret; + u32 mask, val; + + mask = AR9331_SW_PORT_VLAN_8021Q_MODE | + AR9331_SW_PORT_VLAN_FORCE_DEFALUT_VID_EN | + AR9331_SW_PORT_VLAN_FORCE_PORT_VLAN_EN; + val = AR9331_SW_PORT_VLAN_FORCE_DEFALUT_VID_EN | + AR9331_SW_PORT_VLAN_FORCE_PORT_VLAN_EN | + FIELD_PREP(AR9331_SW_PORT_VLAN_8021Q_MODE, + AR9331_SW_8021Q_MODE_FALLBACK); + + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), + mask, val); + if (ret) + return ret; + + return regmap_update_bits(regmap, AR9331_SW_REG_PORT_VLAN(port), + AR9331_SW_PORT_VLAN_PORT_VID, + FIELD_PREP(AR9331_SW_PORT_VLAN_PORT_VID, + pvid)); +} + +static int ar9331_port_vlan_add(struct dsa_switch *ds, int port, + const struct switchdev_obj_port_vlan *vlan, + struct netlink_ext_ack *extack) +{ + struct ar9331_sw_priv *priv = ds->priv; + struct regmap *regmap = priv->regmap; + int ret, mode; + + ret = ar9331_sw_port_vt_rmw(priv, vlan->vid, BIT(port), 0); + if (ret) + goto error; + + if (vlan->flags & BRIDGE_VLAN_INFO_PVID) + ret = ar9331_port_vlan_set_pvid(priv, port, vlan->vid); + + if (ret) + goto error; + + if (vlan->flags & BRIDGE_VLAN_INFO_UNTAGGED) + mode = AR9331_SW_PORT_CTRL_EG_VLAN_MODE_STRIP; + else + mode = AR9331_SW_PORT_CTRL_EG_VLAN_MODE_ADD; + + ret = regmap_update_bits(regmap, AR9331_SW_REG_PORT_CTRL(port), + AR9331_SW_PORT_CTRL_EG_VLAN_MODE, mode); + if (ret) + goto error; + + return 0; +error: + dev_err(priv->dev, "%s: error: %pe\n", __func__, ERR_PTR(ret)); + + return ret; +} + +static int ar9331_port_vlan_del(struct dsa_switch *ds, int port, + const struct switchdev_obj_port_vlan *vlan) +{ + return ar9331_sw_port_vt_rmw(ds->priv, vlan->vid, 0, BIT(port)); +} + static const struct dsa_switch_ops ar9331_sw_ops = { .get_tag_protocol = ar9331_sw_get_tag_protocol, .setup = ar9331_sw_setup, @@ -1162,6 +1390,9 @@ static const struct dsa_switch_ops ar9331_sw_ops = { .set_ageing_time = ar9331_sw_set_ageing_time, .port_bridge_join = ar9331_sw_port_bridge_join, .port_bridge_leave = ar9331_sw_port_bridge_leave, + .port_vlan_filtering = ar9331_port_vlan_filtering, + .port_vlan_add = ar9331_port_vlan_add, + .port_vlan_del = ar9331_port_vlan_del, }; static irqreturn_t ar9331_sw_irq(int irq, void *data)
This switch provides simple VLAN resolution database for 16 entries. With this database we can cover typical functionalities as port based VLANs, untagged and tagged egress. Port based ingress filtering. The VLAN database is working on top of forwarding database. So, potentially, we can have multiple VLANs on top of multiple bridges. Hawing one VLAN on top of multiple bridges will fail on different levels, most probably DSA framework should warn if some one won't to make something likes this. Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> --- drivers/net/dsa/qca/ar9331.c | 235 ++++++++++++++++++++++++++++++++++- 1 file changed, 233 insertions(+), 2 deletions(-)