1 files changed, 7 insertions, 11 deletions

diff --git a/drivers/ata/pata_legacy.c b/drivers/ata/pata_legacy.c
index 930c2208640b..6c1d778b63a9 100644
--- a/drivers/ata/pata_legacy.c
+++ b/drivers/ata/pata_legacy.c
@@ -194,15 +194,12 @@ static int legacy_set_mode(struct ata_link *link, struct ata_device **unused)
 {
         struct ata_device *dev;
 
-        ata_link_for_each_dev(dev, link) {
-                if (ata_dev_enabled(dev)) {
-                        ata_dev_printk(dev, KERN_INFO,
-                                                "configured for PIO\n");
-                        dev->pio_mode = XFER_PIO_0;
-                        dev->xfer_mode = XFER_PIO_0;
-                        dev->xfer_shift = ATA_SHIFT_PIO;
-                        dev->flags |= ATA_DFLAG_PIO;
-                }
+        ata_for_each_dev(dev, link, ENABLED) {
+                ata_dev_printk(dev, KERN_INFO, "configured for PIO\n");
+                dev->pio_mode = XFER_PIO_0;
+                dev->xfer_mode = XFER_PIO_0;
+                dev->xfer_shift = ATA_SHIFT_PIO;
+                dev->flags |= ATA_DFLAG_PIO;
         }
         return 0;
 }
@@ -641,7 +638,6 @@ static void qdi6500_set_piomode(struct ata_port *ap, struct ata_device *adev)
  *      qdi6580dp_set_piomode           -       PIO setup for dual channel
  *      @ap: Port
  *      @adev: Device
- *      @irq: interrupt line
  *
  *      In dual channel mode the 6580 has one clock per channel and we have
  *      to software clockswitch in qc_issue.
@@ -1028,7 +1024,7 @@ static __init int legacy_init_one(struct legacy_probe *probe)
         /* Nothing found means we drop the port as its probably not there */
 
         ret = -ENODEV;
-        ata_link_for_each_dev(dev, &ap->link) {
+        ata_for_each_dev(dev, &ap->link, ALL) {
                 if (!ata_dev_absent(dev)) {
                         legacy_host[probe->slot] = host;
                         ld->platform_dev = pdev;