]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
net: phy: marvell: remove set but not used variable 'pause'
authorYueHaibing <yuehaibing@huawei.com>
Mon, 12 Nov 2018 01:47:30 +0000 (01:47 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 12 Nov 2018 02:19:50 +0000 (18:19 -0800)
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 3c1bcc8614db ("net: ethernet: Convert phydev
advertize and supported from u32 to link mode")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/marvell.c

index 36a0db86c6f47122c885e689402411e5c4af0229..6a9881942e531a3af52ce637c6c043b3cae1d35c 100644 (file)
@@ -847,7 +847,6 @@ static int m88e1510_config_init(struct phy_device *phydev)
 
        /* SGMII-to-Copper mode initialization */
        if (phydev->interface == PHY_INTERFACE_MODE_SGMII) {
-               u32 pause;
 
                /* Select page 18 */
                err = marvell_set_page(phydev, 18);
@@ -878,7 +877,6 @@ static int m88e1510_config_init(struct phy_device *phydev)
                 * This means we can never be truely sure what was advertised,
                 * so disable Pause support.
                 */
-               pause = SUPPORTED_Pause | SUPPORTED_Asym_Pause;
                linkmode_clear_bit(ETHTOOL_LINK_MODE_Asym_Pause_BIT,
                                   phydev->supported);
                linkmode_clear_bit(ETHTOOL_LINK_MODE_Pause_BIT,