diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-27 07:46:45 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-27 07:46:45 -0400 |
commit | 130e886708d6e11f3d54e5d27c266578de56f343 (patch) | |
tree | af6c09224bd1aa21119c052cd3664719916ec795 /drivers/ide/ide-cd.c | |
parent | ba7d479c36dde12821c01ad0696d678635b8fb92 (diff) |
ide: remove ide_end_request()
* Add ide_rq_bytes() helper.
* Add blk_noretry_request() quirk to ide_complete_rq() (currently only fs
requests can be marked as "noretry" so there is no change in behavior).
* Switch current ide_end_request() users to use ide_complete_rq().
[ No need to check for rq->nr_sectors == 0 in {ide_dma,task_pio}_intr(),
nsectors == 0 in cdrom_end_request() and err == 0 in ide_do_devset(). ]
* Remove no longer needed ide_end_request().
There should be no functional changes caused by this patch.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/ide-cd.c')
-rw-r--r-- | drivers/ide/ide-cd.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index e4fa807fdcfa..2f698c6e913f 100644 --- a/drivers/ide/ide-cd.c +++ b/drivers/ide/ide-cd.c | |||
@@ -298,7 +298,7 @@ static void cdrom_end_request(ide_drive_t *drive, int uptodate) | |||
298 | if (blk_fs_request(rq) == 0 && uptodate <= 0 && rq->errors == 0) | 298 | if (blk_fs_request(rq) == 0 && uptodate <= 0 && rq->errors == 0) |
299 | rq->errors = -EIO; | 299 | rq->errors = -EIO; |
300 | 300 | ||
301 | ide_end_request(drive, uptodate, nsectors); | 301 | ide_complete_rq(drive, uptodate ? 0 : -EIO, nsectors << 9); |
302 | } | 302 | } |
303 | 303 | ||
304 | static void ide_dump_status_no_sense(ide_drive_t *drive, const char *msg, u8 st) | 304 | static void ide_dump_status_no_sense(ide_drive_t *drive, const char *msg, u8 st) |
@@ -793,10 +793,11 @@ static ide_startstop_t cdrom_newpc_intr(ide_drive_t *drive) | |||
793 | if (dma_error) | 793 | if (dma_error) |
794 | return ide_error(drive, "dma error", stat); | 794 | return ide_error(drive, "dma error", stat); |
795 | if (blk_fs_request(rq)) { | 795 | if (blk_fs_request(rq)) { |
796 | ide_end_request(drive, 1, rq->nr_sectors); | 796 | ide_complete_rq(drive, 0, rq->nr_sectors |
797 | ? (rq->nr_sectors << 9) : ide_rq_bytes(rq)); | ||
797 | return ide_stopped; | 798 | return ide_stopped; |
798 | } else if (rq->cmd_type == REQ_TYPE_ATA_PC && !rq->bio) { | 799 | } else if (rq->cmd_type == REQ_TYPE_ATA_PC && !rq->bio) { |
799 | ide_end_request(drive, 1, 1); | 800 | ide_complete_rq(drive, 0, 512); |
800 | return ide_stopped; | 801 | return ide_stopped; |
801 | } | 802 | } |
802 | goto end_request; | 803 | goto end_request; |