Use of the time_after() macro, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read. Signed-off-by: Marcelo Feitoza Parisi --- linux/drivers/net/pcmcia/smc91c92_cs.c 2005-07-07 19:11:51.000000000 -0300 +++ linux-kj/drivers/net/pcmcia/smc91c92_cs.c 2005-07-07 23:01:03.239783920 -0300 @@ -41,6 +41,7 @@ #include #include #include +#include #include #include @@ -2097,7 +2098,7 @@ } /* Ignore collisions unless we've had no rx's recently */ - if (jiffies - dev->last_rx > HZ) { + if (time_after(jiffies, dev->last_rx + HZ)) { if (smc->tx_err || (smc->media_status & EPH_16COL)) media |= EPH_16COL; }