Staging: phison: port code to work properly with latest libata
authorGreg Kroah-Hartman <gregkh@suse.de>
Thu, 12 Feb 2009 21:37:51 +0000 (13:37 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 3 Apr 2009 21:53:40 +0000 (14:53 -0700)
This brings the driver up to modern times so that it can build and run
properly with the in-tree libata code.

Cc: Evan Ko <evan_ko@phison.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/phison/phison.c

index fd299b8cc96ecf3952c60d522ea8262c6e9da6a5..270ebcb681a29d4e6eef9f555405335fdccd88f2 100644 (file)
@@ -39,69 +39,20 @@ static int phison_pre_reset(struct ata_link *link, unsigned long deadline)
        return ret;
 }
 
-static void phison_error_handler(struct ata_port *ap)
-{
-       dev_dbg(ap->dev, "phison_error_handler()\n");
-       return ata_bmdma_drive_eh(ap, phison_pre_reset,
-                                 ata_std_softreset, NULL,
-                                 ata_std_postreset);
-}
-
 static struct scsi_host_template phison_sht = {
-       .module                 = THIS_MODULE,
-       .name                   = DRV_NAME,
-       .ioctl                  = ata_scsi_ioctl,
-       .queuecommand   = ata_scsi_queuecmd,
-       .can_queue              = ATA_DEF_QUEUE,
-       .this_id                = ATA_SHT_THIS_ID,
-       .sg_tablesize   = LIBATA_MAX_PRD,
-       .cmd_per_lun    = ATA_SHT_CMD_PER_LUN,
-       .emulated               = ATA_SHT_EMULATED,
-       .use_clustering = ATA_SHT_USE_CLUSTERING,
-       .proc_name              = DRV_NAME,
-       .dma_boundary   = ATA_DMA_BOUNDARY,
-       .slave_configure        = ata_scsi_slave_config,
-       .slave_destroy  = ata_scsi_slave_destroy,
-       /* Use standard CHS mapping rules */
-       .bios_param             = ata_std_bios_param,
+       ATA_BMDMA_SHT(DRV_NAME),
 };
 
-static const struct ata_port_operations phison_ops = {
-       /* Task file is PCI ATA format, use helpers */
-       .tf_load                = ata_tf_load,
-       .tf_read                = ata_tf_read,
-       .check_status           = ata_check_status,
-       .exec_command           = ata_exec_command,
-       .dev_select             = ata_std_dev_select,
-
-       .freeze                 = ata_bmdma_freeze,
-       .thaw                   = ata_bmdma_thaw,
-       .error_handler          = phison_error_handler,
-       .post_internal_cmd      = ata_bmdma_post_internal_cmd,
-
-       /* BMDMA handling is PCI ATA format, use helpers */
-       .bmdma_setup            = ata_bmdma_setup,
-       .bmdma_start            = ata_bmdma_start,
-       .bmdma_stop             = ata_bmdma_stop,
-       .bmdma_status           = ata_bmdma_status,
-       .qc_prep                = ata_qc_prep,
-       .qc_issue               = ata_qc_issue_prot,
-       .data_xfer              = ata_data_xfer,
-
-       /* IRQ-related hooks */
-       .irq_handler            = ata_interrupt,
-       .irq_clear              = ata_bmdma_irq_clear,
-       .irq_on                 = ata_irq_on,
-
-       /* Generic PATA PCI ATA helpers */
-       .port_start             = ata_port_start,
+static struct ata_port_operations phison_ops = {
+       .inherits               = &ata_bmdma_port_ops,
+       .prereset               = phison_pre_reset,
 };
 
 static int phison_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
 {
+       int ret;
        struct ata_port_info info = {
-               .sht            = &phison_sht,
-               .flags  = ATA_FLAG_NO_ATAPI,
+               .flags          = ATA_FLAG_NO_ATAPI,
 
                .pio_mask       = 0x1f,
                .mwdma_mask     = 0x07,
@@ -109,11 +60,9 @@ static int phison_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
 
                .port_ops       = &phison_ops,
        };
-       int ret;
-
        const struct ata_port_info *ppi[] = { &info, NULL };
 
-       ret = ata_pci_init_one(pdev, ppi);
+       ret = ata_pci_sff_init_one(pdev, ppi, &phison_sht, NULL);
 
        dev_dbg(&pdev->dev, "phison_init_one(), ret = %x\n", ret);