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

When getting firmware VNIC version, the logic for representor
ports and other ports is inverse, fix it.

Fixes: c4de52eca76c ("net/nfp: remove redundancy 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: Long Wu <long...@corigine.com>
Reviewed-by: Peng Zhang <peng.zh...@corigine.com>
---
 drivers/net/nfp/nfp_net_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/nfp/nfp_net_common.c b/drivers/net/nfp/nfp_net_common.c
index 20e628bfd1..bd43925960 100644
--- a/drivers/net/nfp/nfp_net_common.c
+++ b/drivers/net/nfp/nfp_net_common.c
@@ -2155,7 +2155,7 @@ nfp_net_firmware_version_get(struct rte_eth_dev *dev,
 
        hw = nfp_net_get_hw(dev);
 
-       if (rte_eth_dev_is_repr(dev)) {
+       if (!rte_eth_dev_is_repr(dev)) {
                snprintf(vnic_version, FW_VER_LEN, "%d.%d.%d.%d",
                        hw->ver.extend, hw->ver.class,
                        hw->ver.major, hw->ver.minor);
-- 
2.39.1

Reply via email to