[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.he at 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