Skip to content

Commit 30beabb

Browse files
YueHaibingdavem330
authored andcommitted
net: phy: marvell: remove set but not used variable 'pause'
Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/phy/marvell.c: In function 'm88e1510_config_init': drivers/net/phy/marvell.c:850:7: warning: variable 'pause' set but not used [-Wunused-but-set-variable] It not used any more after commit 3c1bcc8 ("net: ethernet: Convert phydev advertize and supported from u32 to link mode") Signed-off-by: YueHaibing <[email protected]> Reviewed-by: Andrew Lunn <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 2b9b750 commit 30beabb

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

drivers/net/phy/marvell.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -847,7 +847,6 @@ static int m88e1510_config_init(struct phy_device *phydev)
847847

848848
/* SGMII-to-Copper mode initialization */
849849
if (phydev->interface == PHY_INTERFACE_MODE_SGMII) {
850-
u32 pause;
851850

852851
/* Select page 18 */
853852
err = marvell_set_page(phydev, 18);
@@ -878,7 +877,6 @@ static int m88e1510_config_init(struct phy_device *phydev)
878877
* This means we can never be truely sure what was advertised,
879878
* so disable Pause support.
880879
*/
881-
pause = SUPPORTED_Pause | SUPPORTED_Asym_Pause;
882880
linkmode_clear_bit(ETHTOOL_LINK_MODE_Asym_Pause_BIT,
883881
phydev->supported);
884882
linkmode_clear_bit(ETHTOOL_LINK_MODE_Pause_BIT,

0 commit comments

Comments
 (0)