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/atm/iphase.c 2005-06-17 16:48:29.000000000 -0300 +++ linux-kj/drivers/atm/iphase.c 2005-07-16 01:51:51.088494968 -0300 @@ -60,7 +60,8 @@ #include #include #include -#include +#include +#include #include "iphase.h" #include "suni.h" #define swap(x) (((x & 0xff) << 8) | ((x & 0xff00) >> 8)) @@ -189,7 +190,8 @@ int ltimeout; ia_hack_tcq (dev); - if(((jiffies - timer)>50)||((dev->ffL.tcq_rd==dev->host_tcq_wr))){ + if(time_after(jiffies, timer + 50) + ||((dev->ffL.tcq_rd==dev->host_tcq_wr))){ timer = jiffies; i=0; while (i < dev->num_tx_desc) {