In cn20k, PF and VF bit encoding format in a pcifunc has changed to
accommodate more no of PFs. Hence defining the pf/vf encoding
parameters for cn20k and cn9k/10k platforms.

Signed-off-by: Harman Kalra <hka...@marvell.com>
---
 drivers/common/cnxk/roc_dev.c      |  1 +
 drivers/common/cnxk/roc_dev_priv.h | 31 ++++++++++++++++++++++++------
 drivers/common/cnxk/roc_npc.c      |  5 ++++-
 3 files changed, 30 insertions(+), 7 deletions(-)

diff --git a/drivers/common/cnxk/roc_dev.c b/drivers/common/cnxk/roc_dev.c
index b9bbcac3d8..0b434e4a8a 100644
--- a/drivers/common/cnxk/roc_dev.c
+++ b/drivers/common/cnxk/roc_dev.c
@@ -23,6 +23,7 @@
 /* VF interrupt message pending bits - mbox or flr */
 #define ROC_DEV_MBOX_PEND BIT_ULL(0)
 #define ROC_DEV_FLR_PEND  BIT_ULL(1)
+
 static void *
 mbox_mem_map(off_t off, size_t size)
 {
diff --git a/drivers/common/cnxk/roc_dev_priv.h 
b/drivers/common/cnxk/roc_dev_priv.h
index 976eb48346..75395e5fd5 100644
--- a/drivers/common/cnxk/roc_dev_priv.h
+++ b/drivers/common/cnxk/roc_dev_priv.h
@@ -7,12 +7,19 @@
 
 #define DEV_HWCAP_F_VF BIT_ULL(0) /* VF device */
 
+/* PF and VF bit encoding parameters in pcifunc */
+#define RVU_PFVF_PF_SHIFT_CN20K          9
+#define RVU_PFVF_PF_MASK_CN20K   0x7F
+#define RVU_PFVF_FUNC_SHIFT_CN20K 0
+#define RVU_PFVF_FUNC_MASK_CN20K  0x1FF
+
 #define RVU_PFVF_PF_SHIFT   10
 #define RVU_PFVF_PF_MASK    0x3F
 #define RVU_PFVF_FUNC_SHIFT 0
 #define RVU_PFVF_FUNC_MASK  0x3FF
-#define RVU_MAX_VF         64 /* RVU_PF_VFPF_MBOX_INT(0..1) */
-#define RVU_MAX_INT_RETRY   3
+
+#define RVU_MAX_VF       64 /* RVU_PF_VFPF_MBOX_INT(0..1) */
+#define RVU_MAX_INT_RETRY 3
 
 /* PF/VF message handling timer */
 #define VF_PF_MBOX_TIMER_MS (20 * 1000)
@@ -52,25 +59,37 @@ struct dev_ops {
 static inline int
 dev_get_vf(uint16_t pf_func)
 {
-       return (((pf_func >> RVU_PFVF_FUNC_SHIFT) & RVU_PFVF_FUNC_MASK) - 1);
+       if (roc_model_is_cn20k())
+               return (((pf_func >> RVU_PFVF_FUNC_SHIFT_CN20K) & 
RVU_PFVF_FUNC_MASK_CN20K) - 1);
+       else
+               return (((pf_func >> RVU_PFVF_FUNC_SHIFT) & RVU_PFVF_FUNC_MASK) 
- 1);
 }
 
 static inline int
 dev_get_pf(uint16_t pf_func)
 {
-       return (pf_func >> RVU_PFVF_PF_SHIFT) & RVU_PFVF_PF_MASK;
+       if (roc_model_is_cn20k())
+               return (pf_func >> RVU_PFVF_PF_SHIFT_CN20K) & 
RVU_PFVF_PF_MASK_CN20K;
+       else
+               return (pf_func >> RVU_PFVF_PF_SHIFT) & RVU_PFVF_PF_MASK;
 }
 
 static inline int
 dev_pf_func(int pf, int vf)
 {
-       return (pf << RVU_PFVF_PF_SHIFT) | ((vf << RVU_PFVF_FUNC_SHIFT) + 1);
+       if (roc_model_is_cn20k())
+               return (pf << RVU_PFVF_PF_SHIFT_CN20K) | ((vf << 
RVU_PFVF_FUNC_SHIFT_CN20K) + 1);
+       else
+               return (pf << RVU_PFVF_PF_SHIFT) | ((vf << RVU_PFVF_FUNC_SHIFT) 
+ 1);
 }
 
 static inline int
 dev_is_afvf(uint16_t pf_func)
 {
-       return !(pf_func & ~RVU_PFVF_FUNC_MASK);
+       if (roc_model_is_cn20k())
+               return !(pf_func & ~RVU_PFVF_FUNC_MASK_CN20K);
+       else
+               return !(pf_func & ~RVU_PFVF_FUNC_MASK);
 }
 
 struct mbox_sync {
diff --git a/drivers/common/cnxk/roc_npc.c b/drivers/common/cnxk/roc_npc.c
index 37e1a6a7ef..8a951b6360 100644
--- a/drivers/common/cnxk/roc_npc.c
+++ b/drivers/common/cnxk/roc_npc.c
@@ -616,7 +616,10 @@ npc_parse_actions(struct roc_npc *roc_npc, const struct 
roc_npc_attr *attr,
                case ROC_NPC_ACTION_TYPE_VF:
                        vf_act = (const struct roc_npc_action_vf 
*)actions->conf;
                        req_act |= ROC_NPC_ACTION_TYPE_VF;
-                       vf_id = vf_act->id & RVU_PFVF_FUNC_MASK;
+                       if (roc_model_is_cn20k())
+                               vf_id = vf_act->id & RVU_PFVF_FUNC_MASK_CN20K;
+                       else
+                               vf_id = vf_act->id & RVU_PFVF_FUNC_MASK;
                        pf_func &= (0xfc00);
                        pf_func = (pf_func | (vf_id + 1));
                        break;
-- 
2.25.1

Reply via email to