netdev
[Top] [All Lists]

Re: [patch 07/26] net/cycx_drv: replace delay_cycx() with msleep_interru

To: domen@xxxxxxxxxxxx
Subject: Re: [patch 07/26] net/cycx_drv: replace delay_cycx() with msleep_interruptible()
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Wed, 09 Mar 2005 15:26:20 -0500
Cc: netdev@xxxxxxxxxxx, nacc@xxxxxxxxxx, acme@xxxxxxxxxxxxxxxx, janitor@xxxxxxxxxxxxxx
In-reply-to: <20050306103258.6FC481E46E@xxxxxxxxxxxxxxxxxxx>
References: <20050306103258.6FC481E46E@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.3) Gecko/20040922
domen@xxxxxxxxxxxx wrote:
Use msleep_interruptible() instead of delay_cycx()
to guarantee the task delays as expected. Remove the prototype and
definition of delay_cycx().

Signed-off-by: Nishanth Aravamudan <nacc@xxxxxxxxxx>
Acked-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxx>
Signed-off-by: Maximilian Attems <janitor@xxxxxxxxxxxxxx>
Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx>
---


 kj-domen/drivers/net/wan/cycx_drv.c |   24 ++++++++----------------
 1 files changed, 8 insertions(+), 16 deletions(-)

diff -puN 
drivers/net/wan/cycx_drv.c~msleep_interruptible-drivers_net_wan_cycx_drv 
drivers/net/wan/cycx_drv.c
--- kj/drivers/net/wan/cycx_drv.c~msleep_interruptible-drivers_net_wan_cycx_drv 
2005-03-05 16:09:34.000000000 +0100
+++ kj-domen/drivers/net/wan/cycx_drv.c 2005-03-05 16:09:34.000000000 +0100
@@ -56,7 +56,7 @@
 #include <linux/sched.h> /* for jiffies, HZ, etc. */
 #include <linux/cycx_drv.h>      /* API definitions */
 #include <linux/cycx_cfm.h>      /* CYCX firmware module definitions */
-#include <linux/delay.h> /* udelay */
+#include <linux/delay.h> /* udelay, msleep */
 #include <asm/io.h>              /* read[wl], write[wl], ioremap, iounmap */
#define MOD_VERSION 0
@@ -74,7 +74,6 @@ static int reset_cyc2x(void __iomem *add
 static int detect_cyc2x(void __iomem *addr);
/* Miscellaneous functions */
-static void delay_cycx(int sec);
 static int get_option_index(long *optlist, long optval);
 static u16 checksum(u8 *buf, u32 len);
@@ -259,7 +258,7 @@ static int memory_exists(void __iomem *a
                        if (readw(addr + 0x10) == TEST_PATTERN)
                                return 1;
- delay_cycx(1);
+               msleep_interruptible(1000);

use ssleep_interruptible



<Prev in Thread] Current Thread [Next in Thread>