diff options
author | Tejun Heo <htejun@gmail.com> | 2008-03-24 23:22:48 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-04-17 15:44:17 -0400 |
commit | 6bd99b4e0998571808fc1f09d5162348f21ff8c1 (patch) | |
tree | e338bd445aaa5cda4bbf1f5598acf70f0d6b68de /drivers/ata/pata_mpc52xx.c | |
parent | 358f9a77a668660729e705fde9c3cf69f013aa98 (diff) |
libata: normalize port_info, port_operations and sht tables
Over the time, port info, ops and sht structures developed quite a bit
of inconsistencies. This patch updates drivers.
* Enable/disable_pm callbacks added to all ahci ops tables.
* Every driver for SFF controllers now uses ata_sff_port_start()
instead of ata_port_start() unless the driver has custom
implementation.
* Every driver for SFF controllers now uses ata_pci_default_filter()
unless the driver has custom implementation.
* Removed an odd port_info->sht initialization from ata_piix.c.
Likely a merge byproduct.
* A port which has ATA_FLAG_SATA set doesn't need to set cable_detect
to ata_cable_sata(). Remove it from via and mv port ops.
* Some drivers had unnecessary .max_sectors initialization which is
ignored and was missing .slave_destroy callback. Fixed.
* Removed unnecessary sht initializations port_info's.
* Removed onsolete scsi device suspend/resume callbacks from
pata_bf54x.
* No reason to set ata_pci_default_filter() and bmdma functions for
PIO-only drivers. Remove those callbacks and replace
ata_bmdma_irq_clear with ata_noop_irq_clear.
* pata_platform sets port_start to ata_dummy_ret0. port_start can
just be set to NULL.
* sata_fsl supports NCQ but was missing qc_defer. Fixed.
* pata_rb600_cf implements dummy port_start. Removed.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Diffstat (limited to 'drivers/ata/pata_mpc52xx.c')
-rw-r--r-- | drivers/ata/pata_mpc52xx.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/ata/pata_mpc52xx.c b/drivers/ata/pata_mpc52xx.c index d84e0c8ea02f..fefe71dbed1a 100644 --- a/drivers/ata/pata_mpc52xx.c +++ b/drivers/ata/pata_mpc52xx.c | |||
@@ -272,13 +272,13 @@ static struct scsi_host_template mpc52xx_ata_sht = { | |||
272 | .can_queue = ATA_DEF_QUEUE, | 272 | .can_queue = ATA_DEF_QUEUE, |
273 | .this_id = ATA_SHT_THIS_ID, | 273 | .this_id = ATA_SHT_THIS_ID, |
274 | .sg_tablesize = LIBATA_MAX_PRD, | 274 | .sg_tablesize = LIBATA_MAX_PRD, |
275 | .max_sectors = ATA_MAX_SECTORS, | ||
276 | .cmd_per_lun = ATA_SHT_CMD_PER_LUN, | 275 | .cmd_per_lun = ATA_SHT_CMD_PER_LUN, |
277 | .emulated = ATA_SHT_EMULATED, | 276 | .emulated = ATA_SHT_EMULATED, |
278 | .use_clustering = ATA_SHT_USE_CLUSTERING, | 277 | .use_clustering = ATA_SHT_USE_CLUSTERING, |
279 | .proc_name = DRV_NAME, | 278 | .proc_name = DRV_NAME, |
280 | .dma_boundary = ATA_DMA_BOUNDARY, | 279 | .dma_boundary = ATA_DMA_BOUNDARY, |
281 | .slave_configure = ata_scsi_slave_config, | 280 | .slave_configure = ata_scsi_slave_config, |
281 | .slave_destroy = ata_scsi_slave_destroy, | ||
282 | .bios_param = ata_std_bios_param, | 282 | .bios_param = ata_std_bios_param, |
283 | }; | 283 | }; |
284 | 284 | ||
@@ -298,7 +298,7 @@ static struct ata_port_operations mpc52xx_ata_port_ops = { | |||
298 | .data_xfer = ata_data_xfer, | 298 | .data_xfer = ata_data_xfer, |
299 | .irq_clear = ata_noop_irq_clear, | 299 | .irq_clear = ata_noop_irq_clear, |
300 | .irq_on = ata_irq_on, | 300 | .irq_on = ata_irq_on, |
301 | .port_start = ata_port_start, | 301 | .port_start = ata_sff_port_start, |
302 | }; | 302 | }; |
303 | 303 | ||
304 | static int __devinit | 304 | static int __devinit |