net: mlx4: use new ETHTOOL_G/SSETTINGS API
authorDavid Decotigny <decot@googlers.com>
Wed, 24 Feb 2016 18:58:12 +0000 (10:58 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Feb 2016 03:06:47 +0000 (22:06 -0500)
Signed-off-by: David Decotigny <decot@googlers.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/mellanox/mlx4/en_ethtool.c
drivers/net/ethernet/mellanox/mlx4/en_main.c
drivers/net/ethernet/mellanox/mlx4/mlx4_en.h

index dd84cab..f69584a 100644 (file)
@@ -501,34 +501,30 @@ static u32 mlx4_en_autoneg_get(struct net_device *dev)
        return autoneg;
 }
 
-static u32 ptys_get_supported_port(struct mlx4_ptys_reg *ptys_reg)
+static void ptys2ethtool_update_supported_port(unsigned long *mask,
+                                              struct mlx4_ptys_reg *ptys_reg)
 {
        u32 eth_proto = be32_to_cpu(ptys_reg->eth_proto_cap);
 
        if (eth_proto & (MLX4_PROT_MASK(MLX4_10GBASE_T)
                         | MLX4_PROT_MASK(MLX4_1000BASE_T)
                         | MLX4_PROT_MASK(MLX4_100BASE_TX))) {
-                       return SUPPORTED_TP;
-       }
-
-       if (eth_proto & (MLX4_PROT_MASK(MLX4_10GBASE_CR)
+               __set_bit(ETHTOOL_LINK_MODE_TP_BIT, mask);
+       } else if (eth_proto & (MLX4_PROT_MASK(MLX4_10GBASE_CR)
                         | MLX4_PROT_MASK(MLX4_10GBASE_SR)
                         | MLX4_PROT_MASK(MLX4_56GBASE_SR4)
                         | MLX4_PROT_MASK(MLX4_40GBASE_CR4)
                         | MLX4_PROT_MASK(MLX4_40GBASE_SR4)
                         | MLX4_PROT_MASK(MLX4_1000BASE_CX_SGMII))) {
-                       return SUPPORTED_FIBRE;
-       }
-
-       if (eth_proto & (MLX4_PROT_MASK(MLX4_56GBASE_KR4)
+               __set_bit(ETHTOOL_LINK_MODE_FIBRE_BIT, mask);
+       } else if (eth_proto & (MLX4_PROT_MASK(MLX4_56GBASE_KR4)
                         | MLX4_PROT_MASK(MLX4_40GBASE_KR4)
                         | MLX4_PROT_MASK(MLX4_20GBASE_KR2)
                         | MLX4_PROT_MASK(MLX4_10GBASE_KR)
                         | MLX4_PROT_MASK(MLX4_10GBASE_KX4)
                         | MLX4_PROT_MASK(MLX4_1000BASE_KX))) {
-                       return SUPPORTED_Backplane;
+               __set_bit(ETHTOOL_LINK_MODE_Backplane_BIT, mask);
        }
-       return 0;
 }
 
 static u32 ptys_get_active_port(struct mlx4_ptys_reg *ptys_reg)
@@ -574,122 +570,111 @@ static u32 ptys_get_active_port(struct mlx4_ptys_reg *ptys_reg)
 enum ethtool_report {
        SUPPORTED = 0,
        ADVERTISED = 1,
-       SPEED = 2
 };
 
+struct ptys2ethtool_config {
+       __ETHTOOL_DECLARE_LINK_MODE_MASK(supported);
+       __ETHTOOL_DECLARE_LINK_MODE_MASK(advertised);
+       u32 speed;
+};
+
+static unsigned long *ptys2ethtool_link_mode(struct ptys2ethtool_config *cfg,
+                                            enum ethtool_report report)
+{
+       switch (report) {
+       case SUPPORTED:
+               return cfg->supported;
+       case ADVERTISED:
+               return cfg->advertised;
+       }
+       return NULL;
+}
+
+#define MLX4_BUILD_PTYS2ETHTOOL_CONFIG(reg_, speed_, ...)              \
+       ({                                                              \
+               struct ptys2ethtool_config *cfg;                        \
+               const unsigned int modes[] = { __VA_ARGS__ };           \
+               unsigned int i;                                         \
+               cfg = &ptys2ethtool_map[reg_];                          \
+               cfg->speed = speed_;                                    \
+               bitmap_zero(cfg->supported,                             \
+                           __ETHTOOL_LINK_MODE_MASK_NBITS);            \
+               bitmap_zero(cfg->advertised,                            \
+                           __ETHTOOL_LINK_MODE_MASK_NBITS);            \
+               for (i = 0 ; i < ARRAY_SIZE(modes) ; ++i) {             \
+                       __set_bit(modes[i], cfg->supported);            \
+                       __set_bit(modes[i], cfg->advertised);           \
+               }                                                       \
+       })
+
 /* Translates mlx4 link mode to equivalent ethtool Link modes/speed */
-static u32 ptys2ethtool_map[MLX4_LINK_MODES_SZ][3] = {
-       [MLX4_100BASE_TX] = {
-               SUPPORTED_100baseT_Full,
-               ADVERTISED_100baseT_Full,
-               SPEED_100
-               },
-
-       [MLX4_1000BASE_T] = {
-               SUPPORTED_1000baseT_Full,
-               ADVERTISED_1000baseT_Full,
-               SPEED_1000
-               },
-       [MLX4_1000BASE_CX_SGMII] = {
-               SUPPORTED_1000baseKX_Full,
-               ADVERTISED_1000baseKX_Full,
-               SPEED_1000
-               },
-       [MLX4_1000BASE_KX] = {
-               SUPPORTED_1000baseKX_Full,
-               ADVERTISED_1000baseKX_Full,
-               SPEED_1000
-               },
-
-       [MLX4_10GBASE_T] = {
-               SUPPORTED_10000baseT_Full,
-               ADVERTISED_10000baseT_Full,
-               SPEED_10000
-               },
-       [MLX4_10GBASE_CX4] = {
-               SUPPORTED_10000baseKX4_Full,
-               ADVERTISED_10000baseKX4_Full,
-               SPEED_10000
-               },
-       [MLX4_10GBASE_KX4] = {
-               SUPPORTED_10000baseKX4_Full,
-               ADVERTISED_10000baseKX4_Full,
-               SPEED_10000
-               },
-       [MLX4_10GBASE_KR] = {
-               SUPPORTED_10000baseKR_Full,
-               ADVERTISED_10000baseKR_Full,
-               SPEED_10000
-               },
-       [MLX4_10GBASE_CR] = {
-               SUPPORTED_10000baseKR_Full,
-               ADVERTISED_10000baseKR_Full,
-               SPEED_10000
-               },
-       [MLX4_10GBASE_SR] = {
-               SUPPORTED_10000baseKR_Full,
-               ADVERTISED_10000baseKR_Full,
-               SPEED_10000
-               },
-
-       [MLX4_20GBASE_KR2] = {
-               SUPPORTED_20000baseMLD2_Full | SUPPORTED_20000baseKR2_Full,
-               ADVERTISED_20000baseMLD2_Full | ADVERTISED_20000baseKR2_Full,
-               SPEED_20000
-               },
-
-       [MLX4_40GBASE_CR4] = {
-               SUPPORTED_40000baseCR4_Full,
-               ADVERTISED_40000baseCR4_Full,
-               SPEED_40000
-               },
-       [MLX4_40GBASE_KR4] = {
-               SUPPORTED_40000baseKR4_Full,
-               ADVERTISED_40000baseKR4_Full,
-               SPEED_40000
-               },
-       [MLX4_40GBASE_SR4] = {
-               SUPPORTED_40000baseSR4_Full,
-               ADVERTISED_40000baseSR4_Full,
-               SPEED_40000
-               },
-
-       [MLX4_56GBASE_KR4] = {
-               SUPPORTED_56000baseKR4_Full,
-               ADVERTISED_56000baseKR4_Full,
-               SPEED_56000
-               },
-       [MLX4_56GBASE_CR4] = {
-               SUPPORTED_56000baseCR4_Full,
-               ADVERTISED_56000baseCR4_Full,
-               SPEED_56000
-               },
-       [MLX4_56GBASE_SR4] = {
-               SUPPORTED_56000baseSR4_Full,
-               ADVERTISED_56000baseSR4_Full,
-               SPEED_56000
-               },
+static struct ptys2ethtool_config ptys2ethtool_map[MLX4_LINK_MODES_SZ];
+
+void __init mlx4_en_init_ptys2ethtool_map(void)
+{
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_100BASE_TX, SPEED_100,
+                                      ETHTOOL_LINK_MODE_100baseT_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_1000BASE_T, SPEED_1000,
+                                      ETHTOOL_LINK_MODE_1000baseT_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_1000BASE_CX_SGMII, SPEED_1000,
+                                      ETHTOOL_LINK_MODE_1000baseKX_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_1000BASE_KX, SPEED_1000,
+                                      ETHTOOL_LINK_MODE_1000baseKX_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_T, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseT_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_CX4, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseKX4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_KX4, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseKX4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_KR, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseKR_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_CR, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseKR_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_10GBASE_SR, SPEED_10000,
+                                      ETHTOOL_LINK_MODE_10000baseKR_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_20GBASE_KR2, SPEED_20000,
+                                      ETHTOOL_LINK_MODE_20000baseMLD2_Full_BIT,
+                                      ETHTOOL_LINK_MODE_20000baseKR2_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_40GBASE_CR4, SPEED_40000,
+                                      ETHTOOL_LINK_MODE_40000baseCR4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_40GBASE_KR4, SPEED_40000,
+                                      ETHTOOL_LINK_MODE_40000baseKR4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_40GBASE_SR4, SPEED_40000,
+                                      ETHTOOL_LINK_MODE_40000baseSR4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_56GBASE_KR4, SPEED_56000,
+                                      ETHTOOL_LINK_MODE_56000baseKR4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_56GBASE_CR4, SPEED_56000,
+                                      ETHTOOL_LINK_MODE_56000baseCR4_Full_BIT);
+       MLX4_BUILD_PTYS2ETHTOOL_CONFIG(MLX4_56GBASE_SR4, SPEED_56000,
+                                      ETHTOOL_LINK_MODE_56000baseSR4_Full_BIT);
 };
 
-static u32 ptys2ethtool_link_modes(u32 eth_proto, enum ethtool_report report)
+static void ptys2ethtool_update_link_modes(unsigned long *link_modes,
+                                          u32 eth_proto,
+                                          enum ethtool_report report)
 {
        int i;
-       u32 link_modes = 0;
-
        for (i = 0; i < MLX4_LINK_MODES_SZ; i++) {
                if (eth_proto & MLX4_PROT_MASK(i))
-                       link_modes |= ptys2ethtool_map[i][report];
+                       bitmap_or(link_modes, link_modes,
+                                 ptys2ethtool_link_mode(&ptys2ethtool_map[i],
+                                                        report),
+                                 __ETHTOOL_LINK_MODE_MASK_NBITS);
        }
-       return link_modes;
 }
 
-static u32 ethtool2ptys_link_modes(u32 link_modes, enum ethtool_report report)
+static u32 ethtool2ptys_link_modes(const unsigned long *link_modes,
+                                  enum ethtool_report report)
 {
        int i;
        u32 ptys_modes = 0;
 
        for (i = 0; i < MLX4_LINK_MODES_SZ; i++) {
-               if (ptys2ethtool_map[i][report] & link_modes)
+               if (bitmap_intersects(
+                           ptys2ethtool_link_mode(&ptys2ethtool_map[i],
+                                                  report),
+                           link_modes,
+                           __ETHTOOL_LINK_MODE_MASK_NBITS))
                        ptys_modes |= 1 << i;
        }
        return ptys_modes;
@@ -702,14 +687,15 @@ static u32 speed2ptys_link_modes(u32 speed)
        u32 ptys_modes = 0;
 
        for (i = 0; i < MLX4_LINK_MODES_SZ; i++) {
-               if (ptys2ethtool_map[i][SPEED] == speed)
+               if (ptys2ethtool_map[i].speed == speed)
                        ptys_modes |= 1 << i;
        }
        return ptys_modes;
 }
 
-static int ethtool_get_ptys_settings(struct net_device *dev,
-                                    struct ethtool_cmd *cmd)
+static int
+ethtool_get_ptys_link_ksettings(struct net_device *dev,
+                               struct ethtool_link_ksettings *link_ksettings)
 {
        struct mlx4_en_priv *priv = netdev_priv(dev);
        struct mlx4_ptys_reg ptys_reg;
@@ -737,79 +723,102 @@ static int ethtool_get_ptys_settings(struct net_device *dev,
        en_dbg(DRV, priv, "ptys_reg.eth_proto_lp_adv %x\n",
               be32_to_cpu(ptys_reg.eth_proto_lp_adv));
 
-       cmd->supported = 0;
-       cmd->advertising = 0;
+       /* reset supported/advertising masks */
+       ethtool_link_ksettings_zero_link_mode(link_ksettings, supported);
+       ethtool_link_ksettings_zero_link_mode(link_ksettings, advertising);
 
-       cmd->supported |= ptys_get_supported_port(&ptys_reg);
+       ptys2ethtool_update_supported_port(link_ksettings->link_modes.supported,
+                                          &ptys_reg);
 
        eth_proto = be32_to_cpu(ptys_reg.eth_proto_cap);
-       cmd->supported |= ptys2ethtool_link_modes(eth_proto, SUPPORTED);
+       ptys2ethtool_update_link_modes(link_ksettings->link_modes.supported,
+                                      eth_proto, SUPPORTED);
 
        eth_proto = be32_to_cpu(ptys_reg.eth_proto_admin);
-       cmd->advertising |= ptys2ethtool_link_modes(eth_proto, ADVERTISED);
+       ptys2ethtool_update_link_modes(link_ksettings->link_modes.advertising,
+                                      eth_proto, ADVERTISED);
 
-       cmd->supported |= SUPPORTED_Pause | SUPPORTED_Asym_Pause;
-       cmd->advertising |= (priv->prof->tx_pause) ? ADVERTISED_Pause : 0;
+       ethtool_link_ksettings_add_link_mode(link_ksettings, supported,
+                                            Pause);
+       ethtool_link_ksettings_add_link_mode(link_ksettings, supported,
+                                            Asym_Pause);
 
-       cmd->advertising |= (priv->prof->tx_pause ^ priv->prof->rx_pause) ?
-               ADVERTISED_Asym_Pause : 0;
+       if (priv->prof->tx_pause)
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    advertising, Pause);
+       if (priv->prof->tx_pause ^ priv->prof->rx_pause)
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    advertising, Asym_Pause);
 
-       cmd->port = ptys_get_active_port(&ptys_reg);
-       cmd->transceiver = (SUPPORTED_TP & cmd->supported) ?
-               XCVR_EXTERNAL : XCVR_INTERNAL;
+       link_ksettings->base.port = ptys_get_active_port(&ptys_reg);
 
        if (mlx4_en_autoneg_get(dev)) {
-               cmd->supported |= SUPPORTED_Autoneg;
-               cmd->advertising |= ADVERTISED_Autoneg;
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    supported, Autoneg);
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    advertising, Autoneg);
        }
 
