Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=05d1efffdc9bf84311bb1a3c2e3db55b544ca119
Commit:     05d1efffdc9bf84311bb1a3c2e3db55b544ca119
Parent:     4f34337b1f6f7c1e0f2e3c938eb9eadd340593fe
Author:     Alan Cox <[EMAIL PROTECTED]>
AuthorDate: Fri Aug 10 13:59:49 2007 -0700
Committer:  Jeff Garzik <[EMAIL PROTECTED]>
CommitDate: Fri Oct 12 14:55:33 2007 -0400

    pata_cmd64x: Set up MWDMA modes properly
    
    Set the MWDMA timing by updating the correct registers.  Split the PIO path 
as
    this is mostly shared code.  Wants testing.
    
    Signed-off-by: Alan Cox <[EMAIL PROTECTED]>
    Tested-by: Mikael Pettersson <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]>
---
 drivers/ata/pata_cmd64x.c |   37 ++++++++++++++++++++++++++-----------
 1 files changed, 26 insertions(+), 11 deletions(-)

diff --git a/drivers/ata/pata_cmd64x.c b/drivers/ata/pata_cmd64x.c
index e34b632..922902a 100644
--- a/drivers/ata/pata_cmd64x.c
+++ b/drivers/ata/pata_cmd64x.c
@@ -31,7 +31,7 @@
 #include <linux/libata.h>
 
 #define DRV_NAME "pata_cmd64x"
-#define DRV_VERSION "0.2.4"
+#define DRV_VERSION "0.2.5"
 
 /*
  * CMD64x specific registers definition.
@@ -88,14 +88,15 @@ static int cmd648_cable_detect(struct ata_port *ap)
 }
 
 /**
- *     cmd64x_set_piomode      -       set initial PIO mode data
+ *     cmd64x_set_piomode      -       set PIO and MWDMA timing
  *     @ap: ATA interface
  *     @adev: ATA device
+ *     @mode: mode
  *
- *     Called to do the PIO mode setup.
+ *     Called to do the PIO and MWDMA mode setup.
  */
 
-static void cmd64x_set_piomode(struct ata_port *ap, struct ata_device *adev)
+static void cmd64x_set_timing(struct ata_port *ap, struct ata_device *adev, u8 
mode)
 {
        struct pci_dev *pdev = to_pci_dev(ap->host->dev);
        struct ata_timing t;
@@ -117,8 +118,9 @@ static void cmd64x_set_piomode(struct ata_port *ap, struct 
ata_device *adev)
        int arttim = arttim_port[ap->port_no][adev->devno];
        int drwtim = drwtim_port[ap->port_no][adev->devno];
 
-
-       if (ata_timing_compute(adev, adev->pio_mode, &t, T, 0) < 0) {
+       /* ata_timing_compute is smart and will produce timings for MWDMA
+          that don't violate the drives PIO capabilities. */
+       if (ata_timing_compute(adev, mode, &t, T, 0) < 0) {
                printk(KERN_ERR DRV_NAME ": mode computation failed.\n");
                return;
        }
@@ -168,6 +170,20 @@ static void cmd64x_set_piomode(struct ata_port *ap, struct 
ata_device *adev)
 }
 
 /**
+ *     cmd64x_set_piomode      -       set initial PIO mode data
+ *     @ap: ATA interface
+ *     @adev: ATA device
+ *
+ *     Used when configuring the devices ot set the PIO timings. All the
+ *     actual work is done by the PIO/MWDMA setting helper
+ */
+
+static void cmd64x_set_piomode(struct ata_port *ap, struct ata_device *adev)
+{
+       cmd64x_set_timing(ap, adev, adev->pio_mode);
+}
+
+/**
  *     cmd64x_set_dmamode      -       set initial DMA mode data
  *     @ap: ATA interface
  *     @adev: ATA device
@@ -180,9 +196,6 @@ static void cmd64x_set_dmamode(struct ata_port *ap, struct 
ata_device *adev)
        static const u8 udma_data[] = {
                0x30, 0x20, 0x10, 0x20, 0x10, 0x00
        };
-       static const u8 mwdma_data[] = {
-               0x30, 0x20, 0x10
-       };
 
        struct pci_dev *pdev = to_pci_dev(ap->host->dev);
        u8 regU, regD;
@@ -208,8 +221,10 @@ static void cmd64x_set_dmamode(struct ata_port *ap, struct 
ata_device *adev)
                regU |= 1 << adev->devno; /* UDMA on */
                if (adev->dma_mode > 2) /* 15nS timing */
                        regU |= 4 << adev->devno;
-       } else
-               regD |= mwdma_data[adev->dma_mode - XFER_MW_DMA_0] << shift;
+       } else {
+               regU &= ~ (1 << adev->devno);   /* UDMA off */
+               cmd64x_set_timing(ap, adev, adev->dma_mode);
+       }
 
        regD |= 0x20 << adev->devno;
 
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to