net: e1000: Remove set but not used variable
authorZheng Yongjun <zhengyongjun3@huawei.com>
Sat, 19 Sep 2020 01:50:20 +0000 (09:50 +0800)
committerDavid S. Miller <davem@davemloft.net>
Sat, 19 Sep 2020 21:13:18 +0000 (14:13 -0700)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/net/ethernet/intel/e1000/e1000_hw.c: In function e1000_phy_init_script:
drivers/net/ethernet/intel/e1000/e1000_hw.c:132:6: warning: variable ‘ret_val’ set but not used [-Wunused-but-set-variable]

`ret_val` is never used, so remove it.

Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/intel/e1000/e1000_hw.c

index 4e7a081..f1dbd7b 100644 (file)
@@ -129,7 +129,6 @@ static s32 e1000_set_phy_type(struct e1000_hw *hw)
  */
 static void e1000_phy_init_script(struct e1000_hw *hw)
 {
-       u32 ret_val;
        u16 phy_saved_data;
 
        if (hw->phy_init_script) {
@@ -138,7 +137,7 @@ static void e1000_phy_init_script(struct e1000_hw *hw)
                /* Save off the current value of register 0x2F5B to be restored
                 * at the end of this routine.
                 */
-               ret_val = e1000_read_phy_reg(hw, 0x2F5B, &phy_saved_data);
+               e1000_read_phy_reg(hw, 0x2F5B, &phy_saved_data);
 
                /* Disabled the PHY transmitter */
                e1000_write_phy_reg(hw, 0x2F5B, 0x0003);