Before introducing A-TCAM support we need to make sure all the necessary
fields are configurable and not hard coded to values that worked for the
C-TCAM only use case.

This includes - for example - the ability to configure the eRP table
used by the TCAM region.

Signed-off-by: Ido Schimmel <ido...@mellanox.com>
Reviewed-by: Jiri Pirko <j...@mellanox.com>
---
 drivers/net/ethernet/mellanox/mlxsw/reg.h     | 22 ++++++++++++++++---
 .../mellanox/mlxsw/spectrum_acl_atcam.c       |  2 +-
 2 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlxsw/reg.h 
b/drivers/net/ethernet/mellanox/mlxsw/reg.h
index 596fddfb3850..f4e16ac20fea 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/reg.h
+++ b/drivers/net/ethernet/mellanox/mlxsw/reg.h
@@ -2663,12 +2663,28 @@ MLXSW_ITEM_BIT_ARRAY(reg, pererp, erpt_vector, 0x14, 4, 
1);
  */
 MLXSW_ITEM32(reg, pererp, master_rp_id, 0x18, 0, 4);
 
-static inline void mlxsw_reg_pererp_pack(char *payload, u16 region_id)
+static inline void mlxsw_reg_pererp_erp_vector_pack(char *payload,
+                                                   unsigned long *erp_vector,
+                                                   unsigned long size)
+{
+       unsigned long bit;
+
+       for_each_set_bit(bit, erp_vector, size)
+               mlxsw_reg_pererp_erpt_vector_set(payload, bit, true);
+}
+
+static inline void mlxsw_reg_pererp_pack(char *payload, u16 region_id,
+                                        bool ctcam_le, bool erpt_pointer_valid,
+                                        u8 erpt_bank_pointer, u8 erpt_pointer,
+                                        u8 master_rp_id)
 {
        MLXSW_REG_ZERO(pererp, payload);
        mlxsw_reg_pererp_region_id_set(payload, region_id);
-       mlxsw_reg_pererp_ctcam_le_set(payload, true);
-       mlxsw_reg_pererp_erpt_pointer_valid_set(payload, true);
+       mlxsw_reg_pererp_ctcam_le_set(payload, ctcam_le);
+       mlxsw_reg_pererp_erpt_pointer_valid_set(payload, erpt_pointer_valid);
+       mlxsw_reg_pererp_erpt_bank_pointer_set(payload, erpt_bank_pointer);
+       mlxsw_reg_pererp_erpt_pointer_set(payload, erpt_pointer);
+       mlxsw_reg_pererp_master_rp_id_set(payload, master_rp_id);
 }
 
 /* IEDR - Infrastructure Entry Delete Register
diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c 
b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
index b1b3b0e0c00e..a27d3b0f9fcb 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_atcam.c
@@ -72,7 +72,7 @@ mlxsw_sp_acl_atcam_region_erp_init(struct mlxsw_sp *mlxsw_sp,
 {
        char pererp_pl[MLXSW_REG_PERERP_LEN];
 
-       mlxsw_reg_pererp_pack(pererp_pl, region_id);
+       mlxsw_reg_pererp_pack(pererp_pl, region_id, true, true, 0, 0, 0);
        return mlxsw_reg_write(mlxsw_sp->core, MLXSW_REG(pererp), pererp_pl);
 }
 
-- 
2.17.1

Reply via email to