octeontx2-af: Add NIX Errata workaround on CN10K silicon
authorGeetha sowjanya <gakula@marvell.com>
Fri, 17 Feb 2023 05:51:12 +0000 (11:21 +0530)
committerDavid S. Miller <davem@davemloft.net>
Mon, 20 Feb 2023 10:42:37 +0000 (10:42 +0000)
This patch adds workaround for below 2 HW erratas

1. Due to improper clock gating, NIXRX may free the same
NPA buffer multiple times.. to avoid this, always enable
NIX RX conditional clock.

2. NIX FIFO does not get initialized on reset, if the SMQ
flush is triggered before the first packet is processed, it
will lead to undefined state. The workaround to perform SMQ
flush only if packet count is non-zero in MDQ.

Signed-off-by: Geetha sowjanya <gakula@marvell.com>
Signed-off-by: Sunil Kovvuri Goutham <sgoutham@marvell.com>
Signed-off-by: Sai Krishna <saikrishnag@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/octeontx2/af/rvu.h
drivers/net/ethernet/marvell/octeontx2/af/rvu_cn10k.c
drivers/net/ethernet/marvell/octeontx2/af/rvu_nix.c
drivers/net/ethernet/marvell/octeontx2/af/rvu_reg.h

index 5eea2b6..389663a 100644 (file)
@@ -888,6 +888,9 @@ int rvu_cpt_init(struct rvu *rvu);
 int rvu_set_channels_base(struct rvu *rvu);
 void rvu_program_channels(struct rvu *rvu);
 
+/* CN10K NIX */
+void rvu_nix_block_cn10k_init(struct rvu *rvu, struct nix_hw *nix_hw);
+
 /* CN10K RVU - LMT*/
 void rvu_reset_lmt_map_tbl(struct rvu *rvu, u16 pcifunc);
 
index 7dbbc11..4ad9ff0 100644 (file)
@@ -538,3 +538,21 @@ void rvu_program_channels(struct rvu *rvu)
        rvu_lbk_set_channels(rvu);
        rvu_rpm_set_channels(rvu);
 }
+
+void rvu_nix_block_cn10k_init(struct rvu *rvu, struct nix_hw *nix_hw)
+{
+       int blkaddr = nix_hw->blkaddr;
+       u64 cfg;
+
+       /* Set AF vWQE timer interval to a LF configurable range of
+        * 6.4us to 1.632ms.
+        */
+       rvu_write64(rvu, blkaddr, NIX_AF_VWQE_TIMER, 0x3FULL);
+
+       /* Enable NIX RX stream and global conditional clock to
+        * avoild multiple free of NPA buffers.
+        */
+       cfg = rvu_read64(rvu, blkaddr, NIX_AF_CFG);
+       cfg |= BIT_ULL(1) | BIT_ULL(2);
+       rvu_write64(rvu, blkaddr, NIX_AF_CFG, cfg);
+}
index 89e9456..26e639e 100644 (file)
@@ -2058,6 +2058,13 @@ static int nix_smq_flush(struct rvu *rvu, int blkaddr,
        int err, restore_tx_en = 0;
        u64 cfg;
 
+       if (!is_rvu_otx2(rvu)) {
+               /* Skip SMQ flush if pkt count is zero */
+               cfg = rvu_read64(rvu, blkaddr, NIX_AF_MDQX_IN_MD_COUNT(smq));
+               if (!cfg)
+                       return 0;
+       }
+
        /* enable cgx tx if disabled */
        if (is_pf_cgxmapped(rvu, pf)) {
                rvu_get_cgx_lmac_id(rvu->pf2cgxlmac_map[pf], &cgx_id, &lmac_id);
@@ -4309,6 +4316,9 @@ static int rvu_nix_block_init(struct rvu *rvu, struct nix_hw *nix_hw)
 
        rvu_write64(rvu, blkaddr, NIX_AF_SEB_CFG, cfg);
 
+       if (!is_rvu_otx2(rvu))
+               rvu_nix_block_cn10k_init(rvu, nix_hw);
+
        if (is_block_implemented(hw, blkaddr)) {
                err = nix_setup_txschq(rvu, nix_hw, blkaddr);
                if (err)
index 5437bd2..1729b22 100644 (file)
 #define NIX_AF_RX_CFG                  (0x00D0)
 #define NIX_AF_AVG_DELAY               (0x00E0)
 #define NIX_AF_CINT_DELAY              (0x00F0)
+#define NIX_AF_VWQE_TIMER              (0x00F8)
 #define NIX_AF_RX_MCAST_BASE           (0x0100)
 #define NIX_AF_RX_MCAST_CFG            (0x0110)
 #define NIX_AF_RX_MCAST_BUF_BASE       (0x0120)
 #define NIX_AF_RX_NPC_MIRROR_DROP      (0x4730)
 #define NIX_AF_RX_ACTIVE_CYCLES_PCX(a) (0x4800 | (a) << 16)
 #define NIX_AF_LINKX_CFG(a)            (0x4010 | (a) << 17)
+#define NIX_AF_MDQX_IN_MD_COUNT(a)     (0x14e0 | (a) << 16)
 
 #define NIX_PRIV_AF_INT_CFG            (0x8000000)
 #define NIX_PRIV_LFX_CFG               (0x8000010)