diff options
| author | Yaniv Rosner <yanivr@broadcom.com> | 2014-01-01 13:06:44 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-01-02 07:09:48 +0400 |
| commit | ad1d9ef3f736485ba0f49325fa4c73cd1d963853 (patch) | |
| tree | 7751ab2dbbc040d0216f9202313fea94bc1c360b /drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c | |
| parent | a429ec239cbc42dd3b2bc9933745103ca71d2b9d (diff) | |
| download | linux-ad1d9ef3f736485ba0f49325fa4c73cd1d963853.tar.xz | |
bnx2x: Fix incorrect link-up report
Fix a problem where link is reported to be up when SFP+ module is plugged in
without cable. This occurs with specific module types which may generate
temporary TX_FAULT indication. Solution is to avoid changing any link parameters
when checking TX_FAULT indication while physical link is down.
Signed-off-by: Yaniv Rosner <yanivr@broadcom.com>
Signed-off-by: Ariel Elior <ariele@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c')
| -rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c index 68417e1fb6df..998cce36340c 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c @@ -13360,6 +13360,10 @@ static u8 bnx2x_analyze_link_error(struct link_params *params, DP(NETIF_MSG_LINK, "Link changed:[%x %x]->%x\n", vars->link_up, old_status, status); + /* Do not touch the link in case physical link down */ + if ((vars->phy_flags & PHY_PHYSICAL_LINK_FLAG) == 0) + return 1; + /* a. Update shmem->link_status accordingly * b. Update link_vars->link_up */ |