e1000e: fix incorrect link status when switch module pulled
authorBruce Allan <bruce.w.allan@intel.com>
Sat, 22 Nov 2008 01:01:35 +0000 (17:01 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sat, 22 Nov 2008 01:01:35 +0000 (17:01 -0800)
On 82571 with SerDes, the true link state is not always correct when read
from the STATUS register; use existing e1000_has_link() function instead.

Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/e1000e/e1000.h
drivers/net/e1000e/ethtool.c
drivers/net/e1000e/netdev.c

index 473c41479ece464c9bcffa540bc13552520548ee..37bcb190eef80856c89caa08e3e93df1380b38e7 100644 (file)
@@ -389,6 +389,7 @@ extern int e1000e_setup_tx_resources(struct e1000_adapter *adapter);
 extern void e1000e_free_rx_resources(struct e1000_adapter *adapter);
 extern void e1000e_free_tx_resources(struct e1000_adapter *adapter);
 extern void e1000e_update_stats(struct e1000_adapter *adapter);
+extern bool e1000_has_link(struct e1000_adapter *adapter);
 extern void e1000e_set_interrupt_capability(struct e1000_adapter *adapter);
 extern void e1000e_reset_interrupt_capability(struct e1000_adapter *adapter);
 
index 840e8c4fb75386c6838a3571ec5dce1e46f81e73..34f1f639429de3e9f7de384333488dc69c74f5b2 100644 (file)
@@ -173,11 +173,8 @@ static int e1000_get_settings(struct net_device *netdev,
 static u32 e1000_get_link(struct net_device *netdev)
 {
        struct e1000_adapter *adapter = netdev_priv(netdev);
-       struct e1000_hw *hw = &adapter->hw;
-       u32 status;
-       
-       status = er32(STATUS);
-       return (status & E1000_STATUS_LU) ? 1 : 0;
+
+       return e1000_has_link(adapter);
 }
 
 static int e1000_set_spd_dplx(struct e1000_adapter *adapter, u16 spddplx)
index b3f0b4797016647e8b5983e2b58c324d5e46ffce..b1b534d267ed77f1a1b363f439e9e422f21d652d 100644 (file)
@@ -3421,7 +3421,7 @@ static void e1000_print_link_info(struct e1000_adapter *adapter)
               ((ctrl & E1000_CTRL_TFCE) ? "TX" : "None" )));
 }
 
-static bool e1000_has_link(struct e1000_adapter *adapter)
+bool e1000_has_link(struct e1000_adapter *adapter)
 {
        struct e1000_hw *hw = &adapter->hw;
        bool link_active = 0;