[libata] Use ATA_UDMAx standard masks when filling driver's udma_mask info

The ATA_UDMAx masks are self-documenting, and far better than manually
writing in the hex mask.

Note that pata_it8213 mask differed from the comment.  Added a FIXME there.

Signed-off-by: Jeff Garzik <jeff@garzik.org>

1 files changed, 9 insertions, 9 deletions

diff --git a/drivers/ata/pata_amd.c b/drivers/ata/pata_amd.c
index 533bcc9d1542..b9c44c575ce3 100644
--- a/drivers/ata/pata_amd.c
+++ b/drivers/ata/pata_amd.c
@@ -552,7 +552,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x1f,      /* UDMA 66 */
+                        .udma_mask = ATA_UDMA4, /* UDMA 66 */
                         .port_ops = &amd66_port_ops
                 },
                 {       /* 2: AMD 7409, no swdma errata */
@@ -560,7 +560,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x1f,      /* UDMA 66 */
+                        .udma_mask = ATA_UDMA4, /* UDMA 66 */
                         .port_ops = &amd66_port_ops
                 },
                 {       /* 3: AMD 7411 */
@@ -568,7 +568,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x3f,      /* UDMA 100 */
+                        .udma_mask = ATA_UDMA5, /* UDMA 100 */
                         .port_ops = &amd100_port_ops
                 },
                 {       /* 4: AMD 7441 */
@@ -576,7 +576,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x3f,      /* UDMA 100 */
+                        .udma_mask = ATA_UDMA5, /* UDMA 100 */
                         .port_ops = &amd100_port_ops
                 },
                 {       /* 5: AMD 8111*/
@@ -584,7 +584,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x7f,      /* UDMA 133, no swdma */
+                        .udma_mask = ATA_UDMA6, /* UDMA 133, no swdma */
                         .port_ops = &amd133_port_ops
                 },
                 {       /* 6: AMD 8111 UDMA 100 (Serenade) */
@@ -592,7 +592,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x3f,      /* UDMA 100, no swdma */
+                        .udma_mask = ATA_UDMA5, /* UDMA 100, no swdma */
                         .port_ops = &amd133_port_ops
                 },
                 {       /* 7: Nvidia Nforce */
@@ -600,7 +600,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x3f,      /* UDMA 100 */
+                        .udma_mask = ATA_UDMA5, /* UDMA 100 */
                         .port_ops = &nv100_port_ops
                 },
                 {       /* 8: Nvidia Nforce2 and later */
@@ -608,7 +608,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x7f,      /* UDMA 133, no swdma */
+                        .udma_mask = ATA_UDMA6, /* UDMA 133, no swdma */
                         .port_ops = &nv133_port_ops
                 },
                 {       /* 9: AMD CS5536 (Geode companion) */
@@ -616,7 +616,7 @@ static int amd_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
                         .flags = ATA_FLAG_SLAVE_POSS,
                         .pio_mask = 0x1f,
                         .mwdma_mask = 0x07,
-                        .udma_mask = 0x3f,      /* UDMA 100 */
+                        .udma_mask = ATA_UDMA5, /* UDMA 100 */
                         .port_ops = &amd100_port_ops
                 }
         };