diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-01-26 14:13:11 -0500 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-01-26 14:13:11 -0500 |
commit | 2624565caacedd740fce7803fe2c162842aa5df4 (patch) | |
tree | fe177bad8d72bfefa138a3abd7f8c7b92f549169 /drivers | |
parent | 4906f3b4cddc3e4d62955ed386598561f95602c0 (diff) |
ide: use wait_drive_not_busy() in drive_cmd_intr() (take 2)
Use wait_drive_not_busy() in drive_cmd_intr().
v2:
* Fix wait_drive_not_busy() comment (noticed by Sergei).
Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/ide/ide-io.c | 4 | ||||
-rw-r--r-- | drivers/ide/ide-taskfile.c | 5 |
2 files changed, 3 insertions, 6 deletions
diff --git a/drivers/ide/ide-io.c b/drivers/ide/ide-io.c index 0f3e2f4f9c2c..513a5685db2f 100644 --- a/drivers/ide/ide-io.c +++ b/drivers/ide/ide-io.c | |||
@@ -640,7 +640,6 @@ static ide_startstop_t drive_cmd_intr (ide_drive_t *drive) | |||
640 | ide_hwif_t *hwif = HWIF(drive); | 640 | ide_hwif_t *hwif = HWIF(drive); |
641 | u8 *args = (u8 *) rq->buffer; | 641 | u8 *args = (u8 *) rq->buffer; |
642 | u8 stat = hwif->INB(IDE_STATUS_REG); | 642 | u8 stat = hwif->INB(IDE_STATUS_REG); |
643 | int retries = 10; | ||
644 | 643 | ||
645 | local_irq_enable_in_hardirq(); | 644 | local_irq_enable_in_hardirq(); |
646 | if (rq->cmd_type == REQ_TYPE_ATA_CMD && | 645 | if (rq->cmd_type == REQ_TYPE_ATA_CMD && |
@@ -649,8 +648,7 @@ static ide_startstop_t drive_cmd_intr (ide_drive_t *drive) | |||
649 | drive->io_32bit = 0; | 648 | drive->io_32bit = 0; |
650 | hwif->ata_input_data(drive, &args[4], args[3] * SECTOR_WORDS); | 649 | hwif->ata_input_data(drive, &args[4], args[3] * SECTOR_WORDS); |
651 | drive->io_32bit = io_32bit; | 650 | drive->io_32bit = io_32bit; |
652 | while (((stat = hwif->INB(IDE_STATUS_REG)) & BUSY_STAT) && retries--) | 651 | stat = wait_drive_not_busy(drive); |
653 | udelay(100); | ||
654 | } | 652 | } |
655 | 653 | ||
656 | if (!OK_STAT(stat, READY_STAT, BAD_STAT)) | 654 | if (!OK_STAT(stat, READY_STAT, BAD_STAT)) |
diff --git a/drivers/ide/ide-taskfile.c b/drivers/ide/ide-taskfile.c index c34836c02b14..b559bece6e73 100644 --- a/drivers/ide/ide-taskfile.c +++ b/drivers/ide/ide-taskfile.c | |||
@@ -260,7 +260,7 @@ static ide_startstop_t task_no_data_intr(ide_drive_t *drive) | |||
260 | return ide_stopped; | 260 | return ide_stopped; |
261 | } | 261 | } |
262 | 262 | ||
263 | static u8 wait_drive_not_busy(ide_drive_t *drive) | 263 | u8 wait_drive_not_busy(ide_drive_t *drive) |
264 | { | 264 | { |
265 | ide_hwif_t *hwif = HWIF(drive); | 265 | ide_hwif_t *hwif = HWIF(drive); |
266 | int retries; | 266 | int retries; |
@@ -268,8 +268,7 @@ static u8 wait_drive_not_busy(ide_drive_t *drive) | |||
268 | 268 | ||
269 | /* | 269 | /* |
270 | * Last sector was transfered, wait until drive is ready. | 270 | * Last sector was transfered, wait until drive is ready. |
271 | * This can take up to 10 usec, but we will wait max 1 ms | 271 | * This can take up to 10 usec, but we will wait max 1 ms. |
272 | * (drive_cmd_intr() waits that long). | ||
273 | */ | 272 | */ |
274 | for (retries = 0; retries < 100; retries++) { | 273 | for (retries = 0; retries < 100; retries++) { |
275 | if ((stat = hwif->INB(IDE_STATUS_REG)) & BUSY_STAT) | 274 | if ((stat = hwif->INB(IDE_STATUS_REG)) & BUSY_STAT) |