-       cmd->autoneg = (priv->port_state.flags & MLX4_EN_PORT_ANC) ?
+       link_ksettings->base.autoneg
+               = (priv->port_state.flags & MLX4_EN_PORT_ANC) ?
                AUTONEG_ENABLE : AUTONEG_DISABLE;
 
        eth_proto = be32_to_cpu(ptys_reg.eth_proto_lp_adv);
-       cmd->lp_advertising = ptys2ethtool_link_modes(eth_proto, ADVERTISED);
 
-       cmd->lp_advertising |= (priv->port_state.flags & MLX4_EN_PORT_ANC) ?
-                       ADVERTISED_Autoneg : 0;
+       ethtool_link_ksettings_zero_link_mode(link_ksettings, lp_advertising);
+       ptys2ethtool_update_link_modes(
+               link_ksettings->link_modes.lp_advertising,
+               eth_proto, ADVERTISED);
+       if (priv->port_state.flags & MLX4_EN_PORT_ANC)
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    lp_advertising, Autoneg);
 
-       cmd->phy_address = 0;
-       cmd->mdio_support = 0;
-       cmd->maxtxpkt = 0;
-       cmd->maxrxpkt = 0;
-       cmd->eth_tp_mdix = ETH_TP_MDI_INVALID;
-       cmd->eth_tp_mdix_ctrl = ETH_TP_MDI_AUTO;
+       link_ksettings->base.phy_address = 0;
+       link_ksettings->base.mdio_support = 0;
+       link_ksettings->base.eth_tp_mdix = ETH_TP_MDI_INVALID;
+       link_ksettings->base.eth_tp_mdix_ctrl = ETH_TP_MDI_AUTO;
 
        return ret;
 }
 
