diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-04-28 17:44:36 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-04-28 17:44:36 -0400 |
commit | 9567b349f7e7dd7e2483db99ee8e4a6fe0caca38 (patch) | |
tree | 29abab829b52a451567f8da4280d101fafa44527 /include/linux/ide.h | |
parent | 92d3ab27e8fd23d1a9dc3b69d17b2afb83e5c6f5 (diff) |
ide: merge ->atapi_*put_bytes and ->ata_*put_data methods
* Merge ->atapi_{in,out}put_bytes and ->ata_{in,out}put_data methods
into new ->{in,out}put_data methods which take number of bytes to
transfer as an argument and always do padding.
While at it:
* Use 'hwif' or 'drive->hwif' instead of 'HWIF(drive)'.
There should be no functional changes caused by this patch (all users
of ->ata_{in,out}put_data methods were using multiply-of-4 word counts).
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'include/linux/ide.h')
-rw-r--r-- | include/linux/ide.h | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/include/linux/ide.h b/include/linux/ide.h index 0cbc46bf08a5..b89b95dcb708 100644 --- a/include/linux/ide.h +++ b/include/linux/ide.h | |||
@@ -467,11 +467,8 @@ typedef struct hwif_s { | |||
467 | const struct ide_port_ops *port_ops; | 467 | const struct ide_port_ops *port_ops; |
468 | const struct ide_dma_ops *dma_ops; | 468 | const struct ide_dma_ops *dma_ops; |
469 | 469 | ||
470 | void (*ata_input_data)(ide_drive_t *, struct request *, void *, u32); | 470 | void (*input_data)(ide_drive_t *, struct request *, void *, unsigned); |
471 | void (*ata_output_data)(ide_drive_t *, struct request *, void *, u32); | 471 | void (*output_data)(ide_drive_t *, struct request *, void *, unsigned); |
472 | |||
473 | void (*atapi_input_bytes)(ide_drive_t *, void *, u32); | ||
474 | void (*atapi_output_bytes)(ide_drive_t *, void *, u32); | ||
475 | 472 | ||
476 | void (*ide_dma_clear_irq)(ide_drive_t *drive); | 473 | void (*ide_dma_clear_irq)(ide_drive_t *drive); |
477 | 474 | ||
@@ -547,7 +544,7 @@ typedef ide_startstop_t (ide_handler_t)(ide_drive_t *); | |||
547 | typedef int (ide_expiry_t)(ide_drive_t *); | 544 | typedef int (ide_expiry_t)(ide_drive_t *); |
548 | 545 | ||
549 | /* used by ide-cd, ide-floppy, etc. */ | 546 | /* used by ide-cd, ide-floppy, etc. */ |
550 | typedef void (xfer_func_t)(ide_drive_t *, void *, u32); | 547 | typedef void (xfer_func_t)(ide_drive_t *, struct request *rq, void *, unsigned); |
551 | 548 | ||
552 | typedef struct hwgroup_s { | 549 | typedef struct hwgroup_s { |
553 | /* irq handler, if active */ | 550 | /* irq handler, if active */ |
@@ -1369,7 +1366,7 @@ static inline void ide_atapi_discard_data(ide_drive_t *drive, unsigned bcount) | |||
1369 | { | 1366 | { |
1370 | ide_hwif_t *hwif = drive->hwif; | 1367 | ide_hwif_t *hwif = drive->hwif; |
1371 | 1368 | ||
1372 | /* FIXME: use ->atapi_input_bytes */ | 1369 | /* FIXME: use ->input_data */ |
1373 | while (bcount--) | 1370 | while (bcount--) |
1374 | (void)hwif->INB(hwif->io_ports.data_addr); | 1371 | (void)hwif->INB(hwif->io_ports.data_addr); |
1375 | } | 1372 | } |
@@ -1378,7 +1375,7 @@ static inline void ide_atapi_write_zeros(ide_drive_t *drive, unsigned bcount) | |||
1378 | { | 1375 | { |
1379 | ide_hwif_t *hwif = drive->hwif; | 1376 | ide_hwif_t *hwif = drive->hwif; |
1380 | 1377 | ||
1381 | /* FIXME: use ->atapi_output_bytes */ | 1378 | /* FIXME: use ->output_data */ |
1382 | while (bcount--) | 1379 | while (bcount--) |
1383 | hwif->OUTB(0, hwif->io_ports.data_addr); | 1380 | hwif->OUTB(0, hwif->io_ports.data_addr); |
1384 | } | 1381 | } |