diff options
| author | York Sun <yorksun@freescale.com> | 2014-10-17 13:44:06 -0700 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-10-27 17:54:09 -0400 |
| commit | 76f8cdb2036bf86ec7193ed13129797f0a32d13d (patch) | |
| tree | efcc487ebcfe8916a8b9d54707c990713010cdc3 | |
| parent | 998194584e53db13a9ed01f78c8f856e8f0bec67 (diff) | |
net/e1000: Fix EEPROM access error
Commit 951860634fdb557bbb58e0f99215391bc0c29779 may have changed
the logic unintentially from "if (!(swfw_sync & (fwmask | swmask)))"
to "if ((swfw_sync & swmask) && !(swfw_sync & fwmask))". This change
breaks some e1000 NIC with a message "ERROR: Unable to read EEPROM!".
Signed-off-by: York Sun <yorksun@freescale.com>
CC: Marek Vasut <marex@denx.de>
CC: Tim Harvey <tharvey@gateworks.com>
Acked-by: Marcel Ziswiler <marcel@ziswiler.com>
| -rw-r--r-- | drivers/net/e1000.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/e1000.c b/drivers/net/e1000.c index 6e8765cf7b..b092867272 100644 --- a/drivers/net/e1000.c +++ b/drivers/net/e1000.c @@ -1113,7 +1113,7 @@ e1000_swfw_sync_acquire(struct e1000_hw *hw, uint16_t mask) return -E1000_ERR_SWFW_SYNC; swfw_sync = E1000_READ_REG(hw, SW_FW_SYNC); - if ((swfw_sync & swmask) && !(swfw_sync & fwmask)) + if (!(swfw_sync & (fwmask | swmask))) break; /* firmware currently using resource (fwmask) */ |