-static void ethtool_get_default_settings(struct net_device *dev,
-                                        struct ethtool_cmd *cmd)
+static void
+ethtool_get_default_link_ksettings(
+       struct net_device *dev, struct ethtool_link_ksettings *link_ksettings)
 {
        struct mlx4_en_priv *priv = netdev_priv(dev);
        int trans_type;
 
-       cmd->autoneg = AUTONEG_DISABLE;
-       cmd->supported = SUPPORTED_10000baseT_Full;
-       cmd->advertising = ADVERTISED_10000baseT_Full;
-       trans_type = priv->port_state.transceiver;
+       link_ksettings->base.autoneg = AUTONEG_DISABLE;
+
+       ethtool_link_ksettings_zero_link_mode(link_ksettings, supported);
+       ethtool_link_ksettings_add_link_mode(link_ksettings, supported,
+                                            10000baseT_Full);
 
+       ethtool_link_ksettings_zero_link_mode(link_ksettings, advertising);
+       ethtool_link_ksettings_add_link_mode(link_ksettings, advertising,
+                                            10000baseT_Full);
+
+       trans_type = priv->port_state.transceiver;
        if (trans_type > 0 && trans_type <= 0xC) {
-               cmd->port = PORT_FIBRE;
-               cmd->transceiver = XCVR_EXTERNAL;
-               cmd->supported |= SUPPORTED_FIBRE;
-               cmd->advertising |= ADVERTISED_FIBRE;
+               link_ksettings->base.port = PORT_FIBRE;
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    supported, FIBRE);
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    advertising, FIBRE);
        } else if (trans_type == 0x80 || trans_type == 0) {
-               cmd->port = PORT_TP;
-               cmd->transceiver = XCVR_INTERNAL;
-               cmd->supported |= SUPPORTED_TP;
-               cmd->advertising |= ADVERTISED_TP;
+               link_ksettings->base.port = PORT_TP;
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    supported, TP);
+               ethtool_link_ksettings_add_link_mode(link_ksettings,
+                                                    advertising, TP);
        } else  {
-               cmd->port = -1;
-               cmd->transceiver = -1;
+               link_ksettings->base.port = -1;
        }
 }
 
