CN20K does not have IE engines, hence change the default eng group
for cn20k and use legacy for cn10k or older version.

Signed-off-by: Nithin Dabilpuram <ndabilpu...@marvell.com>
---
 drivers/common/cnxk/roc_cpt.c         |  6 +++---
 drivers/common/cnxk/roc_cpt.h         | 10 +++++++---
 drivers/common/cnxk/roc_nix_inl.c     | 15 +++++++++++----
 drivers/common/cnxk/roc_nix_inl_dev.c | 12 ++++++++----
 drivers/event/cnxk/cn9k_worker.h      |  2 +-
 drivers/net/cnxk/cn10k_rx.h           |  2 +-
 drivers/net/cnxk/cn10k_tx.h           |  4 ++--
 drivers/net/cnxk/cn20k_tx.h           |  4 ++--
 8 files changed, 35 insertions(+), 20 deletions(-)

diff --git a/drivers/common/cnxk/roc_cpt.c b/drivers/common/cnxk/roc_cpt.c
index a6d2d83f76..b4bf0ccd64 100644
--- a/drivers/common/cnxk/roc_cpt.c
+++ b/drivers/common/cnxk/roc_cpt.c
@@ -931,9 +931,9 @@ roc_cpt_iq_reset(struct roc_cpt_lf *lf)
                plt_write64(lf_ctl.u, lf->rbase + CPT_LF_CTL);
 
                if (roc_model_is_cn9k())
-                       cpt_9k_lf_rst_lmtst(lf, ROC_CPT_DFLT_ENG_GRP_SE);
+                       cpt_9k_lf_rst_lmtst(lf, ROC_LEGACY_CPT_DFLT_ENG_GRP_SE);
                else
-                       cpt_10k_lf_rst_lmtst(lf, ROC_CPT_DFLT_ENG_GRP_SE);
+                       cpt_10k_lf_rst_lmtst(lf, 
ROC_LEGACY_CPT_DFLT_ENG_GRP_SE);
 
                plt_read64(lf->rbase + CPT_LF_INPROG);
                plt_delay_us(2);
