leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

net: e1000: Remove set but not used variable

Browse Source 
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>
This commit is contained in:
Zheng Yongjun
2020-09-19 09:50:20 +08:00
committed by David S. Miller
parent 47dae52b00
commit f9d80a09be
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
drivers/net/ethernet/intel/e1000/e1000_hw.c
View 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) static void e1000_phy_init_script(struct e1000_hw *hw)
{ {
u32 ret_val;
u16 phy_saved_data; u16 phy_saved_data;
if (hw->phy_init_script) { 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 /* Save off the current value of register 0x2F5B to be restored
* at the end of this routine. * 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 */ /* Disabled the PHY transmitter */
e1000_write_phy_reg(hw, 0x2F5B, 0x0003); e1000_write_phy_reg(hw, 0x2F5B, 0x0003);