-static int mlx4_en_get_settings(struct net_device *dev, struct ethtool_cmd *cmd)
+static int
+mlx4_en_get_link_ksettings(struct net_device *dev,
+                          struct ethtool_link_ksettings *link_ksettings)
 {
        struct mlx4_en_priv *priv = netdev_priv(dev);
        int ret = -EINVAL;
@@ -822,16 +831,16 @@ static int mlx4_en_get_settings(struct net_device *dev, struct ethtool_cmd *cmd)
               priv->port_state.flags & MLX4_EN_PORT_ANE);
 
        if (priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_ETH_PROT_CTRL)
-               ret = ethtool_get_ptys_settings(dev, cmd);
+               ret = ethtool_get_ptys_link_ksettings(dev, link_ksettings);
        if (ret) /* ETH PROT CRTL is not supported or PTYS CMD failed */
-               ethtool_get_default_settings(dev, cmd);
+               ethtool_get_default_link_ksettings(dev, link_ksettings);
 
        if (netif_carrier_ok(dev)) {
-               ethtool_cmd_speed_set(cmd, priv->port_state.link_speed);
-               cmd->duplex = DUPLEX_FULL;
+               link_ksettings->base.speed = priv->port_state.link_speed;
+               link_ksettings->base.duplex = DUPLEX_FULL;
        } else {
-               ethtool_cmd_speed_set(cmd, SPEED_UNKNOWN);
-               cmd->duplex = DUPLEX_UNKNOWN;
+               link_ksettings->base.speed = SPEED_UNKNOWN;
+               link_ksettings->base.duplex = DUPLEX_UNKNOWN;
        }
        return 0;
 }
