diff options
author | Shani Moideen <shani.moideen@wipro.com> | 2007-04-28 11:05:43 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-04-28 11:05:43 -0400 |
commit | 1c0d6dcf885badc72f72bfba9bc5b4f1469b8501 (patch) | |
tree | 31dd65febf04b3bb9e73337962608bf76cef4cf6 /drivers | |
parent | 693aa9470d8273a0ded8b211a8f5f7c0835adf30 (diff) |
[netdrvr] eexpress: minor corrections
A few corrections related to time_after and time_before in
drivers/net/eexpress.c as suggested by Marcin slusarz.
Signed-off-by: Shani Moideen <shani.moideen@wipro.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/eexpress.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/eexpress.c b/drivers/net/eexpress.c index 2600e2c27a82..7934ea37f944 100644 --- a/drivers/net/eexpress.c +++ b/drivers/net/eexpress.c | |||
@@ -777,7 +777,7 @@ static unsigned short eexp_start_irq(struct net_device *dev, | |||
777 | static void eexp_cmd_clear(struct net_device *dev) | 777 | static void eexp_cmd_clear(struct net_device *dev) |
778 | { | 778 | { |
779 | unsigned long int oldtime = jiffies; | 779 | unsigned long int oldtime = jiffies; |
780 | while (scb_rdcmd(dev) && ((jiffies-oldtime)<10)); | 780 | while (scb_rdcmd(dev) && (time_before(jiffies, oldtime + 10))); |
781 | if (scb_rdcmd(dev)) { | 781 | if (scb_rdcmd(dev)) { |
782 | printk("%s: command didn't clear\n", dev->name); | 782 | printk("%s: command didn't clear\n", dev->name); |
783 | } | 783 | } |
@@ -1650,7 +1650,7 @@ eexp_set_multicast(struct net_device *dev) | |||
1650 | #endif | 1650 | #endif |
1651 | oj = jiffies; | 1651 | oj = jiffies; |
1652 | while ((SCB_CUstat(scb_status(dev)) == 2) && | 1652 | while ((SCB_CUstat(scb_status(dev)) == 2) && |
1653 | (time_after(jiffies, oj + 2000))); | 1653 | (time_before(jiffies, oj + 2000))); |
1654 | if (SCB_CUstat(scb_status(dev)) == 2) | 1654 | if (SCB_CUstat(scb_status(dev)) == 2) |
1655 | printk("%s: warning, CU didn't stop\n", dev->name); | 1655 | printk("%s: warning, CU didn't stop\n", dev->name); |
1656 | lp->started &= ~(STARTED_CU); | 1656 | lp->started &= ~(STARTED_CU); |