diff options
author | yongari <yongari@FreeBSD.org> | 2006-08-02 02:36:59 +0000 |
---|---|---|
committer | yongari <yongari@FreeBSD.org> | 2006-08-02 02:36:59 +0000 |
commit | 1c044fd132c6765720a3c35007a656a163e27fc7 (patch) | |
tree | 93692be8a801ce110aa2e2799c7f381feb8430e6 | |
parent | 9f4698b4431ffb0afd6c5bf6d57e4862a4bcff30 (diff) | |
download | FreeBSD-src-1c044fd132c6765720a3c35007a656a163e27fc7.zip FreeBSD-src-1c044fd132c6765720a3c35007a656a163e27fc7.tar.gz |
Fix incorrect busy check for PHY write operation.
While I'm here remove unnecessary return statement.
-rw-r--r-- | sys/dev/sk/if_sk.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sys/dev/sk/if_sk.c b/sys/dev/sk/if_sk.c index 22c50b8..184d7e8 100644 --- a/sys/dev/sk/if_sk.c +++ b/sys/dev/sk/if_sk.c @@ -793,13 +793,11 @@ sk_marv_miibus_writereg(sc_if, phy, reg, val) for (i = 0; i < SK_TIMEOUT; i++) { DELAY(1); - if (SK_YU_READ_2(sc_if, YUKON_SMICR) & YU_SMICR_BUSY) + if ((SK_YU_READ_2(sc_if, YUKON_SMICR) & YU_SMICR_BUSY) == 0) break; } - if (i == SK_TIMEOUT) { + if (i == SK_TIMEOUT) if_printf(sc_if->sk_ifp, "phy write timeout\n"); - return (0); - } return(0); } |