@@ -855,21 +864,29 @@ static __be32 speed_set_ptys_admin(struct mlx4_en_priv *priv, u32 speed,
        return proto_admin;
 }
 
-static int mlx4_en_set_settings(struct net_device *dev, struct ethtool_cmd *cmd)
+static int
+mlx4_en_set_link_ksettings(struct net_device *dev,
+                          const struct ethtool_link_ksettings *link_ksettings)
 {
        struct mlx4_en_priv *priv = netdev_priv(dev);
        struct mlx4_ptys_reg ptys_reg;
        __be32 proto_admin;
        int ret;
 
-       u32 ptys_adv = ethtool2ptys_link_modes(cmd->advertising, ADVERTISED);
-       int speed = ethtool_cmd_speed(cmd);
+       u32 ptys_adv = ethtool2ptys_link_modes(
+               link_ksettings->link_modes.advertising, ADVERTISED);
+       const int speed = link_ksettings->base.speed;
 
-       en_dbg(DRV, priv, "Set Speed=%d adv=0x%x autoneg=%d duplex=%d\n",
-              speed, cmd->advertising, cmd->autoneg, cmd->duplex);
+       en_dbg(DRV, priv,
+              "Set Speed=%d adv={%*pbl} autoneg=%d duplex=%d\n",
+              speed, __ETHTOOL_LINK_MODE_MASK_NBITS,
+              link_ksettings->link_modes.advertising,
+              link_ksettings->base.autoneg,
+              link_ksettings->base.duplex);
 
-       if (!(priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_ETH_PROT_CTRL) ||
-           (cmd->duplex == DUPLEX_HALF))
+       if (!(priv->mdev->dev->caps.flags2 &
+             MLX4_DEV_CAP_FLAG2_ETH_PROT_CTRL) ||
+           (link_ksettings->base.duplex == DUPLEX_HALF))
                return -EINVAL;
 
        memset(&ptys_reg, 0, sizeof(ptys_reg));
@@ -883,7 +900,7 @@ static int mlx4_en_set_settings(struct net_device *dev, struct ethtool_cmd *cmd)
                return 0;
        }
 
