Commit a620e134 authored by Ben Hutchings's avatar Ben Hutchings Committed by Jeff Garzik

sfc: Stop generating bogus events in tenxpress_check_hw()

mdio_clause45_links_ok() correctly checks efx_phy_mode_disabled(), so
tenxpress_link_ok() doesn't need to.

From: Steve Hodgson <shodgson@solarflare.com>
Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 94be0c54
...@@ -377,8 +377,7 @@ static int tenxpress_phy_check_hw(struct efx_nic *efx) ...@@ -377,8 +377,7 @@ static int tenxpress_phy_check_hw(struct efx_nic *efx)
struct tenxpress_phy_data *phy_data = efx->phy_data; struct tenxpress_phy_data *phy_data = efx->phy_data;
bool link_ok; bool link_ok;
link_ok = (phy_data->phy_mode == PHY_MODE_NORMAL && link_ok = tenxpress_link_ok(efx, true);
tenxpress_link_ok(efx, true));
if (link_ok != efx->link_up) if (link_ok != efx->link_up)
falcon_xmac_sim_phy_event(efx); falcon_xmac_sim_phy_event(efx);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment