ncr5380: Harmonize jiffies conversion with msecs_to_jiffies

Instances of var * HZ / 1000 are replaced by msecs_to_jiffies(var).
In addition some timing constants that assumed HZ 100 were adjusted
to HZ independent settings based on review comments from Michael Schmitz
<schmitzmic@gmail.com> and review of the original drivers in 1.0.31 and
2.2.16.

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Acked-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>

diff --git a/drivers/scsi/NCR5380.c b/drivers/scsi/NCR5380.c
index 8981701802ca72720d70c16e56d0dec30cc28b08..a777e5c412df262b3f7fc72b55676d681ade9cb8 100644 (file)
--- a/drivers/scsi/NCR5380.c
+++ b/drivers/scsi/NCR5380.c
@@ -474,11 +474,11 @@ static void NCR5380_print_phase(struct Scsi_Host *instance)
  */
 #ifndef USLEEP_SLEEP
 /* 20 ms (reasonable hard disk speed) */
-#define USLEEP_SLEEP (20*HZ/1000)
+#define USLEEP_SLEEP msecs_to_jiffies(20)
 #endif
 /* 300 RPM (floppy speed) */
 #ifndef USLEEP_POLL
-#define USLEEP_POLL (200*HZ/1000)
+#define USLEEP_POLL msecs_to_jiffies(200)
 #endif
 #ifndef USLEEP_WAITLONG
 /* RvC: (reasonable time to wait on select error) */
@@ -576,7 +576,7 @@ static int __init __maybe_unused NCR5380_probe_irq(struct Scsi_Host *instance,
                if ((mask & possible) && (request_irq(i, &probe_intr, 0, "NCR-probe", NULL) == 0))
                        trying_irqs |= mask;
 
-       timeout = jiffies + (250 * HZ / 1000);
+       timeout = jiffies + msecs_to_jiffies(250);
        probe_irq = NO_IRQ;
 
        /*
@@ -634,7 +634,7 @@ static void prepare_info(struct Scsi_Host *instance)
                 "sg_tablesize %d, this_id %d, "
                 "flags { %s%s%s}, "
 #if defined(USLEEP_POLL) && defined(USLEEP_WAITLONG)
-                "USLEEP_POLL %d, USLEEP_WAITLONG %d, "
+                "USLEEP_POLL %lu, USLEEP_WAITLONG %lu, "
 #endif
                 "options { %s} ",
                 instance->hostt->name, instance->io_port, instance->n_io_port,
@@ -1346,7 +1346,7 @@ static int NCR5380_select(struct Scsi_Host *instance, struct scsi_cmnd *cmd)
         * selection.
         */
 
-       timeout = jiffies + (250 * HZ / 1000);
+       timeout = jiffies + msecs_to_jiffies(250);
 
        /* 
         * XXX very interesting - we're seeing a bounce where the BSY we 

diff --git a/drivers/scsi/atari_NCR5380.c b/drivers/scsi/atari_NCR5380.c
index a70255413e7f97aa3e786ebe446d435ac4ab3d95..db87ece6edb2cd317d463c1afb21f7b9a9942b33 100644 (file)
--- a/drivers/scsi/atari_NCR5380.c
+++ b/drivers/scsi/atari_NCR5380.c
@@ -1486,7 +1486,7 @@ static int NCR5380_select(struct Scsi_Host *instance, struct scsi_cmnd *cmd)
         * selection.
         */
 
-       timeout = jiffies + (250 * HZ / 1000);
+       timeout = jiffies + msecs_to_jiffies(250);
 
        /*
         * XXX very interesting - we're seeing a bounce where the BSY we

diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c
index 74ec2f5669aba5f504594c14b54438524eed9877..f8d2478b11cc7a53453899db7b256543c7f21a33 100644 (file)
--- a/drivers/scsi/g_NCR5380.c
+++ b/drivers/scsi/g_NCR5380.c
@@ -57,9 +57,9 @@
  */
 
 /* settings for DTC3181E card with only Mustek scanner attached */
-#define USLEEP_POLL    1
-#define USLEEP_SLEEP   20
-#define USLEEP_WAITLONG        500
+#define USLEEP_POLL    msecs_to_jiffies(10)
+#define USLEEP_SLEEP   msecs_to_jiffies(200)
+#define USLEEP_WAITLONG        msecs_to_jiffies(5000)
 
 #define AUTOPROBE_IRQ