mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-11-24 05:10:51 +07:00
stmmac: simple cleanups
This adds simple cleanups for stmmac, removing test we know is always true, fixing whitespace, and moving code out of if(). Signed-off-by: Pavel Machek <pavel@denx.de> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
b209af9981
commit
a77e4accce
@ -261,11 +261,11 @@ static int stmmac_ethtool_getsettings(struct net_device *dev,
|
||||
ethtool_cmd_speed_set(cmd, priv->xstats.pcs_speed);
|
||||
|
||||
/* Get and convert ADV/LP_ADV from the HW AN registers */
|
||||
if (priv->hw->mac->get_adv)
|
||||
priv->hw->mac->get_adv(priv->hw, &adv);
|
||||
else
|
||||
if (!priv->hw->mac->get_adv)
|
||||
return -EOPNOTSUPP; /* should never happen indeed */
|
||||
|
||||
priv->hw->mac->get_adv(priv->hw, &adv);
|
||||
|
||||
/* Encoding of PSE bits is defined in 802.3z, 37.2.1.4 */
|
||||
|
||||
if (adv.pause & STMMAC_PCS_PAUSE)
|
||||
@ -340,19 +340,17 @@ static int stmmac_ethtool_setsettings(struct net_device *dev,
|
||||
if (cmd->autoneg != AUTONEG_ENABLE)
|
||||
return -EINVAL;
|
||||
|
||||
if (cmd->autoneg == AUTONEG_ENABLE) {
|
||||
mask &= (ADVERTISED_1000baseT_Half |
|
||||
mask &= (ADVERTISED_1000baseT_Half |
|
||||
ADVERTISED_1000baseT_Full |
|
||||
ADVERTISED_100baseT_Half |
|
||||
ADVERTISED_100baseT_Full |
|
||||
ADVERTISED_10baseT_Half |
|
||||
ADVERTISED_10baseT_Full);
|
||||
|
||||
spin_lock(&priv->lock);
|
||||
if (priv->hw->mac->ctrl_ane)
|
||||
priv->hw->mac->ctrl_ane(priv->hw, 1);
|
||||
spin_unlock(&priv->lock);
|
||||
}
|
||||
spin_lock(&priv->lock);
|
||||
if (priv->hw->mac->ctrl_ane)
|
||||
priv->hw->mac->ctrl_ane(priv->hw, 1);
|
||||
spin_unlock(&priv->lock);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
@ -818,7 +818,7 @@ static int stmmac_init_phy(struct net_device *dev)
|
||||
/* Stop Advertising 1000BASE Capability if interface is not GMII */
|
||||
if ((interface == PHY_INTERFACE_MODE_MII) ||
|
||||
(interface == PHY_INTERFACE_MODE_RMII) ||
|
||||
(max_speed < 1000 && max_speed > 0))
|
||||
(max_speed < 1000 && max_speed > 0))
|
||||
phydev->advertising &= ~(SUPPORTED_1000baseT_Half |
|
||||
SUPPORTED_1000baseT_Full);
|
||||
|
||||
|
@ -253,7 +253,7 @@ int stmmac_mdio_register(struct net_device *ndev)
|
||||
}
|
||||
|
||||
/*
|
||||
* If we're going to bind the MAC to this PHY bus,
|
||||
* If we're going to bind the MAC to this PHY bus,
|
||||
* and no PHY number was provided to the MAC,
|
||||
* use the one probed here.
|
||||
*/
|
||||
|
Loading…
Reference in New Issue
Block a user