@@ -1205,7 +1205,7 @@ roc_cpt_ctx_write(struct roc_cpt_lf *lf, void *sa_dptr, 
void *sa_cptr,
 
        if (lf->roc_cpt == NULL) {
                if (roc_cpt_has_ie_engines())
-                       egrp = ROC_CPT_DFLT_ENG_GRP_SE_IE;
+                       egrp = ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE;
                else
                        egrp = ROC_CPT_DFLT_ENG_GRP_SE;
        } else {
diff --git a/drivers/common/cnxk/roc_cpt.h b/drivers/common/cnxk/roc_cpt.h
index ac27479371..30bd2a094d 100644
--- a/drivers/common/cnxk/roc_cpt.h
+++ b/drivers/common/cnxk/roc_cpt.h
@@ -15,10 +15,14 @@
 #define ROC_LOADFVC_MAJOR_OP 0x01UL
 #define ROC_LOADFVC_MINOR_OP 0x08UL
 
+/* Default engine groups for CN9K, CN10K */
+#define ROC_LEGACY_CPT_DFLT_ENG_GRP_SE   0UL
+#define ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE 1UL
+#define ROC_LEGACY_CPT_DFLT_ENG_GRP_AE   2UL
+
 /* Default engine groups */
-#define ROC_CPT_DFLT_ENG_GRP_SE           0UL
-#define ROC_CPT_DFLT_ENG_GRP_SE_IE 1UL
-#define ROC_CPT_DFLT_ENG_GRP_AE           2UL
+#define ROC_CPT_DFLT_ENG_GRP_SE 0UL
+#define ROC_CPT_DFLT_ENG_GRP_AE 1UL
 
 #define ROC_CPT_MAX_LFS 64
 #define ROC_CPT_MAX_BLKS 2
diff --git a/drivers/common/cnxk/roc_nix_inl.c 
b/drivers/common/cnxk/roc_nix_inl.c
index 88d5a678b1..6b7532b1f0 100644
--- a/drivers/common/cnxk/roc_nix_inl.c
+++ b/drivers/common/cnxk/roc_nix_inl.c
@@ -811,7 +811,10 @@ nix_inl_eng_caps_get(struct nix *nix)
        inst.rptr = (uint64_t)rptr;
        inst.w4.s.opcode_major = ROC_LOADFVC_MAJOR_OP;
        inst.w4.s.opcode_minor = ROC_LOADFVC_MINOR_OP;
-       inst.w7.s.egrp = ROC_CPT_DFLT_ENG_GRP_SE;
+       if (roc_model_is_cn9k() || roc_model_is_cn10k())
+               inst.w7.s.egrp = ROC_LEGACY_CPT_DFLT_ENG_GRP_SE;
+       else
+               inst.w7.s.egrp = ROC_CPT_DFLT_ENG_GRP_SE;
 
        /* Use 1 min timeout for the poll */
        const uint64_t timeout = plt_tsc_cycles() + 60 * plt_tsc_hz();
@@ -1053,10 +1056,14 @@ roc_nix_inl_outb_init(struct roc_nix *roc_nix)
                ctx_ilen_valid = true;
        }
 
+       if (roc_model_is_cn9k() || roc_model_is_cn10k())
+               eng_grpmask = (1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_SE |
+                              1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE |
+                              1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_AE);
+       else
+               eng_grpmask = (1ULL << ROC_CPT_DFLT_ENG_GRP_SE | 1ULL << 
ROC_CPT_DFLT_ENG_GRP_AE);
+
        /* Alloc CPT LF */
-       eng_grpmask = (1ULL << ROC_CPT_DFLT_ENG_GRP_SE |
-                      1ULL << ROC_CPT_DFLT_ENG_GRP_SE_IE |
-                      1ULL << ROC_CPT_DFLT_ENG_GRP_AE);
        rc = cpt_lfs_alloc(dev, eng_grpmask, blkaddr,
                           !roc_nix->ipsec_out_sso_pffunc, ctx_ilen_valid, 
ctx_ilen,
                           rx_inj, nb_lf - 1);
diff --git a/drivers/common/cnxk/roc_nix_inl_dev.c 
b/drivers/common/cnxk/roc_nix_inl_dev.c
index d26cbee0cc..da28b22bcc 100644
--- a/drivers/common/cnxk/roc_nix_inl_dev.c
+++ b/drivers/common/cnxk/roc_nix_inl_dev.c
@@ -179,15 +179,19 @@ nix_inl_cpt_setup(struct nix_inl_dev *inl_dev, bool 
inl_dev_sso)
        if (!inl_dev->attach_cptlf)
                return 0;
 
-       /* Alloc CPT LF */
-       eng_grpmask = (1ULL << ROC_CPT_DFLT_ENG_GRP_SE |
-                      1ULL << ROC_CPT_DFLT_ENG_GRP_SE_IE |
-                      1ULL << ROC_CPT_DFLT_ENG_GRP_AE);
+       if (roc_model_is_cn9k() || roc_model_is_cn10k())
+               eng_grpmask = (1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_SE |
+                              1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE |
+                              1ULL << ROC_LEGACY_CPT_DFLT_ENG_GRP_AE);
+       else
+               eng_grpmask = (1ULL << ROC_CPT_DFLT_ENG_GRP_SE | 1ULL << 
ROC_CPT_DFLT_ENG_GRP_AE);
+
        if (roc_errata_cpt_has_ctx_fetch_issue()) {
                ctx_ilen = (ROC_NIX_INL_OT_IPSEC_INB_HW_SZ / 128) - 1;
                ctx_ilen_valid = true;
        }
 
+       /* Alloc CPT LF */
        rc = cpt_lfs_alloc(dev, eng_grpmask, RVU_BLOCK_ADDR_CPT0, inl_dev_sso, 
ctx_ilen_valid,
                           ctx_ilen, inl_dev->rx_inj_ena, inl_dev->nb_cptlf - 
1);
        if (rc) {
diff --git a/drivers/event/cnxk/cn9k_worker.h b/drivers/event/cnxk/cn9k_worker.h
index 10abbdfbb5..513d397991 100644
--- a/drivers/event/cnxk/cn9k_worker.h
+++ b/drivers/event/cnxk/cn9k_worker.h
@@ -652,7 +652,7 @@ cn9k_sso_hws_xmit_sec_one(const struct cn9k_eth_txq *txq, 
uint64_t base,
        sa_base &= ~(ROC_NIX_INL_SA_BASE_ALIGN - 1);
 
        sa = (uintptr_t)roc_nix_inl_on_ipsec_outb_sa(sa_base, mdata.sa_idx);
-       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | sa);
+       ucode_cmd[3] = (ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE << 61 | sa);
        ucode_cmd[0] = (((ROC_IE_ON_OUTB_MAX_CTX_LEN << 8) |
                         ROC_IE_ON_MAJOR_OP_PROCESS_OUTBOUND_IPSEC)
                                << 48 |
diff --git a/drivers/net/cnxk/cn10k_rx.h b/drivers/net/cnxk/cn10k_rx.h
index 990dfbee3e..3430318193 100644
--- a/drivers/net/cnxk/cn10k_rx.h
+++ b/drivers/net/cnxk/cn10k_rx.h
@@ -1363,7 +1363,7 @@ cn10k_nix_inj_pkts(struct rte_security_session **sess, 
struct cnxk_ethdev_inj_cf
                                ((uint64_t)sess_priv.dec_ttl) << 34 | 
m->pkt_len);
 
                ucode_cmd[2] = 0;
-               ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | 
sa);
+               ucode_cmd[3] = (ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL 
<< 60 | sa);
 
                /* Move to our line */
                laddr = LMT_OFF(c_lbase, lnum, loff ? 64 : 0);
diff --git a/drivers/net/cnxk/cn10k_tx.h b/drivers/net/cnxk/cn10k_tx.h
index 7d9b259a5f..5a8e728bc1 100644
--- a/drivers/net/cnxk/cn10k_tx.h
+++ b/drivers/net/cnxk/cn10k_tx.h
@@ -547,7 +547,7 @@ cn10k_nix_prep_sec_vec(struct rte_mbuf *m, uint64x2_t 
*cmd0, uint64x2_t *cmd1,
        tag = sa_base & 0xFFFFUL;
        sa_base &= ~0xFFFFUL;
        sa = (uintptr_t)roc_nix_inl_ot_ipsec_outb_sa(sa_base, sess_priv.sa_idx);
-       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | sa);
+       ucode_cmd[3] = (ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | 
sa);
        ucode_cmd[0] = (ROC_IE_OT_MAJOR_OP_PROCESS_OUTBOUND_IPSEC << 48 | 1UL 
<< 54 |
                        ((uint64_t)sess_priv.chksum) << 32 | 
((uint64_t)sess_priv.dec_ttl) << 34 |
                        pkt_len);
@@ -687,7 +687,7 @@ cn10k_nix_prep_sec(struct rte_mbuf *m, uint64_t *cmd, 
uintptr_t *nixtx_addr,
        tag = sa_base & 0xFFFFUL;
        sa_base &= ~0xFFFFUL;
        sa = (uintptr_t)roc_nix_inl_ot_ipsec_outb_sa(sa_base, sess_priv.sa_idx);
-       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | sa);
+       ucode_cmd[3] = (ROC_LEGACY_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | 
sa);
        ucode_cmd[0] = (ROC_IE_OT_MAJOR_OP_PROCESS_OUTBOUND_IPSEC << 48 | 1UL 
<< 54 |
                        ((uint64_t)sess_priv.chksum) << 32 | 
((uint64_t)sess_priv.dec_ttl) << 34 |
                        pkt_len);
diff --git a/drivers/net/cnxk/cn20k_tx.h b/drivers/net/cnxk/cn20k_tx.h
index c731406529..7674c1644a 100644
--- a/drivers/net/cnxk/cn20k_tx.h
+++ b/drivers/net/cnxk/cn20k_tx.h
@@ -533,7 +533,7 @@ cn20k_nix_prep_sec_vec(struct rte_mbuf *m, uint64x2_t 
*cmd0, uint64x2_t *cmd1,
        tag = sa_base & 0xFFFFUL;
        sa_base &= ~0xFFFFUL;
        sa = (uintptr_t)roc_nix_inl_ot_ipsec_outb_sa(sa_base, sess_priv.sa_idx);
-       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | sa);
+       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE << 61 | 1UL << 60 | sa);
        ucode_cmd[0] = (ROC_IE_OT_MAJOR_OP_PROCESS_OUTBOUND_IPSEC << 48 | 1UL 
<< 54 |
                        ((uint64_t)sess_priv.chksum) << 32 | 
((uint64_t)sess_priv.dec_ttl) << 34 |
                        pkt_len);
@@ -671,7 +671,7 @@ cn20k_nix_prep_sec(struct rte_mbuf *m, uint64_t *cmd, 
uintptr_t *nixtx_addr, uin
        tag = sa_base & 0xFFFFUL;
        sa_base &= ~0xFFFFUL;
        sa = (uintptr_t)roc_nix_inl_ot_ipsec_outb_sa(sa_base, sess_priv.sa_idx);
-       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE_IE << 61 | 1UL << 60 | sa);
+       ucode_cmd[3] = (ROC_CPT_DFLT_ENG_GRP_SE << 61 | 1UL << 60 | sa);
        ucode_cmd[0] = (ROC_IE_OT_MAJOR_OP_PROCESS_OUTBOUND_IPSEC << 48 | 1UL 
<< 54 |
                        ((uint64_t)sess_priv.chksum) << 32 | 
((uint64_t)sess_priv.dec_ttl) << 34 |
                        pkt_len);
-- 
2.34.1

Reply via email to