]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
igb: fix link reporting when using sgmii
authorAlexander Duyck <alexander.h.duyck@intel.com>
Sat, 31 Jan 2009 08:53:18 +0000 (00:53 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sat, 31 Jan 2009 08:53:18 +0000 (00:53 -0800)
When using sgmii the link was not being properly passed up to the driver
from the underlying link management functions.  This change corrects it so
that get_link_status is cleared when a link has been found.

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>
drivers/net/igb/e1000_82575.c

index f5e2e7235fcbf197a5d427b49df86cef9d3d7c8b..13ca73f96ec6e397971ab9b7dbb42924430e8408 100644 (file)
@@ -699,11 +699,18 @@ static s32 igb_check_for_link_82575(struct e1000_hw *hw)
 
        /* SGMII link check is done through the PCS register. */
        if ((hw->phy.media_type != e1000_media_type_copper) ||
-           (igb_sgmii_active_82575(hw)))
+           (igb_sgmii_active_82575(hw))) {
                ret_val = igb_get_pcs_speed_and_duplex_82575(hw, &speed,
                                                             &duplex);
-       else
+               /*
+                * Use this flag to determine if link needs to be checked or
+                * not.  If  we have link clear the flag so that we do not
+                * continue to check for link.
+                */
+               hw->mac.get_link_status = !hw->mac.serdes_has_link;
+       } else {
                ret_val = igb_check_for_copper_link(hw);
+       }
 
        return ret_val;
 }