-       proto_admin = cmd->autoneg == AUTONEG_ENABLE ?
+       proto_admin = link_ksettings->base.autoneg == AUTONEG_ENABLE ?
                cpu_to_be32(ptys_adv) :
                speed_set_ptys_admin(priv, speed,
                                     ptys_reg.eth_proto_cap);
@@ -1982,8 +1999,8 @@ static int mlx4_en_set_phys_id(struct net_device *dev,
 
 const struct ethtool_ops mlx4_en_ethtool_ops = {
        .get_drvinfo = mlx4_en_get_drvinfo,
-       .get_settings = mlx4_en_get_settings,
-       .set_settings = mlx4_en_set_settings,
+       .get_link_ksettings = mlx4_en_get_link_ksettings,
+       .set_link_ksettings = mlx4_en_set_link_ksettings,
        .get_link = ethtool_op_get_link,
        .get_strings = mlx4_en_get_strings,
        .get_sset_count = mlx4_en_get_sset_count,
index e0ec280..bf7628d 100644 (file)
@@ -382,6 +382,7 @@ static void mlx4_en_verify_params(void)
 static int __init mlx4_en_init(void)
 {
        mlx4_en_verify_params();
+       mlx4_en_init_ptys2ethtool_map();
 
        return mlx4_register_interface(&mlx4_en_interface);
 }
index 35de7d2..d12ab6a 100644 (file)
@@ -607,6 +607,7 @@ static inline struct mlx4_cqe *mlx4_en_get_cqe(void *buf, int idx, int cqe_sz)
 
 #define MLX4_EN_WOL_DO_MODIFY (1ULL << 63)
 
+void mlx4_en_init_ptys2ethtool_map(void);
 void mlx4_en_update_loopback_state(struct net_device *dev,
                                   netdev_features_t features);