net: remove unused ndo_get_devlink_port
authorJiri Pirko <jiri@nvidia.com>
Wed, 2 Nov 2022 16:02:10 +0000 (17:02 +0100)
committerJakub Kicinski <kuba@kernel.org>
Fri, 4 Nov 2022 03:48:36 +0000 (20:48 -0700)
Remove ndo_get_devlink_port which is no longer used alongside with the
implementations in drivers.

Signed-off-by: Jiri Pirko <jiri@nvidia.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
22 files changed:
drivers/net/ethernet/broadcom/bnxt/bnxt.c
drivers/net/ethernet/fungible/funeth/funeth_main.c
drivers/net/ethernet/intel/ice/ice_main.c
drivers/net/ethernet/intel/ice/ice_repr.c
drivers/net/ethernet/marvell/prestera/prestera_devlink.c
drivers/net/ethernet/marvell/prestera/prestera_devlink.h
drivers/net/ethernet/marvell/prestera/prestera_main.c
drivers/net/ethernet/mellanox/mlx5/core/en/devlink.c
drivers/net/ethernet/mellanox/mlx5/core/en/devlink.h
drivers/net/ethernet/mellanox/mlx5/core/en_main.c
drivers/net/ethernet/mellanox/mlx5/core/en_rep.c
drivers/net/ethernet/mellanox/mlxsw/minimal.c
drivers/net/ethernet/mellanox/mlxsw/spectrum.c
drivers/net/ethernet/mscc/ocelot_net.c
drivers/net/ethernet/netronome/nfp/nfp_app.h
drivers/net/ethernet/netronome/nfp/nfp_devlink.c
drivers/net/ethernet/netronome/nfp/nfp_net_common.c
drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
drivers/net/ethernet/ti/am65-cpsw-nuss.c
drivers/net/netdevsim/netdev.c
include/linux/netdevice.h
net/dsa/slave.c

index 9604294b67aa67147578690142977a2c18e0e3f8..03272aa485117e00d6ad6803f742c53d092e05cc 100644 (file)
@@ -13073,13 +13073,6 @@ int bnxt_get_port_parent_id(struct net_device *dev,
        return 0;
 }
 
-static struct devlink_port *bnxt_get_devlink_port(struct net_device *dev)
-{
-       struct bnxt *bp = netdev_priv(dev);
-
-       return &bp->dl_port;
-}
-
 static const struct net_device_ops bnxt_netdev_ops = {
        .ndo_open               = bnxt_open,
        .ndo_start_xmit         = bnxt_start_xmit,
@@ -13111,7 +13104,6 @@ static const struct net_device_ops bnxt_netdev_ops = {
        .ndo_xdp_xmit           = bnxt_xdp_xmit,
        .ndo_bridge_getlink     = bnxt_bridge_getlink,
        .ndo_bridge_setlink     = bnxt_bridge_setlink,
-       .ndo_get_devlink_port   = bnxt_get_devlink_port,
 };
 
 static void bnxt_remove_one(struct pci_dev *pdev)
index 208dc89f4972c2c8935f0e3c8326b2bc6dbf9c27..b4cce30e526a079cb7949a34c95ca1490ae23679 100644 (file)
@@ -1178,13 +1178,6 @@ static int fun_xdp(struct net_device *dev, struct netdev_bpf *xdp)
        }
 }
 
-static struct devlink_port *fun_get_devlink_port(struct net_device *netdev)
-{
-       struct funeth_priv *fp = netdev_priv(netdev);
-
-       return &fp->dl_port;
-}
-
 static int fun_init_vports(struct fun_ethdev *ed, unsigned int n)
 {
        if (ed->num_vports)
@@ -1350,7 +1343,6 @@ static const struct net_device_ops fun_netdev_ops = {
        .ndo_set_vf_vlan        = fun_set_vf_vlan,
        .ndo_set_vf_rate        = fun_set_vf_rate,
        .ndo_get_vf_config      = fun_get_vf_config,
-       .ndo_get_devlink_port   = fun_get_devlink_port,
 };
 
 #define GSO_ENCAP_FLAGS (NETIF_F_GSO_GRE | NETIF_F_GSO_IPXIP4 | \
index 74d25fda11bdf4053c306257aa76334d22777f1c..a9fc89aebebeac7fa6e0175a73845d19787a48a8 100644 (file)
@@ -298,20 +298,6 @@ static int ice_clear_promisc(struct ice_vsi *vsi, u8 promisc_m)
        return status;
 }
 
