EEPROM_INIT_CONTROL2_REG is already read once before entering this switch statement, so there's not much use in reading its value for the second time. Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> --- drivers/net/e1000/main.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/drivers/net/e1000/main.c b/drivers/net/e1000/main.c index 66ec72f..84459da 100644 --- a/drivers/net/e1000/main.c +++ b/drivers/net/e1000/main.c @@ -840,12 +840,6 @@ static int e1000_setup_link(struct e1000_hw *hw) hw->fc = e1000_fc_full; break; default: - ret_val = e1000_read_eeprom(hw, EEPROM_INIT_CONTROL2_REG, 1, &eeprom_data); - if (ret_val) { - dev_dbg(hw->dev, "EEPROM Read Error\n"); - return -E1000_ERR_EEPROM; - } - if ((eeprom_data & EEPROM_WORD0F_PAUSE_MASK) == 0) hw->fc = e1000_fc_none; else if ((eeprom_data & EEPROM_WORD0F_PAUSE_MASK) == EEPROM_WORD0F_ASM_DIR) -- 2.5.0 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox