From: Qin Ke <qin...@corigine.com>

The callback function of promiscuous_disable in nfp_flower_pf_repr_dev_ops
and nfp_flower_repr_dev_ops should be nfp_net_promisc_disable, not
nfp_net_promisc_enable, fix it.

Fixes: 39d82d2 ("net/nfp: fix promiscuous mode for representor port")
Cc: Chaoyong He <chaoyong...@corigine.com>
Cc: sta...@dpdk.org

Signed-off-by: Qin Ke <qin...@corigine.com>
Reviewed-by: Chaoyong He <chaoyong...@corigine.com>
Reviewed-by: Niklas Söderlund <niklas.soderl...@corigine.com>
---
 drivers/net/nfp/flower/nfp_flower_representor.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c 
b/drivers/net/nfp/flower/nfp_flower_representor.c
index 6b4ba5b9a2..3225a4b84a 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.c
+++ b/drivers/net/nfp/flower/nfp_flower_representor.c
@@ -518,7 +518,7 @@ static const struct eth_dev_ops nfp_flower_pf_repr_dev_ops 
= {
        .stats_reset          = nfp_flower_repr_stats_reset,
 
        .promiscuous_enable   = nfp_net_promisc_enable,
-       .promiscuous_disable  = nfp_net_promisc_enable,
+       .promiscuous_disable  = nfp_net_promisc_disable,
 
        .mac_addr_set         = nfp_flower_repr_mac_addr_set,
        .fw_version_get       = nfp_repr_firmware_version_get,
@@ -540,7 +540,7 @@ static const struct eth_dev_ops nfp_flower_repr_dev_ops = {
        .stats_reset          = nfp_flower_repr_stats_reset,
 
        .promiscuous_enable   = nfp_net_promisc_enable,
-       .promiscuous_disable  = nfp_net_promisc_enable,
+       .promiscuous_disable  = nfp_net_promisc_disable,
 
        .mac_addr_set         = nfp_flower_repr_mac_addr_set,
        .fw_version_get       = nfp_repr_firmware_version_get,
-- 
2.39.1

Reply via email to