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

Rely on the previously introduced fallback and let the core
call devlink in order to get the physical port name.

Signed-off-by: Jiri Pirko <j...@mellanox.com>
---
v1->v2:
- new patch
---
 drivers/net/ethernet/broadcom/bnxt/bnxt.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c 
b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
index eca36cac594e..35e34e23ba33 100644
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
@@ -10048,23 +10048,6 @@ static int bnxt_bridge_setlink(struct net_device *dev, 
struct nlmsghdr *nlh,
        return rc;
 }
 
-static int bnxt_get_phys_port_name(struct net_device *dev, char *buf,
-                                  size_t len)
-{
-       struct bnxt *bp = netdev_priv(dev);
-       int rc;
-
-       /* The PF and it's VF-reps only support the switchdev framework */
-       if (!BNXT_PF(bp))
-               return -EOPNOTSUPP;
-
-       rc = snprintf(buf, len, "p%d", bp->pf.port_id);
-
-       if (rc >= len)
-               return -EOPNOTSUPP;
-       return 0;
-}
-
 int bnxt_get_port_parent_id(struct net_device *dev,
                            struct netdev_phys_item_id *ppid)
 {
@@ -10122,7 +10105,6 @@ static const struct net_device_ops bnxt_netdev_ops = {
        .ndo_bridge_getlink     = bnxt_bridge_getlink,
        .ndo_bridge_setlink     = bnxt_bridge_setlink,
        .ndo_get_port_parent_id = bnxt_get_port_parent_id,
-       .ndo_get_phys_port_name = bnxt_get_phys_port_name,
        .ndo_get_devlink_port   = bnxt_get_devlink_port,
 };
 
-- 
2.17.2

Reply via email to