libata hpt3xn: Hopefully sort out the DPLL logic versus the vendor code
authorAlan <alan@lxorguk.ukuu.org.uk>
Wed, 24 Jan 2007 11:51:38 +0000 (11:51 +0000)
committerJeff Garzik <jeff@garzik.org>
Thu, 25 Jan 2007 00:52:31 +0000 (19:52 -0500)
Rather than ending up with two layers of negation jut rename the variable
and lose one.

Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/ata/pata_hpt3x2n.c

index f6817b4093a4e8ff3d74a5dc2e487858a9d00c79..886fab9aa62c5cdaec19c0036e3b32eb1cf3148c 100644 (file)
@@ -25,7 +25,7 @@
 #include <linux/libata.h>
 
 #define DRV_NAME       "pata_hpt3x2n"
-#define DRV_VERSION    "0.3"
+#define DRV_VERSION    "0.3.2"
 
 enum {
        HPT_PCI_FAST    =       (1 << 31),
@@ -297,11 +297,11 @@ static int hpt3x2n_pair_idle(struct ata_port *ap)
        return 0;
 }
 
-static int hpt3x2n_use_dpll(struct ata_port *ap, int reading)
+static int hpt3x2n_use_dpll(struct ata_port *ap, int writing)
 {
        long flags = (long)ap->host->private_data;
        /* See if we should use the DPLL */
-       if (reading == 0)
+       if (writing)
                return USE_DPLL;        /* Needed for write */
        if (flags & PCI66)
                return USE_DPLL;        /* Needed at 66Mhz */