From: Jiri Pirko <j...@mellanox.com>

If nn->port is defined it means that devlink_port has been registered
for this port as well. Devlink core is handling the port name
formatting.

Signed-off-by: Jiri Pirko <j...@mellanox.com>
---
v1->v2:
- new patch
---
 drivers/net/ethernet/netronome/nfp/nfp_net_common.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c 
b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
index b676943e54f4..99200b5dac76 100644
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
@@ -3324,8 +3324,11 @@ nfp_net_get_phys_port_name(struct net_device *netdev, 
char *name, size_t len)
        struct nfp_net *nn = netdev_priv(netdev);
        int n;
 
+       /* If port is defined, devlink_port is registered and devlink core
+        * is taking care of name formatting.
+        */
        if (nn->port)
-               return nfp_port_get_phys_port_name(netdev, name, len);
+               return -EOPNOTSUPP;
 
        if (nn->dp.is_vf || nn->vnic_no_name)
                return -EOPNOTSUPP;
-- 
2.17.2

Reply via email to