From: Marcelo Feitoza Parisi <marcelo@xxxxxxxxxxxxxx> Use of the time_after_eq() macro, defined at linux/jiffies.h, which deal with wrapping correctly and are nicer to read. Signed-off-by: Marcelo Feitoza Parisi <marcelo@xxxxxxxxxxxxxx> Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx> --- olympic.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletion(-) Index: quilt/drivers/net/tokenring/olympic.c =================================================================== --- quilt.orig/drivers/net/tokenring/olympic.c +++ quilt/drivers/net/tokenring/olympic.c @@ -100,6 +100,7 @@ #include <linux/pci.h> #include <linux/spinlock.h> #include <linux/bitops.h> +#include <linux/jiffies.h> #include <net/checksum.h> @@ -312,7 +313,7 @@ static int __devinit olympic_init(struct t=jiffies; while((readl(olympic_mmio+BCTL)) & BCTL_SOFTRESET) { schedule(); - if(jiffies-t > 40*HZ) { + if(time_after(jiffies, t + 40*HZ)) { printk(KERN_ERR "IBM PCI tokenring card not responding.\n"); return -ENODEV; } -- - : send the line "unsubscribe linux-net" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html