From: Vladimir Oltean <vladimir.olt...@nxp.com> Even though we should really share the implementation with the ocelot switchdev driver, that one needs a little bit of rework first, since its struct ocelot_port_tc only supports one tc matchall action at a time, which at the moment is used for port policers. Whereas DSA keeps a list of port-based actions in struct dsa_slave_priv::mall_tc_list, so it is much more easily extensible. It is too tempting to add the implementation for the port priority directly in Felix at the moment, which is what we do.
Signed-off-by: Vladimir Oltean <vladimir.olt...@nxp.com> --- drivers/net/dsa/ocelot/felix.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/drivers/net/dsa/ocelot/felix.c b/drivers/net/dsa/ocelot/felix.c index 768a74dc462a..5cc42c3aaf0d 100644 --- a/drivers/net/dsa/ocelot/felix.c +++ b/drivers/net/dsa/ocelot/felix.c @@ -739,6 +739,20 @@ static void felix_port_policer_del(struct dsa_switch *ds, int port) ocelot_port_policer_del(ocelot, port); } +static int felix_port_priority_set(struct dsa_switch *ds, int port, + struct dsa_mall_skbedit_tc_entry *skbedit) +{ + struct ocelot *ocelot = ds->priv; + + ocelot_rmw_gix(ocelot, + ANA_PORT_QOS_CFG_QOS_DEFAULT_VAL(skbedit->priority), + ANA_PORT_QOS_CFG_QOS_DEFAULT_VAL_M, + ANA_PORT_QOS_CFG, + port); + + return 0; +} + static int felix_port_setup_tc(struct dsa_switch *ds, int port, enum tc_setup_type type, void *type_data) @@ -786,6 +800,7 @@ const struct dsa_switch_ops felix_switch_ops = { .port_max_mtu = felix_get_max_mtu, .port_policer_add = felix_port_policer_add, .port_policer_del = felix_port_policer_del, + .port_priority_set = felix_port_priority_set, .cls_flower_add = felix_cls_flower_add, .cls_flower_del = felix_cls_flower_del, .cls_flower_stats = felix_cls_flower_stats, -- 2.25.1