[PATCH v6 02/27] net/nfp: fix the promiscuous mode control functions

[Chaoyong He]

The original functions of promiscuous mode can't process the
representor port rightly, revise the logic to do that.
Fixes: e1124c4f8a45 ("net/nfp: add flower representor framework")

Signed-off-by: Chaoyong He <chaoyong...@corigine.com>
---
 drivers/net/nfp/flower/nfp_flower_representor.c | 26 +++++++++++++++++++++++--
 1 file changed, 24 insertions(+), 2 deletions(-)

diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c 
b/drivers/net/nfp/flower/nfp_flower_representor.c
index 0e60f50..cecdf46 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.c
+++ b/drivers/net/nfp/flower/nfp_flower_representor.c
@@ -415,6 +415,9 @@
 static int
 nfp_flower_repr_promiscuous_enable(struct rte_eth_dev *dev)
 {
+       int ret;
+       uint32_t update;
+       uint32_t new_ctrl;
        struct nfp_net_hw *pf_hw;
        struct nfp_flower_representor *repr;
 
@@ -431,12 +434,23 @@
                return 0;
        }
 
-       return nfp_net_promisc_enable(pf_hw->eth_dev);
+       new_ctrl = pf_hw->ctrl | NFP_NET_CFG_CTRL_PROMISC;
+       update = NFP_NET_CFG_UPDATE_GEN;
+       ret = nfp_net_reconfig(pf_hw, new_ctrl, update);
+       if (ret < 0)
+               return ret;
+
+       pf_hw->ctrl = new_ctrl;
+
+       return 0;
 }
 
 static int
 nfp_flower_repr_promiscuous_disable(struct rte_eth_dev *dev)
 {
+       int ret;
+       uint32_t update;
+       uint32_t new_ctrl;
        struct nfp_net_hw *pf_hw;
        struct nfp_flower_representor *repr;
 
@@ -448,7 +462,15 @@
                return 0;
        }
 
-       return nfp_net_promisc_disable(pf_hw->eth_dev);
+       new_ctrl = pf_hw->ctrl & ~NFP_NET_CFG_CTRL_PROMISC;
+       update = NFP_NET_CFG_UPDATE_GEN;
+       ret = nfp_net_reconfig(pf_hw, new_ctrl, update);
+       if (ret < 0)
+               return ret;
+
+       pf_hw->ctrl = new_ctrl;
+
+       return 0;
 }
 
 static int
-- 
1.8.3.1