diff options
author | Tejun Heo <htejun@gmail.com> | 2008-01-26 14:13:11 -0500 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-01-26 14:13:11 -0500 |
commit | 4d7a984bdcbdda69fc6b2a4a655415140270aa7b (patch) | |
tree | 92d331aa25949277fe76354dfdaf5a5d6e62aaf4 /drivers | |
parent | 79f21b84d94c02b941818df282b61f2a5a028722 (diff) |
ide: task_end_request() fix
task_end_request() modified to always call ide_end_drive_cmd()
for taskfile requests. Previously, ide_end_drive_cmd() was
called only when IDE_TFLAG_FLAGGED was set. Also,
ide_dma_intr() is modified to use task_end_request().
Enables TASKFILE ioctls to get valid register outputs on
successful completion.
Bart:
- ported it over recent IDE changes
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/ide/ide-dma.c | 8 | ||||
-rw-r--r-- | drivers/ide/ide-taskfile.c | 11 |
2 files changed, 5 insertions, 14 deletions
diff --git a/drivers/ide/ide-dma.c b/drivers/ide/ide-dma.c index c9648b1ef220..5bf32038dc43 100644 --- a/drivers/ide/ide-dma.c +++ b/drivers/ide/ide-dma.c | |||
@@ -153,13 +153,7 @@ ide_startstop_t ide_dma_intr (ide_drive_t *drive) | |||
153 | if (!dma_stat) { | 153 | if (!dma_stat) { |
154 | struct request *rq = HWGROUP(drive)->rq; | 154 | struct request *rq = HWGROUP(drive)->rq; |
155 | 155 | ||
156 | if (rq->rq_disk) { | 156 | task_end_request(drive, rq, stat); |
157 | ide_driver_t *drv; | ||
158 | |||
159 | drv = *(ide_driver_t **)rq->rq_disk->private_data; | ||
160 | drv->end_request(drive, 1, rq->nr_sectors); | ||
161 | } else | ||
162 | ide_end_request(drive, 1, rq->nr_sectors); | ||
163 | return ide_stopped; | 157 | return ide_stopped; |
164 | } | 158 | } |
165 | printk(KERN_ERR "%s: dma_intr: bad DMA status (dma_stat=%x)\n", | 159 | printk(KERN_ERR "%s: dma_intr: bad DMA status (dma_stat=%x)\n", |
diff --git a/drivers/ide/ide-taskfile.c b/drivers/ide/ide-taskfile.c index 3bbb438f4f92..17c2c0467297 100644 --- a/drivers/ide/ide-taskfile.c +++ b/drivers/ide/ide-taskfile.c | |||
@@ -408,16 +408,13 @@ static ide_startstop_t task_error(ide_drive_t *drive, struct request *rq, | |||
408 | return ide_error(drive, s, stat); | 408 | return ide_error(drive, s, stat); |
409 | } | 409 | } |
410 | 410 | ||
411 | static void task_end_request(ide_drive_t *drive, struct request *rq, u8 stat) | 411 | void task_end_request(ide_drive_t *drive, struct request *rq, u8 stat) |
412 | { | 412 | { |
413 | if (rq->cmd_type == REQ_TYPE_ATA_TASKFILE) { | 413 | if (rq->cmd_type == REQ_TYPE_ATA_TASKFILE) { |
414 | ide_task_t *task = rq->special; | 414 | u8 err = drive->hwif->INB(IDE_ERROR_REG); |
415 | 415 | ||
416 | if (task->tf_flags & IDE_TFLAG_FLAGGED) { | 416 | ide_end_drive_cmd(drive, stat, err); |
417 | u8 err = drive->hwif->INB(IDE_ERROR_REG); | 417 | return; |
418 | ide_end_drive_cmd(drive, stat, err); | ||
419 | return; | ||
420 | } | ||
421 | } | 418 | } |
422 | 419 | ||
423 | if (rq->rq_disk) { | 420 | if (rq->rq_disk) { |