-/**
- * ice_get_devlink_port - Get devlink port from netdev
- * @netdev: the netdevice structure
- */
-static struct devlink_port *ice_get_devlink_port(struct net_device *netdev)
-{
-       struct ice_pf *pf = ice_netdev_to_pf(netdev);
-
-       if (!ice_is_switchdev_running(pf))
-               return NULL;
-
-       return &pf->devlink_port;
-}
-
 /**
  * ice_vsi_sync_fltr - Update the VSI filter list to the HW
  * @vsi: ptr to the VSI
@@ -9107,5 +9093,4 @@ static const struct net_device_ops ice_netdev_ops = {
        .ndo_bpf = ice_xdp,
        .ndo_xdp_xmit = ice_xdp_xmit,
        .ndo_xsk_wakeup = ice_xsk_wakeup,
-       .ndo_get_devlink_port = ice_get_devlink_port,
 };
index 663a7a0e1814cc19397f412c637bdf7bcda340a7..0483eb14c288ac5e551f4de1f082a6a9ebcc89a7 100644 (file)
@@ -134,14 +134,6 @@ static int ice_repr_stop(struct net_device *netdev)
        return 0;
 }
 
-static struct devlink_port *
-ice_repr_get_devlink_port(struct net_device *netdev)
-{
-       struct ice_repr *repr = ice_netdev_to_repr(netdev);
-
-       return &repr->vf->devlink_port;
-}
-
 /**
  * ice_repr_sp_stats64 - get slow path stats for port representor
  * @dev: network interface device structure
@@ -250,7 +242,6 @@ static const struct net_device_ops ice_repr_netdev_ops = {
        .ndo_open = ice_repr_open,
        .ndo_stop = ice_repr_stop,
        .ndo_start_xmit = ice_eswitch_port_start_xmit,
-       .ndo_get_devlink_port = ice_repr_get_devlink_port,
        .ndo_setup_tc = ice_repr_setup_tc,
        .ndo_has_offload_stats = ice_repr_ndo_has_offload_stats,
        .ndo_get_offload_stats = ice_repr_ndo_get_offload_stats,
index 637b8fee65e74c6658d8d8bd1280c99529ea01a0..84ad05c9f12deeaaecb039615bb137318fe098e1 100644 (file)
@@ -445,13 +445,6 @@ void prestera_devlink_port_unregister(struct prestera_port *port)
        devlink_port_unregister(&port->dl_port);
 }
 
-struct devlink_port *prestera_devlink_get_port(struct net_device *dev)
-{
-       struct prestera_port *port = netdev_priv(dev);
-
-       return &port->dl_port;
-}
-
 int prestera_devlink_traps_register(struct prestera_switch *sw)
 {
        const u32 groups_count = ARRAY_SIZE(prestera_trap_groups_arr);
index 04e8556f748a4b83da2e7a224a0b0587a6b7c20b..bf84ad6fd87ec868af42865a7cefdb632a4bfe61 100644 (file)
@@ -15,8 +15,6 @@ void prestera_devlink_unregister(struct prestera_switch *sw);
 int prestera_devlink_port_register(struct prestera_port *port);
 void prestera_devlink_port_unregister(struct prestera_port *port);
 
-struct devlink_port *prestera_devlink_get_port(struct net_device *dev);
-
 void prestera_devlink_trap_report(struct prestera_port *port,
                                  struct sk_buff *skb, u8 cpu_code);
 int prestera_devlink_traps_register(struct prestera_switch *sw);
index a76bf3606aa619f515ea40e3f8dc38df197eca7c..edbdda8f958da46aed485d991d8b76b09dedeef3 100644 (file)
@@ -569,7 +569,6 @@ static const struct net_device_ops prestera_netdev_ops = {
        .ndo_change_mtu = prestera_port_change_mtu,
        .ndo_get_stats64 = prestera_port_get_stats64,
        .ndo_set_mac_address = prestera_port_set_mac_address,
-       .ndo_get_devlink_port = prestera_devlink_get_port,
 };
 
 int prestera_port_autoneg_set(struct prestera_port *port, u64 link_modes)
index ce0e56f856d6c16bc8dfd723576e0b62086935ff..83adaabf59f5256b0cf102f4327f795ac326f971 100644 (file)
@@ -62,13 +62,3 @@ void mlx5e_devlink_port_unregister(struct mlx5e_priv *priv)
        if (!(priv->mdev->priv.flags & MLX5_PRIV_FLAGS_MLX5E_LOCKED_FLOW))
                devl_unlock(devlink);
 }
-
-struct devlink_port *mlx5e_get_devlink_port(struct net_device *dev)
-{
-       struct mlx5e_priv *priv = netdev_priv(dev);
-
-       if (!netif_device_present(dev))
-               return NULL;
-
-       return mlx5e_devlink_get_dl_port(priv);
-}
index 1c203257ac306fc29a32036ad8ad10e244e94264..4f238d4fff55cd9bd11f9b46ec28e2a38b64b201 100644 (file)
@@ -9,7 +9,6 @@
 
 int mlx5e_devlink_port_register(struct mlx5e_priv *priv);
 void mlx5e_devlink_port_unregister(struct mlx5e_priv *priv);
-struct devlink_port *mlx5e_get_devlink_port(struct net_device *dev);
 
 static inline struct devlink_port *
 mlx5e_devlink_get_dl_port(struct mlx5e_priv *priv)
index 97e876472a06c5ffa5d0b37bc9bcc1c11ee988ef..78d34c039fa739bd12d64eb35635347a7dd0e4d5 100644 (file)
@@ -4897,7 +4897,6 @@ const struct net_device_ops mlx5e_netdev_ops = {
        .ndo_has_offload_stats   = mlx5e_has_offload_stats,
        .ndo_get_offload_stats   = mlx5e_get_offload_stats,
 #endif
-       .ndo_get_devlink_port    = mlx5e_get_devlink_port,
 };
 
 static u32 mlx5e_choose_lro_timeout(struct mlx5_core_dev *mdev, u32 wanted_timeout)
index 5412633b9c8c8a1d287a2b68a7c4cabaa9b5a07a..1b53e8852c86c9f0a87f7d4f2def900074985bf9 100644 (file)
@@ -607,15 +607,6 @@ static int mlx5e_rep_change_mtu(struct net_device *netdev, int new_mtu)
        return mlx5e_change_mtu(netdev, new_mtu, NULL);
 }
 
-static struct devlink_port *mlx5e_rep_get_devlink_port(struct net_device *netdev)
-{
-       struct mlx5e_priv *priv = netdev_priv(netdev);
-       struct mlx5e_rep_priv *rpriv = priv->ppriv;
-       struct mlx5_core_dev *dev = priv->mdev;
-
-       return mlx5_esw_offloads_devlink_port(dev->priv.eswitch, rpriv->rep->vport);
-}
-
 static int mlx5e_rep_change_carrier(struct net_device *dev, bool new_carrier)
 {
        struct mlx5e_priv *priv = netdev_priv(dev);
@@ -644,7 +635,6 @@ static const struct net_device_ops mlx5e_netdev_ops_rep = {
        .ndo_stop                = mlx5e_rep_close,
        .ndo_start_xmit          = mlx5e_xmit,
        .ndo_setup_tc            = mlx5e_rep_setup_tc,
-       .ndo_get_devlink_port    = mlx5e_rep_get_devlink_port,
        .ndo_get_stats64         = mlx5e_rep_get_stats,
        .ndo_has_offload_stats   = mlx5e_rep_has_offload_stats,
        .ndo_get_offload_stats   = mlx5e_rep_get_offload_stats,
index 177cf7e4db34c970d8c1f16e3121d724c9a884a1..6b56eadd736e5783343e9cd0b7a2edd71709d38f 100644 (file)
@@ -81,20 +81,9 @@ static int mlxsw_m_port_stop(struct net_device *dev)
        return 0;
 }
 
-static struct devlink_port *
-mlxsw_m_port_get_devlink_port(struct net_device *dev)
-{
-       struct mlxsw_m_port *mlxsw_m_port = netdev_priv(dev);
-       struct mlxsw_m *mlxsw_m = mlxsw_m_port->mlxsw_m;
-
-       return mlxsw_core_port_devlink_port_get(mlxsw_m->core,
-                                               mlxsw_m_port->local_port);
-}
-
 static const struct net_device_ops mlxsw_m_port_netdev_ops = {
        .ndo_open               = mlxsw_m_port_open,
        .ndo_stop               = mlxsw_m_port_stop,
-       .ndo_get_devlink_port   = mlxsw_m_port_get_devlink_port,
 };
 
 static void mlxsw_m_module_get_drvinfo(struct net_device *dev,
index b13739e1f8f3079183bf48d3f871693f9b4320cf..04dc79da6024fa95fcf226f4dead83a39e1af197 100644 (file)
@@ -1259,16 +1259,6 @@ static int mlxsw_sp_set_features(struct net_device *dev,
        return 0;
 }
 
-static struct devlink_port *
-mlxsw_sp_port_get_devlink_port(struct net_device *dev)
-{
-       struct mlxsw_sp_port *mlxsw_sp_port = netdev_priv(dev);
-       struct mlxsw_sp *mlxsw_sp = mlxsw_sp_port->mlxsw_sp;
-
-       return mlxsw_core_port_devlink_port_get(mlxsw_sp->core,
-                                               mlxsw_sp_port->local_port);
-}
-
 static int mlxsw_sp_port_hwtstamp_set(struct mlxsw_sp_port *mlxsw_sp_port,
                                      struct ifreq *ifr)
 {
@@ -1342,7 +1332,6 @@ static const struct net_device_ops mlxsw_sp_port_netdev_ops = {
        .ndo_vlan_rx_add_vid    = mlxsw_sp_port_add_vid,
        .ndo_vlan_rx_kill_vid   = mlxsw_sp_port_kill_vid,
        .ndo_set_features       = mlxsw_sp_set_features,
-       .ndo_get_devlink_port   = mlxsw_sp_port_get_devlink_port,
        .ndo_eth_ioctl          = mlxsw_sp_port_ioctl,
 };
 
index 5efc07751c8d7eb02bd9304b76399009515f6aab..f50dada2bb8e1fbbcc7d5984de2903153f06ab5d 100644 (file)
@@ -194,15 +194,6 @@ void ocelot_port_devlink_teardown(struct ocelot *ocelot, int port)
        devlink_port_unregister(dlp);
 }
 
-static struct devlink_port *ocelot_get_devlink_port(struct net_device *dev)
-{
-       struct ocelot_port_private *priv = netdev_priv(dev);
-       struct ocelot *ocelot = priv->port.ocelot;
-       int port = priv->port.index;
-
-       return &ocelot->devlink_ports[port];
-}
-
 int ocelot_setup_tc_cls_flower(struct ocelot_port_private *priv,
                               struct flow_cls_offload *f,
                               bool ingress)
@@ -925,7 +916,6 @@ static const struct net_device_ops ocelot_port_netdev_ops = {
        .ndo_set_features               = ocelot_set_features,
        .ndo_setup_tc                   = ocelot_setup_tc,
        .ndo_eth_ioctl                  = ocelot_ioctl,
-       .ndo_get_devlink_port           = ocelot_get_devlink_port,
 };
 
 struct net_device *ocelot_port_to_netdev(struct ocelot *ocelot, int port)
index dd56207df246510d4f1eaa6123e283fad738d962..90707346a4ef72fc4765fc95fc5f5228157dede3 100644 (file)
@@ -445,6 +445,4 @@ int nfp_app_nic_vnic_alloc(struct nfp_app *app, struct nfp_net *nn,
 int nfp_app_nic_vnic_init_phy_port(struct nfp_pf *pf, struct nfp_app *app,
                                   struct nfp_net *nn, unsigned int id);
 
-struct devlink_port *nfp_devlink_get_devlink_port(struct net_device *netdev);
-
 #endif
index f3f0f11d8b520ab5649f66585cb27766fb32f3b7..8bfd48d50ef0d9322ad2c1a4e8b9309ee347143c 100644 (file)
@@ -362,14 +362,3 @@ void nfp_devlink_port_unregister(struct nfp_port *port)
 {
        devl_port_unregister(&port->dl_port);
 }
-
-struct devlink_port *nfp_devlink_get_devlink_port(struct net_device *netdev)
-{
-       struct nfp_port *port;
-
-       port = nfp_port_from_netdev(netdev);
-       if (!port)
-               return NULL;
-
-       return &port->dl_port;
-}
index a5ca5c4a7896f7960b5b65f776e39f6f09140281..8c1a870bc0e52caf88e480bfa48646f33867476d 100644 (file)
@@ -2013,7 +2013,6 @@ const struct net_device_ops nfp_nfd3_netdev_ops = {
        .ndo_get_phys_port_name = nfp_net_get_phys_port_name,
        .ndo_bpf                = nfp_net_xdp,
        .ndo_xsk_wakeup         = nfp_net_xsk_wakeup,
-       .ndo_get_devlink_port   = nfp_devlink_get_devlink_port,
        .ndo_bridge_getlink     = nfp_net_bridge_getlink,
        .ndo_bridge_setlink     = nfp_net_bridge_setlink,
 };
@@ -2044,7 +2043,6 @@ const struct net_device_ops nfp_nfdk_netdev_ops = {
        .ndo_features_check     = nfp_net_features_check,
        .ndo_get_phys_port_name = nfp_net_get_phys_port_name,
        .ndo_bpf                = nfp_net_xdp,
-       .ndo_get_devlink_port   = nfp_devlink_get_devlink_port,
        .ndo_bridge_getlink     = nfp_net_bridge_getlink,
        .ndo_bridge_setlink     = nfp_net_bridge_setlink,
 };
index a6b6ca1fd55eec081bc99ce051f41bd065d67250..3af1229a3f08d244fac1001409bb9e5fb71ca54e 100644 (file)
@@ -275,7 +275,6 @@ const struct net_device_ops nfp_repr_netdev_ops = {
        .ndo_set_features       = nfp_port_set_features,
        .ndo_set_mac_address    = eth_mac_addr,
        .ndo_get_port_parent_id = nfp_port_get_port_parent_id,
-       .ndo_get_devlink_port   = nfp_devlink_get_devlink_port,
 };
 
 void
index b2a08f3f0e9322a75076e5083896e27bf3d0523e..0dbdce4dc077039b6003f567d924a3c11350a596 100644 (file)
@@ -1380,13 +1380,6 @@ static void am65_cpsw_nuss_ndo_get_stats(struct net_device *dev,
        stats->tx_dropped       = dev->stats.tx_dropped;
 }
 
-static struct devlink_port *am65_cpsw_ndo_get_devlink_port(struct net_device *ndev)
-{
-       struct am65_cpsw_port *port = am65_ndev_to_port(ndev);
-
-       return &port->devlink_port;
-}
-
 static const struct net_device_ops am65_cpsw_nuss_netdev_ops = {
        .ndo_open               = am65_cpsw_nuss_ndo_slave_open,
        .ndo_stop               = am65_cpsw_nuss_ndo_slave_stop,
@@ -1400,7 +1393,6 @@ static const struct net_device_ops am65_cpsw_nuss_netdev_ops = {
        .ndo_vlan_rx_kill_vid   = am65_cpsw_nuss_ndo_slave_kill_vid,
        .ndo_eth_ioctl          = am65_cpsw_nuss_ndo_slave_ioctl,
        .ndo_setup_tc           = am65_cpsw_qos_ndo_setup_tc,
-       .ndo_get_devlink_port   = am65_cpsw_ndo_get_devlink_port,
 };
 
 static void am65_cpsw_nuss_mac_config(struct phylink_config *config, unsigned int mode,
index e7cbae743e86d8668cf32567a1de6739cc58b369..6db6a75ff9b9b0f741badca9a45783feef67df8d 100644 (file)
@@ -238,13 +238,6 @@ nsim_set_features(struct net_device *dev, netdev_features_t features)
        return 0;
 }
 
-static struct devlink_port *nsim_get_devlink_port(struct net_device *dev)
-{
-       struct netdevsim *ns = netdev_priv(dev);
-
-       return &ns->nsim_dev_port->devlink_port;
-}
-
 static const struct net_device_ops nsim_netdev_ops = {
        .ndo_start_xmit         = nsim_start_xmit,
        .ndo_set_rx_mode        = nsim_set_rx_mode,
@@ -263,7 +256,6 @@ static const struct net_device_ops nsim_netdev_ops = {
        .ndo_setup_tc           = nsim_setup_tc,
        .ndo_set_features       = nsim_set_features,
        .ndo_bpf                = nsim_bpf,
-       .ndo_get_devlink_port   = nsim_get_devlink_port,
 };
 
 static const struct net_device_ops nsim_vf_netdev_ops = {
@@ -275,7 +267,6 @@ static const struct net_device_ops nsim_vf_netdev_ops = {
        .ndo_get_stats64        = nsim_get_stats64,
        .ndo_setup_tc           = nsim_setup_tc,
        .ndo_set_features       = nsim_set_features,
-       .ndo_get_devlink_port   = nsim_get_devlink_port,
 };
 
 static void nsim_setup(struct net_device *dev)
index f048a30ea10bcb4c7e133e09f8e0abb447128dfb..d45713a065680f943530a551092a45cf6f788a5a 100644 (file)
@@ -1366,10 +1366,6 @@ struct netdev_net_notifier {
  *     queue id bound to an AF_XDP socket. The flags field specifies if
  *     only RX, only Tx, or both should be woken up using the flags
  *     XDP_WAKEUP_RX and XDP_WAKEUP_TX.
- * struct devlink_port *(*ndo_get_devlink_port)(struct net_device *dev);
- *     Get devlink port instance associated with a given netdev.
- *     Called with a reference on the netdevice and devlink locks only,
- *     rtnl_lock is not held.
  * int (*ndo_tunnel_ctl)(struct net_device *dev, struct ip_tunnel_parm *p,
  *                      int cmd);
  *     Add, change, delete or get information on an IPv4 tunnel.
@@ -1600,7 +1596,6 @@ struct net_device_ops {
                                                          struct xdp_buff *xdp);
        int                     (*ndo_xsk_wakeup)(struct net_device *dev,
                                                  u32 queue_id, u32 flags);
-       struct devlink_port *   (*ndo_get_devlink_port)(struct net_device *dev);
        int                     (*ndo_tunnel_ctl)(struct net_device *dev,
                                                  struct ip_tunnel_parm *p, int cmd);
        struct net_device *     (*ndo_get_peer_dev)(struct net_device *dev);
index 158bb3f1aa0b91458b93eadb65301b3cb9f25bfb..4176482cd03f674d048e4357ada4eabbf7c708ca 100644 (file)
@@ -2165,13 +2165,6 @@ static const struct dcbnl_rtnl_ops __maybe_unused dsa_slave_dcbnl_ops = {
        .ieee_delapp            = dsa_slave_dcbnl_ieee_delapp,
 };
 
-static struct devlink_port *dsa_slave_get_devlink_port(struct net_device *dev)
-{
-       struct dsa_port *dp = dsa_slave_to_port(dev);
-
-       return &dp->devlink_port;
-}
-
 static void dsa_slave_get_stats64(struct net_device *dev,
                                  struct rtnl_link_stats64 *s)
 {
@@ -2219,7 +2212,6 @@ static const struct net_device_ops dsa_slave_netdev_ops = {
        .ndo_get_stats64        = dsa_slave_get_stats64,
        .ndo_vlan_rx_add_vid    = dsa_slave_vlan_rx_add_vid,
        .ndo_vlan_rx_kill_vid   = dsa_slave_vlan_rx_kill_vid,
-       .ndo_get_devlink_port   = dsa_slave_get_devlink_port,
        .ndo_change_mtu         = dsa_slave_change_mtu,
        .ndo_fill_forward_path  = dsa_slave_fill_forward_path,
 };