diff options
| author | roel kluin <roel.kluin@gmail.com> | 2009-09-21 10:08:48 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-09-30 20:03:15 -0700 |
| commit | ec1652af18ef02c7c6ceeabb64f56f16eaf40ae9 (patch) | |
| tree | 94596543902a232043993a3de5b2999a2b127b82 /drivers | |
| parent | ebd6e7744f26b1a0e10d8a46ee57a3e76ceec6f9 (diff) | |
| download | op-kernel-dev-ec1652af18ef02c7c6ceeabb64f56f16eaf40ae9.zip op-kernel-dev-ec1652af18ef02c7c6ceeabb64f56f16eaf40ae9.tar.gz | |
bcm63xx_enet: timeout off by one in do_mdio_op()
`while (limit-- >= 0)' reaches -2 after the loop upon timeout.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Maxime Bizon <mbizon@freebox.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/bcm63xx_enet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/bcm63xx_enet.c b/drivers/net/bcm63xx_enet.c index 09d2709..ba29dc3 100644 --- a/drivers/net/bcm63xx_enet.c +++ b/drivers/net/bcm63xx_enet.c @@ -90,7 +90,7 @@ static int do_mdio_op(struct bcm_enet_priv *priv, unsigned int data) if (enet_readl(priv, ENET_IR_REG) & ENET_IR_MII) break; udelay(1); - } while (limit-- >= 0); + } while (limit-- > 0); return (limit < 0) ? 1 : 0; } |
