]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
dpaa2-switch: rename dpaa2_switch_tc_parse_action to specify the ACL
authorIoana Ciornei <ioana.ciornei@nxp.com>
Thu, 29 Jul 2021 17:18:53 +0000 (20:18 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 29 Jul 2021 20:34:45 +0000 (21:34 +0100)
Until now, the dpaa2_switch_tc_parse_action() function was used for all
the supported tc actions since all of them were implemented by adding
ACL table entries. In the next commits, the dpaa2-switch driver will
gain mirroring support which is not using the same HW feature.

Make sure that we specify the ACL in the function name so that we make
it clear that it's only used for specific actions.

Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/dpaa2/dpaa2-switch-flower.c

index f9451ec5f2cbbc22790c1d46404bf96ff99e8d0e..639efb3edeec7c83d888aeb38fd295c8bf84cbbd 100644 (file)
@@ -339,10 +339,10 @@ dpaa2_switch_acl_tbl_remove_entry(struct dpaa2_switch_acl_tbl *acl_tbl,
        return 0;
 }
 
-static int dpaa2_switch_tc_parse_action(struct ethsw_core *ethsw,
-                                       struct flow_action_entry *cls_act,
-                                       struct dpsw_acl_result *dpsw_act,
-                                       struct netlink_ext_ack *extack)
+static int dpaa2_switch_tc_parse_action_acl(struct ethsw_core *ethsw,
+                                           struct flow_action_entry *cls_act,
+                                           struct dpsw_acl_result *dpsw_act,
+                                           struct netlink_ext_ack *extack)
 {
        int err = 0;
 
@@ -403,8 +403,8 @@ int dpaa2_switch_cls_flower_replace(struct dpaa2_switch_acl_tbl *acl_tbl,
                goto free_acl_entry;
 
        act = &rule->action.entries[0];
-       err = dpaa2_switch_tc_parse_action(ethsw, act,
-                                          &acl_entry->cfg.result, extack);
+       err = dpaa2_switch_tc_parse_action_acl(ethsw, act,
+                                              &acl_entry->cfg.result, extack);
        if (err)
                goto free_acl_entry;
 
@@ -459,8 +459,8 @@ int dpaa2_switch_cls_matchall_replace(struct dpaa2_switch_acl_tbl *acl_tbl,
                return -ENOMEM;
 
        act = &cls->rule->action.entries[0];
-       err = dpaa2_switch_tc_parse_action(ethsw, act,
-                                          &acl_entry->cfg.result, extack);
+       err = dpaa2_switch_tc_parse_action_acl(ethsw, act,
+                                              &acl_entry->cfg.result, extack);
        if (err)
                goto free_acl_entry;