diff options
author | Alexander Duyck <alexander.h.duyck@intel.com> | 2009-12-23 13:21:46 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-12-23 20:47:51 -0800 |
commit | 58b8b042509f53955ba660a4245e221c5405f124 (patch) | |
tree | d5223270808511fb8bf7a478f87407114cd6a638 | |
parent | d405ea3ef2499eb46834418a643973c884ff7f30 (diff) | |
download | op-kernel-dev-58b8b042509f53955ba660a4245e221c5405f124.zip op-kernel-dev-58b8b042509f53955ba660a4245e221c5405f124.tar.gz |
igb: check both function bits in status register in wol exception
The ethtool code for enabling Wake on Lan was not correctly checking the
status register bits so as a result ports 0 and 2 were both being allowed
to set WOL to enabled even though it is only supported on the first port
for our adapters.
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/igb/igb_ethtool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/igb/igb_ethtool.c b/drivers/net/igb/igb_ethtool.c index ac9d527..f771a6c 100644 --- a/drivers/net/igb/igb_ethtool.c +++ b/drivers/net/igb/igb_ethtool.c @@ -1795,7 +1795,7 @@ static int igb_wol_exclusion(struct igb_adapter *adapter, /* dual port cards only support WoL on port A from now on * unless it was enabled in the eeprom for port B * so exclude FUNC_1 ports from having WoL enabled */ - if (rd32(E1000_STATUS) & E1000_STATUS_FUNC_1 && + if ((rd32(E1000_STATUS) & E1000_STATUS_FUNC_MASK) && !adapter->eeprom_wol) { wol->supported = 0; break; |