drivers/net/eexpress.c : Use of the time_after() and
Subsystems (2)
PATCH 1/1 drivers/net/eexpress.c : Use of the time_after() and
Use of the time_after() and time_before() macros, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read.
Signed-off-by Marcelo Feitoza Parisi <[email protected]>
drivers/net/eexpress.c +4 -4
@@ -557,7 +557,7 @@if (lp->started) { -if ((jiffies - dev->trans_start)>50 ) +if (time_after (jiffies, dev->trans_start +50 )) {if (lp->tx_link==lp->last_tx_restart) {
@@ -613,7 +613,7 @@ }else { -if ((jiffies-lp->init_time)>10 ) +if (time_after (jiffies, lp->init_time +10 )) {unsigned short status =scb_status (dev); printk(KERN_WARNING "%s: i82586 startup timed out, status %04x, resetting...\n",
@@ -785,7 +785,7 @@static void eexp_cmd_clear (struct net_device *dev) {unsigned long int oldtime = jiffies; -while (scb_rdcmd (dev) && ((jiffies-oldtime)<10 )); +while (scb_rdcmd (dev) && (time_before (jiffies, oldtime +10 )));if (scb_rdcmd (dev)) {printk ("%s: command didn't clear\n" , dev->name); }
@@ -1666,7 +1666,7 @@#endif oj = jiffies;while ((SCB_CUstat (scb_status (dev)) ==2 ) && - ((jiffies-oj) <2000 )); + (time_before (jiffies, oj +2000 )));if (SCB_CUstat (scb_status (dev)) ==2 )printk ("%s: warning, CU didn't stop\n" , dev->name); lp->started &= ~(STARTED_CU );
_______________________________________________
Kernel-janitors mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/kernel-janitors