[PATCH] parport: fix-up schedule_timeout() usage

Use schedule_timeout_interruptible() instead of
set_current_state()/schedule_timeout() to reduce kernel size.  Also use
human-time to jiffies units conversion functions rather than direct HZ
division to avoid rounding issues.

Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 4 insertions, 6 deletions

diff --git a/drivers/parport/ieee1284_ops.c b/drivers/parport/ieee1284_ops.c
index 6624278c6ed8..ce1e2aad8b10 100644
--- a/drivers/parport/ieee1284_ops.c
+++ b/drivers/parport/ieee1284_ops.c
@@ -60,7 +60,7 @@ size_t parport_ieee1284_write_compat (struct parport *port,
         parport_data_forward (port);
         while (count < len) {
                 unsigned long expire = jiffies + dev->timeout;
-                long wait = (HZ + 99) / 100;
+                long wait = msecs_to_jiffies(10);
                 unsigned char mask = (PARPORT_STATUS_ERROR
                                       | PARPORT_STATUS_BUSY);
                 unsigned char val = (PARPORT_STATUS_ERROR
@@ -97,8 +97,7 @@ size_t parport_ieee1284_write_compat (struct parport *port,
                            our interrupt handler called. */
                         if (count && no_irq) {
                                 parport_release (dev);
-                                __set_current_state (TASK_INTERRUPTIBLE);
-                                schedule_timeout (wait);
+                                schedule_timeout_interruptible(wait);
                                 parport_claim_or_block (dev);
                         }
                         else
@@ -542,13 +541,12 @@ size_t parport_ieee1284_ecp_read_data (struct parport *port,
                         /* Yield the port for a while. */
                         if (count && dev->port->irq != PARPORT_IRQ_NONE) {
                                 parport_release (dev);
-                                __set_current_state (TASK_INTERRUPTIBLE);
-                                schedule_timeout ((HZ + 24) / 25);
+                                schedule_timeout_interruptible(msecs_to_jiffies(40));
                                 parport_claim_or_block (dev);
                         }
                         else
                                 /* We must have the device claimed here. */
-                                parport_wait_event (port, (HZ + 24) / 25);
+                                parport_wait_event (port, msecs_to_jiffies(40));
 
                         /* Is there a signal pending? */
                         if (signal_pending (current))