Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d393aa0326436efd3cb800916b08cdac4dd1091a
Commit:     d393aa0326436efd3cb800916b08cdac4dd1091a
Parent:     69e88d2a754fcaf7ef8ca465586b811cf40dc479
Author:     Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
AuthorDate: Sat Oct 20 00:32:36 2007 +0200
Committer:  Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
CommitDate: Sat Oct 20 00:32:36 2007 +0200

    ide: take ide_lock for prefetch disable/enable in do_special()
    
    Take ide_lock for prefetch disable/enable in do_special(),
    then cleanup cmd640 and ht6560b host drivers.
    
    Signed-off-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
---
 drivers/ide/ide-io.c         |   15 ++++++++++++++-
 drivers/ide/legacy/ht6560b.c |    5 -----
 drivers/ide/pci/cmd640.c     |    6 ------
 3 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/drivers/ide/ide-io.c b/drivers/ide/ide-io.c
index 32eaa3f..e2eb408 100644
--- a/drivers/ide/ide-io.c
+++ b/drivers/ide/ide-io.c
@@ -800,7 +800,20 @@ static ide_startstop_t do_special (ide_drive_t *drive)
                s->b.set_tune = 0;
 
                if (set_pio_mode_abuse(drive->hwif, req_pio)) {
-                       if (hwif->set_pio_mode)
+
+                       if (hwif->set_pio_mode == NULL)
+                               return ide_stopped;
+
+                       /*
+                        * take ide_lock for drive->[no_]unmask/[no_]io_32bit
+                        */
+                       if (req_pio == 8 || req_pio == 9) {
+                               unsigned long flags;
+
+                               spin_lock_irqsave(&ide_lock, flags);
+                               hwif->set_pio_mode(drive, req_pio);
+                               spin_unlock_irqrestore(&ide_lock, flags);
+                       } else
                                hwif->set_pio_mode(drive, req_pio);
                } else {
                        int keep_dma = drive->using_dma;
diff --git a/drivers/ide/legacy/ht6560b.c b/drivers/ide/legacy/ht6560b.c
index 4a93adb..a4245d1 100644
--- a/drivers/ide/legacy/ht6560b.c
+++ b/drivers/ide/legacy/ht6560b.c
@@ -286,12 +286,7 @@ static void ht6560b_set_pio_mode(ide_drive_t *drive, const 
u8 pio)
        switch (pio) {
        case 8:         /* set prefetch off */
        case 9:         /* set prefetch on */
-               /*
-                * take ide_lock for drive->[no_]unmask
-                */
-               spin_lock_irqsave(&ide_lock, flags);
                ht_set_prefetch(drive, pio & 1);
-               spin_unlock_irqrestore(&ide_lock, flags);
                return;
        }
 
diff --git a/drivers/ide/pci/cmd640.c b/drivers/ide/pci/cmd640.c
index 0e9275d..4aa4810 100644
--- a/drivers/ide/pci/cmd640.c
+++ b/drivers/ide/pci/cmd640.c
@@ -632,7 +632,6 @@ static void cmd640_set_mode (unsigned int index, u8 
pio_mode, unsigned int cycle
 
 static void cmd640_set_pio_mode(ide_drive_t *drive, const u8 pio)
 {
-       unsigned long flags;
        unsigned int index = 0, cycle_time;
        u8 b;
 
@@ -655,12 +654,7 @@ static void cmd640_set_pio_mode(ide_drive_t *drive, const 
u8 pio)
 
                case 8: /* set prefetch off */
                case 9: /* set prefetch on */
-                       /*
-                        * take ide_lock for drive->[no_]unmask/[no_]io_32bit
-                        */
-                       spin_lock_irqsave(&ide_lock, flags);
                        set_prefetch_mode(index, pio & 1);
-                       spin_unlock_irqrestore(&ide_lock, flags);
                        printk("%s: %sabled cmd640 prefetch\n", drive->name, 
(pio & 1) ? "en" : "dis");
                        return;
        }
-
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