net: marvell: sky2: use new api ethtool_{get|set}_link_ksettings

The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.

Signed-off-by: Philippe Reynes <tremyfr@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Philippe Reynes 2017-01-14 23:26:22 +01:00 committed by David S. Miller
parent 0f826385a6
commit 55f78fcdd8

View file

@ -3589,47 +3589,59 @@ static u32 sky2_supported_modes(const struct sky2_hw *hw)
| SUPPORTED_1000baseT_Full; | SUPPORTED_1000baseT_Full;
} }
static int sky2_get_settings(struct net_device *dev, struct ethtool_cmd *ecmd) static int sky2_get_link_ksettings(struct net_device *dev,
struct ethtool_link_ksettings *cmd)
{ {
struct sky2_port *sky2 = netdev_priv(dev); struct sky2_port *sky2 = netdev_priv(dev);
struct sky2_hw *hw = sky2->hw; struct sky2_hw *hw = sky2->hw;
u32 supported, advertising;
ecmd->transceiver = XCVR_INTERNAL; supported = sky2_supported_modes(hw);
ecmd->supported = sky2_supported_modes(hw); cmd->base.phy_address = PHY_ADDR_MARV;
ecmd->phy_address = PHY_ADDR_MARV;
if (sky2_is_copper(hw)) { if (sky2_is_copper(hw)) {
ecmd->port = PORT_TP; cmd->base.port = PORT_TP;
ethtool_cmd_speed_set(ecmd, sky2->speed); cmd->base.speed = sky2->speed;
ecmd->supported |= SUPPORTED_Autoneg | SUPPORTED_TP; supported |= SUPPORTED_Autoneg | SUPPORTED_TP;
} else { } else {
ethtool_cmd_speed_set(ecmd, SPEED_1000); cmd->base.speed = SPEED_1000;
ecmd->port = PORT_FIBRE; cmd->base.port = PORT_FIBRE;
ecmd->supported |= SUPPORTED_Autoneg | SUPPORTED_FIBRE; supported |= SUPPORTED_Autoneg | SUPPORTED_FIBRE;
} }
ecmd->advertising = sky2->advertising; advertising = sky2->advertising;
ecmd->autoneg = (sky2->flags & SKY2_FLAG_AUTO_SPEED) cmd->base.autoneg = (sky2->flags & SKY2_FLAG_AUTO_SPEED)
? AUTONEG_ENABLE : AUTONEG_DISABLE; ? AUTONEG_ENABLE : AUTONEG_DISABLE;
ecmd->duplex = sky2->duplex; cmd->base.duplex = sky2->duplex;
ethtool_convert_legacy_u32_to_link_mode(cmd->link_modes.supported,
supported);
ethtool_convert_legacy_u32_to_link_mode(cmd->link_modes.advertising,
advertising);
return 0; return 0;
} }
static int sky2_set_settings(struct net_device *dev, struct ethtool_cmd *ecmd) static int sky2_set_link_ksettings(struct net_device *dev,
const struct ethtool_link_ksettings *cmd)
{ {
struct sky2_port *sky2 = netdev_priv(dev); struct sky2_port *sky2 = netdev_priv(dev);
const struct sky2_hw *hw = sky2->hw; const struct sky2_hw *hw = sky2->hw;
u32 supported = sky2_supported_modes(hw); u32 supported = sky2_supported_modes(hw);
u32 new_advertising;
if (ecmd->autoneg == AUTONEG_ENABLE) { ethtool_convert_link_mode_to_legacy_u32(&new_advertising,
if (ecmd->advertising & ~supported) cmd->link_modes.advertising);
if (cmd->base.autoneg == AUTONEG_ENABLE) {
if (new_advertising & ~supported)
return -EINVAL; return -EINVAL;
if (sky2_is_copper(hw)) if (sky2_is_copper(hw))
sky2->advertising = ecmd->advertising | sky2->advertising = new_advertising |
ADVERTISED_TP | ADVERTISED_TP |
ADVERTISED_Autoneg; ADVERTISED_Autoneg;
else else
sky2->advertising = ecmd->advertising | sky2->advertising = new_advertising |
ADVERTISED_FIBRE | ADVERTISED_FIBRE |
ADVERTISED_Autoneg; ADVERTISED_Autoneg;
@ -3638,30 +3650,30 @@ static int sky2_set_settings(struct net_device *dev, struct ethtool_cmd *ecmd)
sky2->speed = -1; sky2->speed = -1;
} else { } else {
u32 setting; u32 setting;
u32 speed = ethtool_cmd_speed(ecmd); u32 speed = cmd->base.speed;
switch (speed) { switch (speed) {
case SPEED_1000: case SPEED_1000:
if (ecmd->duplex == DUPLEX_FULL) if (cmd->base.duplex == DUPLEX_FULL)
setting = SUPPORTED_1000baseT_Full; setting = SUPPORTED_1000baseT_Full;
else if (ecmd->duplex == DUPLEX_HALF) else if (cmd->base.duplex == DUPLEX_HALF)
setting = SUPPORTED_1000baseT_Half; setting = SUPPORTED_1000baseT_Half;
else else
return -EINVAL; return -EINVAL;
break; break;
case SPEED_100: case SPEED_100:
if (ecmd->duplex == DUPLEX_FULL) if (cmd->base.duplex == DUPLEX_FULL)
setting = SUPPORTED_100baseT_Full; setting = SUPPORTED_100baseT_Full;
else if (ecmd->duplex == DUPLEX_HALF) else if (cmd->base.duplex == DUPLEX_HALF)
setting = SUPPORTED_100baseT_Half; setting = SUPPORTED_100baseT_Half;
else else
return -EINVAL; return -EINVAL;
break; break;
case SPEED_10: case SPEED_10:
if (ecmd->duplex == DUPLEX_FULL) if (cmd->base.duplex == DUPLEX_FULL)
setting = SUPPORTED_10baseT_Full; setting = SUPPORTED_10baseT_Full;
else if (ecmd->duplex == DUPLEX_HALF) else if (cmd->base.duplex == DUPLEX_HALF)
setting = SUPPORTED_10baseT_Half; setting = SUPPORTED_10baseT_Half;
else else
return -EINVAL; return -EINVAL;
@ -3674,7 +3686,7 @@ static int sky2_set_settings(struct net_device *dev, struct ethtool_cmd *ecmd)
return -EINVAL; return -EINVAL;
sky2->speed = speed; sky2->speed = speed;
sky2->duplex = ecmd->duplex; sky2->duplex = cmd->base.duplex;
sky2->flags &= ~SKY2_FLAG_AUTO_SPEED; sky2->flags &= ~SKY2_FLAG_AUTO_SPEED;
} }
@ -4405,8 +4417,6 @@ static int sky2_set_features(struct net_device *dev, netdev_features_t features)
} }
static const struct ethtool_ops sky2_ethtool_ops = { static const struct ethtool_ops sky2_ethtool_ops = {
.get_settings = sky2_get_settings,
.set_settings = sky2_set_settings,
.get_drvinfo = sky2_get_drvinfo, .get_drvinfo = sky2_get_drvinfo,
.get_wol = sky2_get_wol, .get_wol = sky2_get_wol,
.set_wol = sky2_set_wol, .set_wol = sky2_set_wol,
@ -4429,6 +4439,8 @@ static const struct ethtool_ops sky2_ethtool_ops = {
.set_phys_id = sky2_set_phys_id, .set_phys_id = sky2_set_phys_id,
.get_sset_count = sky2_get_sset_count, .get_sset_count = sky2_get_sset_count,
.get_ethtool_stats = sky2_get_ethtool_stats, .get_ethtool_stats = sky2_get_ethtool_stats,
.get_link_ksettings = sky2_get_link_ksettings,
.set_link_ksettings = sky2_set_link_ksettings,
}; };
#ifdef CONFIG_SKY2_DEBUG #ifdef CONFIG_SKY2_DEBUG