diff options
| author | Andrey Smirnov <andrew.smirnov@gmail.com> | 2016-06-01 21:58:36 -0700 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2016-06-03 09:14:14 +0200 |
| commit | 7477a1500a54483690e37dec3826119768f2b779 (patch) | |
| tree | 8d596debda33d527cb981ca595543d95de28d800 /drivers/net/e1000/eeprom.c | |
| parent | 3d6832758711278393ef60f4e6568c77009d59d2 (diff) | |
| download | barebox-7477a1500a54483690e37dec3826119768f2b779.tar.gz barebox-7477a1500a54483690e37dec3826119768f2b779.tar.xz | |
e1000: Remove unneeded i210 specific register code
Despite having EEC(EECD) and EERD at "non-standard" offsets i210
aliases those registers to be accesible via regular addresses so none
of the code removed by this commit is really necessary.
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/net/e1000/eeprom.c')
| -rw-r--r-- | drivers/net/e1000/eeprom.c | 25 |
1 files changed, 5 insertions, 20 deletions
diff --git a/drivers/net/e1000/eeprom.c b/drivers/net/e1000/eeprom.c index c16f69a2b7..497fb7b3f1 100644 --- a/drivers/net/e1000/eeprom.c +++ b/drivers/net/e1000/eeprom.c @@ -274,10 +274,7 @@ int32_t e1000_init_eeprom_params(struct e1000_hw *hw) int32_t ret_val = E1000_SUCCESS; uint16_t eeprom_size; - if (hw->mac_type == e1000_igb) - eecd = e1000_read_reg(hw, E1000_I210_EECD); - else - eecd = e1000_read_reg(hw, E1000_EECD); + eecd = e1000_read_reg(hw, E1000_EECD); DEBUGFUNC(); @@ -455,10 +452,7 @@ static int32_t e1000_poll_eerd_eewr_done(struct e1000_hw *hw, int eerd) for (i = 0; i < attempts; i++) { if (eerd == E1000_EEPROM_POLL_READ) { - if (hw->mac_type == e1000_igb) - reg = e1000_read_reg(hw, E1000_I210_EERD); - else - reg = e1000_read_reg(hw, E1000_EERD); + reg = e1000_read_reg(hw, E1000_EERD); } else { if (hw->mac_type == e1000_igb) reg = e1000_read_reg(hw, E1000_I210_EEWR); @@ -496,24 +490,15 @@ static int32_t e1000_read_eeprom_eerd(struct e1000_hw *hw, eerd = ((offset+i) << E1000_EEPROM_RW_ADDR_SHIFT) + E1000_EEPROM_RW_REG_START; - if (hw->mac_type == e1000_igb) - e1000_write_reg(hw, E1000_I210_EERD, eerd); - else - e1000_write_reg(hw, E1000_EERD, eerd); + e1000_write_reg(hw, E1000_EERD, eerd); error = e1000_poll_eerd_eewr_done(hw, E1000_EEPROM_POLL_READ); if (error) break; - if (hw->mac_type == e1000_igb) { - data[i] = (e1000_read_reg(hw, E1000_I210_EERD) >> - E1000_EEPROM_RW_REG_DATA); - } else { - data[i] = (e1000_read_reg(hw, E1000_EERD) >> - E1000_EEPROM_RW_REG_DATA); - } - + data[i] = (e1000_read_reg(hw, E1000_EERD) >> + E1000_EEPROM_RW_REG_DATA); } return error; |