ide: merge flagged_taskfile() into do_rw_taskfile()
Based on the earlier work by Tejun Heo. task->data_phase == TASKFILE_MULTI_{IN,OUT} vs drive->mult_count == 0 check is needed also for ide_taskfile_ioctl() requests that don't have IDE_TFLAG_FLAGGED taskfile flag set. Cc: Tejun Heo <htejun@gmail.com> Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
This commit is contained in:
parent
a7bbd20b50
commit
1edee60e9d
|
@ -851,9 +851,6 @@ static ide_startstop_t execute_drive_cmd (ide_drive_t *drive,
|
|||
break;
|
||||
}
|
||||
|
||||
if (task->tf_flags & IDE_TFLAG_FLAGGED)
|
||||
return flagged_taskfile(drive, task);
|
||||
|
||||
return do_rw_taskfile(drive, task);
|
||||
}
|
||||
|
||||
|
|
|
@ -156,6 +156,18 @@ ide_startstop_t do_rw_taskfile (ide_drive_t *drive, ide_task_t *task)
|
|||
ide_hwif_t *hwif = HWIF(drive);
|
||||
struct ide_taskfile *tf = &task->tf;
|
||||
|
||||
if (task->data_phase == TASKFILE_MULTI_IN ||
|
||||
task->data_phase == TASKFILE_MULTI_OUT) {
|
||||
if (!drive->mult_count) {
|
||||
printk(KERN_ERR "%s: multimode not set!\n",
|
||||
drive->name);
|
||||
return ide_stopped;
|
||||
}
|
||||
}
|
||||
|
||||
if (task->tf_flags & IDE_TFLAG_FLAGGED)
|
||||
task->tf_flags |= IDE_TFLAG_FLAGGED_SET_IN_FLAGS;
|
||||
|
||||
ide_tf_load(drive, task);
|
||||
|
||||
if (task->handler != NULL) {
|
||||
|
@ -839,23 +851,3 @@ int ide_task_ioctl (ide_drive_t *drive, unsigned int cmd, unsigned long arg)
|
|||
|
||||
return err;
|
||||
}
|
||||
|
||||
/*
|
||||
* NOTICE: This is additions from IBM to provide a discrete interface,
|
||||
* for selective taskregister access operations. Nice JOB Klaus!!!
|
||||
* Glad to be able to work and co-develop this with you and IBM.
|
||||
*/
|
||||
ide_startstop_t flagged_taskfile (ide_drive_t *drive, ide_task_t *task)
|
||||
{
|
||||
if (task->data_phase == TASKFILE_MULTI_IN ||
|
||||
task->data_phase == TASKFILE_MULTI_OUT) {
|
||||
if (!drive->mult_count) {
|
||||
printk(KERN_ERR "%s: multimode not set!\n", drive->name);
|
||||
return ide_stopped;
|
||||
}
|
||||
}
|
||||
|
||||
task->tf_flags |= IDE_TFLAG_FLAGGED_SET_IN_FLAGS;
|
||||
|
||||
return do_rw_taskfile(drive, task);
|
||||
}
|
||||
|
|
|
@ -972,11 +972,6 @@ void ide_pktcmd_tf_load(ide_drive_t *, u32, u16, u8);
|
|||
*/
|
||||
extern ide_startstop_t do_rw_taskfile(ide_drive_t *, ide_task_t *);
|
||||
|
||||
/*
|
||||
* Special Flagged Register Validation Caller
|
||||
*/
|
||||
extern ide_startstop_t flagged_taskfile(ide_drive_t *, ide_task_t *);
|
||||
|
||||
extern ide_startstop_t set_multmode_intr(ide_drive_t *);
|
||||
extern ide_startstop_t set_geometry_intr(ide_drive_t *);
|
||||
extern ide_startstop_t recal_intr(ide_drive_t *);
|
||||
|
|
Loading…
Reference in New Issue