diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2011-02-25 02:59:15 +0300 |
---|---|---|
committer | Ben Hutchings <bhutchings@solarflare.com> | 2011-03-01 02:57:23 +0300 |
commit | a461103ba2e22cbb70771588b36f40df39a50f46 (patch) | |
tree | 526fe1532d3c69d9b322a458b8b26341058c6aa3 /drivers/net/sfc/txc43128_phy.c | |
parent | e5f0fd278084d79d6be0920043519749374b0507 (diff) | |
download | linux-a461103ba2e22cbb70771588b36f40df39a50f46.tar.xz |
sfc: Do not read STAT1.FAULT in efx_mdio_check_mmd()
This field does not exist in all MMDs we want to check, and all
callers allow it to be set (fault_fatal = 0).
Remove the loopback condition, as STAT2.DEVPRST should be valid
regardless of any fault.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Diffstat (limited to 'drivers/net/sfc/txc43128_phy.c')
-rw-r--r-- | drivers/net/sfc/txc43128_phy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/sfc/txc43128_phy.c b/drivers/net/sfc/txc43128_phy.c index 351794a79215..4e2b48a8f5c4 100644 --- a/drivers/net/sfc/txc43128_phy.c +++ b/drivers/net/sfc/txc43128_phy.c @@ -193,7 +193,7 @@ static int txc_reset_phy(struct efx_nic *efx) goto fail; /* Check that all the MMDs we expect are present and responding. */ - rc = efx_mdio_check_mmds(efx, TXC_REQUIRED_DEVS, 0); + rc = efx_mdio_check_mmds(efx, TXC_REQUIRED_DEVS); if (rc < 0) goto fail; |