mlxsw: spectrum_acl: Remove unnecessary arg on action_replace call path

No need to pass ruleset/group and chunk pointers on action_replace call
path, nobody uses them.

Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum.h b/drivers/net/ethernet/mellanox/mlxsw/spectrum.h
index 9384b108c8c26e1b8ea4a6c1c48cdc95287b6ff9..4fe0996c7cddb8c2ad0dee198f498ea0a76cff70 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.h
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.h
@@ -721,8 +721,7 @@ struct mlxsw_sp_acl_tcam_ops {
                          void *region_priv, void *chunk_priv,
                          void *entry_priv);
        int (*entry_action_replace)(struct mlxsw_sp *mlxsw_sp,
-                                   void *region_priv, void *chunk_priv,
-                                   void *entry_priv,
+                                   void *region_priv, void *entry_priv,
                                    struct mlxsw_sp_acl_rule_info *rulei);
        int (*entry_activity_get)(struct mlxsw_sp *mlxsw_sp,
                                  void *region_priv, void *entry_priv,

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum1_acl_tcam.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum1_acl_tcam.c
index fe270c1a26a621b99a37463315f0dbf70e03f953..6e444525713fb611b7a84061cc9315dc462a632b 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum1_acl_tcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum1_acl_tcam.c
@@ -194,8 +194,7 @@ static void mlxsw_sp1_acl_tcam_entry_del(struct mlxsw_sp *mlxsw_sp,
 
 static int
 mlxsw_sp1_acl_tcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
-                                       void *region_priv, void *chunk_priv,
-                                       void *entry_priv,
+                                       void *region_priv, void *entry_priv,
                                        struct mlxsw_sp_acl_rule_info *rulei)
 {
        return -EOPNOTSUPP;

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum2_acl_tcam.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum2_acl_tcam.c
index 234ab51916db9031ed4635e9dacf9bc7983e82a5..d380b34039603585e38dbdfcbb0738c6d617ae26 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum2_acl_tcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum2_acl_tcam.c
@@ -212,18 +212,15 @@ static void mlxsw_sp2_acl_tcam_entry_del(struct mlxsw_sp *mlxsw_sp,
 
 static int
 mlxsw_sp2_acl_tcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
-                                       void *region_priv, void *chunk_priv,
-                                       void *entry_priv,
+                                       void *region_priv, void *entry_priv,
                                        struct mlxsw_sp_acl_rule_info *rulei)
 {
        struct mlxsw_sp2_acl_tcam_region *region = region_priv;
-       struct mlxsw_sp2_acl_tcam_chunk *chunk = chunk_priv;
        struct mlxsw_sp2_acl_tcam_entry *entry = entry_priv;
 
        entry->act_block = rulei->act_block;
        return mlxsw_sp_acl_atcam_entry_action_replace(mlxsw_sp,
                                                       &region->aregion,
-                                                      &chunk->achunk,
                                                       &entry->aentry, rulei);
 }
 

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl.c
index 695d33358988e8d1e899299dc79a84e4d3b2bfe9..bd40ed065f3e860662797c0d572478e030fd65ae 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl.c
@@ -742,8 +742,7 @@ int mlxsw_sp_acl_rule_action_replace(struct mlxsw_sp *mlxsw_sp,
        rulei = mlxsw_sp_acl_rule_rulei(rule);
        rulei->act_block = afa_block;
 
-       return ops->rule_action_replace(mlxsw_sp, ruleset->priv, rule->priv,
-                                       rule->rulei);
+       return ops->rule_action_replace(mlxsw_sp, rule->priv, rule->rulei);
 }
 
 struct mlxsw_sp_acl_rule *

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
index 80fb268d51a5e24dc67469d47ac6925abc72b853..40dc76a5c4128691c087f505f21df7a0cee9cca0 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
@@ -603,7 +603,6 @@ void mlxsw_sp_acl_atcam_entry_del(struct mlxsw_sp *mlxsw_sp,
 int
 mlxsw_sp_acl_atcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
                                        struct mlxsw_sp_acl_atcam_region *aregion,
-                                       struct mlxsw_sp_acl_atcam_chunk *achunk,
                                        struct mlxsw_sp_acl_atcam_entry *aentry,
                                        struct mlxsw_sp_acl_rule_info *rulei)
 {
@@ -612,7 +611,6 @@ mlxsw_sp_acl_atcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
        if (mlxsw_sp_acl_atcam_is_centry(aentry))
                err = mlxsw_sp_acl_ctcam_entry_action_replace(mlxsw_sp,
                                                              &aregion->cregion,
-                                                             &achunk->cchunk,
                                                              &aentry->centry,
                                                              rulei);
        else

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_ctcam.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_ctcam.c
index ac222833a5cff65560e9e584c84ef3ff382aef3f..05680a7e6c56ef8b6561a97e679e46f8b495178c 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_ctcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_ctcam.c
@@ -223,7 +223,6 @@ void mlxsw_sp_acl_ctcam_entry_del(struct mlxsw_sp *mlxsw_sp,
 
 int mlxsw_sp_acl_ctcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
                                            struct mlxsw_sp_acl_ctcam_region *cregion,
-                                           struct mlxsw_sp_acl_ctcam_chunk *cchunk,
                                            struct mlxsw_sp_acl_ctcam_entry *centry,
                                            struct mlxsw_sp_acl_rule_info *rulei)
 {

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c
index fe230acf92a9913373f07173c74bfd4c49ab82a8..5d29c5050401a19c8b070123b8e206753a5479b1 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c
@@ -781,7 +781,6 @@ static void mlxsw_sp_acl_tcam_entry_del(struct mlxsw_sp *mlxsw_sp,
 
 static int
 mlxsw_sp_acl_tcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
-                                      struct mlxsw_sp_acl_tcam_group *group,
                                       struct mlxsw_sp_acl_tcam_entry *entry,
                                       struct mlxsw_sp_acl_rule_info *rulei)
 {
@@ -789,7 +788,7 @@ mlxsw_sp_acl_tcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
        struct mlxsw_sp_acl_tcam_chunk *chunk = entry->chunk;
        struct mlxsw_sp_acl_tcam_region *region = chunk->region;
 
-       return ops->entry_action_replace(mlxsw_sp, region->priv, chunk->priv,
+       return ops->entry_action_replace(mlxsw_sp, region->priv,
                                         entry->priv, rulei);
 }
 
@@ -955,7 +954,6 @@ mlxsw_sp_acl_tcam_flower_rule_del(struct mlxsw_sp *mlxsw_sp, void *rule_priv)
 
 static int
 mlxsw_sp_acl_tcam_flower_rule_action_replace(struct mlxsw_sp *mlxsw_sp,
-                                            void *ruleset_priv,
                                             void *rule_priv,
                                             struct mlxsw_sp_acl_rule_info *rulei)
 {
@@ -1084,14 +1082,13 @@ mlxsw_sp_acl_tcam_mr_rule_del(struct mlxsw_sp *mlxsw_sp, void *rule_priv)
 
 static int
 mlxsw_sp_acl_tcam_mr_rule_action_replace(struct mlxsw_sp *mlxsw_sp,
-                                        void *ruleset_priv, void *rule_priv,
+                                        void *rule_priv,
                                         struct mlxsw_sp_acl_rule_info *rulei)
 {
-       struct mlxsw_sp_acl_tcam_mr_ruleset *ruleset = ruleset_priv;
        struct mlxsw_sp_acl_tcam_mr_rule *rule = rule_priv;
 
-       return mlxsw_sp_acl_tcam_entry_action_replace(mlxsw_sp, &ruleset->group,
-                                                     &rule->entry, rulei);
+       return mlxsw_sp_acl_tcam_entry_action_replace(mlxsw_sp, &rule->entry,
+                                                     rulei);
 }
 
 static int

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.h b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.h
index 0f1a9dee63de2a76afcda76996d13236a59c5ac0..10512b7c6d50836706234cfaaa17a31be2f68a02 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.h
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.h
@@ -48,8 +48,7 @@ struct mlxsw_sp_acl_profile_ops {
                        void *ruleset_priv, void *rule_priv,
                        struct mlxsw_sp_acl_rule_info *rulei);
        void (*rule_del)(struct mlxsw_sp *mlxsw_sp, void *rule_priv);
-       int (*rule_action_replace)(struct mlxsw_sp *mlxsw_sp,
-                                  void *ruleset_priv, void *rule_priv,
+       int (*rule_action_replace)(struct mlxsw_sp *mlxsw_sp, void *rule_priv,
                                   struct mlxsw_sp_acl_rule_info *rulei);
        int (*rule_activity_get)(struct mlxsw_sp *mlxsw_sp, void *rule_priv,
                                 bool *activity);
@@ -126,7 +125,6 @@ void mlxsw_sp_acl_ctcam_entry_del(struct mlxsw_sp *mlxsw_sp,
                                  struct mlxsw_sp_acl_ctcam_entry *centry);
 int mlxsw_sp_acl_ctcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
                                            struct mlxsw_sp_acl_ctcam_region *cregion,
-                                           struct mlxsw_sp_acl_ctcam_chunk *cchunk,
                                            struct mlxsw_sp_acl_ctcam_entry *centry,
                                            struct mlxsw_sp_acl_rule_info *rulei);
 static inline unsigned int
@@ -224,7 +222,6 @@ void mlxsw_sp_acl_atcam_entry_del(struct mlxsw_sp *mlxsw_sp,
                                  struct mlxsw_sp_acl_atcam_entry *aentry);
 int mlxsw_sp_acl_atcam_entry_action_replace(struct mlxsw_sp *mlxsw_sp,
                                            struct mlxsw_sp_acl_atcam_region *aregion,
-                                           struct mlxsw_sp_acl_atcam_chunk *achunk,
                                            struct mlxsw_sp_acl_atcam_entry *aentry,
                                            struct mlxsw_sp_acl_rule_info *rulei);
 int mlxsw_sp_acl_atcam_init(struct mlxsw_sp *mlxsw_sp,