summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoel Kluin <12o3l@tiscali.nl>2007-11-13 03:16:17 -0800
committerDavid S. Miller <davem@davemloft.net>2007-11-13 03:16:17 -0800
commit9db7720cca8f55a1e0c8d0dc1587e35dca50afbf (patch)
tree6ca945d7594933d8aa4157c273da08c178000808
parent81d9ddae856678c45297550e9353c8a5a7fd6438 (diff)
downloadblackbird-op-linux-9db7720cca8f55a1e0c8d0dc1587e35dca50afbf.tar.gz
blackbird-op-linux-9db7720cca8f55a1e0c8d0dc1587e35dca50afbf.zip
[MYRI_SBUS]: Prevent that myri_do_handshake lies about ticks.
With '<=' tick can be incremented up to 26, The last loop is redundant since even when 'softstate' becomes 'STATE_READY', 'if (tick > 25)' will still cause the function to return -1, Signed-off-by: Roel Kluin <12o3l@tiscali.nl> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/myri_sbus.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/myri_sbus.c b/drivers/net/myri_sbus.c
index 8d29319cc5cb..656a260fc956 100644
--- a/drivers/net/myri_sbus.c
+++ b/drivers/net/myri_sbus.c
@@ -134,7 +134,7 @@ static int myri_do_handshake(struct myri_eth *mp)
myri_disable_irq(mp->lregs, cregs);
- while (tick++ <= 25) {
+ while (tick++ < 25) {
u32 softstate;
/* Wake it up. */
OpenPOWER on IntegriCloud