Skip to content

Commit

Permalink
net: phy: remove phy_ethtool_sset()
Browse files Browse the repository at this point in the history
There are no users of phy_ethtool_sset() in the kernel anymore, and
as of commit 3c1bcc8 ("net: ethernet: Convert phydev advertize
and supported from u32 to link mode"), the implementation is slightly
buggy - it doesn't correctly check the masked advertising mask as it
used to.

Remove it, and update the phy documentation to refer to its replacement
function.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Russell King authored and Jakub Kicinski committed Nov 23, 2019
1 parent 84bb46c commit e3cf8b3
Showing 3 changed files with 2 additions and 62 deletions.
3 changes: 2 additions & 1 deletion Documentation/networking/phy.rst
Original file line number Diff line number Diff line change
@@ -352,7 +352,8 @@ Fills the phydev structure with up-to-date information about the current
settings in the PHY.
::

int phy_ethtool_sset(struct phy_device *phydev, struct ethtool_cmd *cmd);
int phy_ethtool_ksettings_set(struct phy_device *phydev,
const struct ethtool_link_ksettings *cmd);

Ethtool convenience functions.
::
60 changes: 0 additions & 60 deletions drivers/net/phy/phy.c
Original file line number Diff line number Diff line change
@@ -253,66 +253,6 @@ static void phy_sanitize_settings(struct phy_device *phydev)
}
}

/**
* phy_ethtool_sset - generic ethtool sset function, handles all the details
* @phydev: target phy_device struct
* @cmd: ethtool_cmd
*
* A few notes about parameter checking:
*
* - We don't set port or transceiver, so we don't care what they
* were set to.
* - phy_start_aneg() will make sure forced settings are sane, and
* choose the next best ones from the ones selected, so we don't
* care if ethtool tries to give us bad values.
*/
int phy_ethtool_sset(struct phy_device *phydev, struct ethtool_cmd *cmd)
{
__ETHTOOL_DECLARE_LINK_MODE_MASK(advertising);
u32 speed = ethtool_cmd_speed(cmd);

if (cmd->phy_address != phydev->mdio.addr)
return -EINVAL;

/* We make sure that we don't pass unsupported values in to the PHY */
ethtool_convert_legacy_u32_to_link_mode(advertising, cmd->advertising);
linkmode_and(advertising, advertising, phydev->supported);

/* Verify the settings we care about. */
if (cmd->autoneg != AUTONEG_ENABLE && cmd->autoneg != AUTONEG_DISABLE)
return -EINVAL;

if (cmd->autoneg == AUTONEG_ENABLE && cmd->advertising == 0)
return -EINVAL;

if (cmd->autoneg == AUTONEG_DISABLE &&
((speed != SPEED_1000 &&
speed != SPEED_100 &&
speed != SPEED_10) ||
(cmd->duplex != DUPLEX_HALF &&
cmd->duplex != DUPLEX_FULL)))
return -EINVAL;

phydev->autoneg = cmd->autoneg;

phydev->speed = speed;

linkmode_copy(phydev->advertising, advertising);

linkmode_mod_bit(ETHTOOL_LINK_MODE_Autoneg_BIT,
phydev->advertising, AUTONEG_ENABLE == cmd->autoneg);

phydev->duplex = cmd->duplex;

phydev->mdix_ctrl = cmd->eth_tp_mdix_ctrl;

/* Restart the PHY */
phy_start_aneg(phydev);

return 0;
}
EXPORT_SYMBOL(phy_ethtool_sset);

int phy_ethtool_ksettings_set(struct phy_device *phydev,
const struct ethtool_link_ksettings *cmd)
{
1 change: 0 additions & 1 deletion include/linux/phy.h
Original file line number Diff line number Diff line change
@@ -1160,7 +1160,6 @@ void phy_queue_state_machine(struct phy_device *phydev, unsigned long jiffies);
void phy_mac_interrupt(struct phy_device *phydev);
void phy_start_machine(struct phy_device *phydev);
void phy_stop_machine(struct phy_device *phydev);
int phy_ethtool_sset(struct phy_device *phydev, struct ethtool_cmd *cmd);
void phy_ethtool_ksettings_get(struct phy_device *phydev,
struct ethtool_link_ksettings *cmd);
int phy_ethtool_ksettings_set(struct phy_device *phydev,

0 comments on commit e3cf8b3

Please sign in to comment.