diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-27 07:46:47 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-27 07:46:47 -0400 |
commit | 22117d6eaac50d366d9013c88318a869ea4d8739 (patch) | |
tree | 7dae55b028f8c7c249341f026257f94c88eed3cf /drivers/ide/ide-iops.c | |
parent | 60c0cd02b254805691cdc61101ada6af7bd56fde (diff) |
ide: add ->dma_timer_expiry method and remove ->dma_exec_cmd one (v2)
* Rename dma_timer_expiry() to ide_dma_sff_timer_expiry() and export it.
* Add ->dma_timer_expiry method and use it to set hwif->expiry for
ATA_PROT_DMA protocol in do_rw_taskfile().
* Initialize ->dma_timer_expiry to ide_dma_sff_timer_expiry() for SFF hosts.
* Move setting hwif->expiry from ide_execute_command() to its users and drop
'expiry' argument.
* Use ide_execute_command() instead of ->dma_exec_cmd in do_rw_taskfile().
* Remove ->dma_exec_cmd method and its implementations.
* Unexport ide_execute_command() and ide_dma_intr().
v2:
* Fix CONFIG_BLK_DEV_IDEDMA=n build (noticed by Randy Dunlap).
* Fix *dma_expiry naming (suggested by Sergei Shtylyov).
There should be no functional changes caused by this patch.
Cc: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/ide-iops.c')
-rw-r--r-- | drivers/ide/ide-iops.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/ide/ide-iops.c b/drivers/ide/ide-iops.c index 916495ba45df..52c1258ba9f4 100644 --- a/drivers/ide/ide-iops.c +++ b/drivers/ide/ide-iops.c | |||
@@ -454,7 +454,6 @@ EXPORT_SYMBOL(ide_set_handler); | |||
454 | * @command: command byte to write | 454 | * @command: command byte to write |
455 | * @handler: handler for next phase | 455 | * @handler: handler for next phase |
456 | * @timeout: timeout for command | 456 | * @timeout: timeout for command |
457 | * @expiry: handler to run on timeout | ||
458 | * | 457 | * |
459 | * Helper function to issue an IDE command. This handles the | 458 | * Helper function to issue an IDE command. This handles the |
460 | * atomicity requirements, command timing and ensures that the | 459 | * atomicity requirements, command timing and ensures that the |
@@ -463,13 +462,11 @@ EXPORT_SYMBOL(ide_set_handler); | |||
463 | */ | 462 | */ |
464 | 463 | ||
465 | void ide_execute_command(ide_drive_t *drive, u8 cmd, ide_handler_t *handler, | 464 | void ide_execute_command(ide_drive_t *drive, u8 cmd, ide_handler_t *handler, |
466 | unsigned timeout, ide_expiry_t *expiry) | 465 | unsigned timeout) |
467 | { | 466 | { |
468 | ide_hwif_t *hwif = drive->hwif; | 467 | ide_hwif_t *hwif = drive->hwif; |
469 | unsigned long flags; | 468 | unsigned long flags; |
470 | 469 | ||
471 | hwif->expiry = expiry; | ||
472 | |||
473 | spin_lock_irqsave(&hwif->lock, flags); | 470 | spin_lock_irqsave(&hwif->lock, flags); |
474 | __ide_set_handler(drive, handler, timeout); | 471 | __ide_set_handler(drive, handler, timeout); |
475 | hwif->tp_ops->exec_command(hwif, cmd); | 472 | hwif->tp_ops->exec_command(hwif, cmd); |
@@ -482,7 +479,6 @@ void ide_execute_command(ide_drive_t *drive, u8 cmd, ide_handler_t *handler, | |||
482 | ndelay(400); | 479 | ndelay(400); |
483 | spin_unlock_irqrestore(&hwif->lock, flags); | 480 | spin_unlock_irqrestore(&hwif->lock, flags); |
484 | } | 481 | } |
485 | EXPORT_SYMBOL(ide_execute_command); | ||
486 | 482 | ||
487 | void ide_execute_pkt_cmd(ide_drive_t *drive) | 483 | void ide_execute_pkt_cmd(ide_drive_t *drive) |
488 | { | 484 | { |