diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2011-02-24 18:59:15 -0500 |
---|---|---|
committer | Ben Hutchings <bhutchings@solarflare.com> | 2011-02-28 18:57:23 -0500 |
commit | a461103ba2e22cbb70771588b36f40df39a50f46 (patch) | |
tree | 526fe1532d3c69d9b322a458b8b26341058c6aa3 /drivers/net/sfc/txc43128_phy.c | |
parent | e5f0fd278084d79d6be0920043519749374b0507 (diff) |
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) | |||
193 | goto fail; | 193 | goto fail; |
194 | 194 | ||
195 | /* Check that all the MMDs we expect are present and responding. */ | 195 | /* Check that all the MMDs we expect are present and responding. */ |
196 | rc = efx_mdio_check_mmds(efx, TXC_REQUIRED_DEVS, 0); | 196 | rc = efx_mdio_check_mmds(efx, TXC_REQUIRED_DEVS); |
197 | if (rc < 0) | 197 | if (rc < 0) |
198 | goto fail; | 198 | goto fail; |
199 | 199 | ||