nfp: do not handle nn->port defined case in nfp_net_get_phys_port_name()

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 <[email protected]>
Reviewed-by: Jakub Kicinski <[email protected]>
Signed-off-by: David S. Miller <[email protected]>

diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
index b676943e54f45b82b80a6a9a5727e1364336f8d3..99200b5dac76d50f5669243da2949d4f45b5c155 100644 (file)
--- 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;