ide: add ide_no_data_taskfile() helper

* Add ide_no_data_taskfile() helper and convert ide_raw_taskfile() w/ NO DATA
  protocol users to use it instead.

* Set ->data_phase explicitly in ide_no_data_taskfile()
  (TASKFILE_NO_DATA is defined as 0x0000).

* Unexport task_no_data_intr().

Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

1 files changed, 8 insertions, 24 deletions

diff --git a/drivers/ide/ide-disk.c b/drivers/ide/ide-disk.c
index 6387222dd200..b534fe97d476 100644
--- a/drivers/ide/ide-disk.c
+++ b/drivers/ide/ide-disk.c
@@ -303,10 +303,8 @@ static u64 idedisk_read_native_max_address(ide_drive_t *drive, int lba48)
         else
                 tf->command = WIN_READ_NATIVE_MAX;
         tf->device  = ATA_LBA;
-        args.command_type                       = IDE_DRIVE_TASK_NO_DATA;
-        args.handler                            = &task_no_data_intr;
         /* submit command request */
-        ide_raw_taskfile(drive, &args, NULL);
+        ide_no_data_taskfile(drive, &args);
 
         /* if OK, compute maximum address value */
         if ((tf->status & 0x01) == 0) {
@@ -350,10 +348,8 @@ static u64 idedisk_set_max_address(ide_drive_t *drive, u64 addr_req, int lba48)
                 tf->command  = WIN_SET_MAX;
         }
         tf->device |= ATA_LBA;
-        args.command_type                       = IDE_DRIVE_TASK_NO_DATA;
-        args.handler                            = &task_no_data_intr;
         /* submit command request */
-        ide_raw_taskfile(drive, &args, NULL);
+        ide_no_data_taskfile(drive, &args);
         /* if OK, compute maximum address value */
         if ((tf->status & 0x01) == 0) {
                 u32 high, low;
@@ -500,9 +496,7 @@ static int smart_enable(ide_drive_t *drive)
         tf->lbam    = SMART_LCYL_PASS;
         tf->lbah    = SMART_HCYL_PASS;
         tf->command = WIN_SMART;
-        args.command_type                       = IDE_DRIVE_TASK_NO_DATA;
-        args.handler                            = &task_no_data_intr;
-        return ide_raw_taskfile(drive, &args, NULL);
+        return ide_no_data_taskfile(drive, &args);
 }
 
 static int get_smart_data(ide_drive_t *drive, u8 *buf, u8 sub_cmd)
@@ -695,9 +689,7 @@ static int write_cache(ide_drive_t *drive, int arg)
                 args.tf.feature = arg ?
                         SETFEATURES_EN_WCACHE : SETFEATURES_DIS_WCACHE;
                 args.tf.command = WIN_SETFEATURES;
-                args.command_type               = IDE_DRIVE_TASK_NO_DATA;
-                args.handler                    = &task_no_data_intr;
-                err = ide_raw_taskfile(drive, &args, NULL);
+                err = ide_no_data_taskfile(drive, &args);
                 if (err == 0)
                         drive->wcache = arg;
         }
@@ -716,9 +708,7 @@ static int do_idedisk_flushcache (ide_drive_t *drive)
                 args.tf.command = WIN_FLUSH_CACHE_EXT;
         else
                 args.tf.command = WIN_FLUSH_CACHE;
-        args.command_type                       = IDE_DRIVE_TASK_NO_DATA;
-        args.handler                            = &task_no_data_intr;
-        return ide_raw_taskfile(drive, &args, NULL);
+        return ide_no_data_taskfile(drive, &args);
 }
 
 static int set_acoustic (ide_drive_t *drive, int arg)
@@ -732,9 +722,7 @@ static int set_acoustic (ide_drive_t *drive, int arg)
         args.tf.feature = arg ? SETFEATURES_EN_AAM : SETFEATURES_DIS_AAM;
         args.tf.nsect   = arg;
         args.tf.command = WIN_SETFEATURES;
-        args.command_type = IDE_DRIVE_TASK_NO_DATA;
-        args.handler      = &task_no_data_intr;
-        ide_raw_taskfile(drive, &args, NULL);
+        ide_no_data_taskfile(drive, &args);
         drive->acoustic = arg;
         return 0;
 }
@@ -996,15 +984,13 @@ static int idedisk_open(struct inode *inode, struct file *filp)
                 ide_task_t args;
                 memset(&args, 0, sizeof(ide_task_t));
                 args.tf.command = WIN_DOORLOCK;
-                args.command_type = IDE_DRIVE_TASK_NO_DATA;
-                args.handler      = &task_no_data_intr;
                 check_disk_change(inode->i_bdev);
                 /*
                  * Ignore the return code from door_lock,
                  * since the open() has already succeeded,
                  * and the door_lock is irrelevant at this point.
                  */
-                if (drive->doorlocking && ide_raw_taskfile(drive, &args, NULL))
+                if (drive->doorlocking && ide_no_data_taskfile(drive, &args))
                         drive->doorlocking = 0;
         }
         return 0;
@@ -1023,9 +1009,7 @@ static int idedisk_release(struct inode *inode, struct file *filp)
                 ide_task_t args;
                 memset(&args, 0, sizeof(ide_task_t));
                 args.tf.command = WIN_DOORUNLOCK;
-                args.command_type = IDE_DRIVE_TASK_NO_DATA;
-                args.handler      = &task_no_data_intr;
-                if (drive->doorlocking && ide_raw_taskfile(drive, &args, NULL))
+                if (drive->doorlocking && ide_no_data_taskfile(drive, &args))
                         drive->doorlocking = 0;
         }