libata: improve 0xff status handling
[powerpc.git] / drivers / ata / pata_ali.c
index 1d695df..d40edeb 100644 (file)
@@ -34,7 +34,7 @@
 #include <linux/dmi.h>
 
 #define DRV_NAME "pata_ali"
-#define DRV_VERSION "0.6.6"
+#define DRV_VERSION "0.7.4"
 
 /*
  *     Cable special cases
@@ -78,7 +78,7 @@ static int ali_c2_cable_detect(struct ata_port *ap)
           implement the detect logic */
 
        if (ali_cable_override(pdev))
-               return ATA_CBL_PATA80;
+               return ATA_CBL_PATA40_SHORT;
 
        /* Host view cable detect 0x4A bit 0 primary bit 1 secondary
           Bit set for 40 pin */
@@ -89,59 +89,6 @@ static int ali_c2_cable_detect(struct ata_port *ap)
                return ATA_CBL_PATA80;
 }
 
-/**
- *     ali_early_error_handler -       reset for eary chip
- *     @ap: ATA port
- *
- *     Handle the reset callback for the later chips with cable detect
- */
-
-static int ali_c2_pre_reset(struct ata_port *ap)
-{
-       ap->cbl = ali_c2_cable_detect(ap);
-       return ata_std_prereset(ap);
-}
-
-static void ali_c2_error_handler(struct ata_port *ap)
-{
-       ata_bmdma_drive_eh(ap, ali_c2_pre_reset,
-                              ata_std_softreset, NULL,
-                              ata_std_postreset);
-}
-
-/**
- *     ali_early_cable_detect  -       cable detection
- *     @ap: ATA port
- *
- *     Perform cable detection for older chipsets. This turns out to be
- *     rather easy to implement
- */
-
-static int ali_early_cable_detect(struct ata_port *ap)
-{
-       return ATA_CBL_PATA40;
-}
-
-/**
- *     ali_early_probe_init    -       reset for early chip
- *     @ap: ATA port
- *
- *     Handle the reset callback for the early (pre cable detect) chips.
- */
-
-static int ali_early_pre_reset(struct ata_port *ap)
-{
-       ap->cbl = ali_early_cable_detect(ap);
-       return ata_std_prereset(ap);
-}
-
-static void ali_early_error_handler(struct ata_port *ap)
-{
-       return ata_bmdma_drive_eh(ap, ali_early_pre_reset,
-                                    ata_std_softreset, NULL,
-                                    ata_std_postreset);
-}
-
 /**
  *     ali_20_filter           -       filter for earlier ALI DMA
  *     @ap: ALi ATA port
@@ -151,16 +98,16 @@ static void ali_early_error_handler(struct ata_port *ap)
  *     fix that later on. Also ensure we do not do UDMA on WDC drives
  */
 
-static unsigned long ali_20_filter(const struct ata_port *ap, struct ata_device *adev, unsigned long mask)
+static unsigned long ali_20_filter(struct ata_device *adev, unsigned long mask)
 {
-       char model_num[40];
+       char model_num[ATA_ID_PROD_LEN + 1];
        /* No DMA on anything but a disk for now */
        if (adev->class != ATA_DEV_ATA)
                mask &= ~(ATA_MASK_MWDMA | ATA_MASK_UDMA);
-       ata_id_string(adev->id, model_num, ATA_ID_PROD_OFS, sizeof(model_num));
+       ata_id_c_string(adev->id, model_num, ATA_ID_PROD, sizeof(model_num));
        if (strstr(model_num, "WDC"))
                return mask &= ~ATA_MASK_UDMA;
-       return ata_pci_default_filter(ap, adev, mask);
+       return ata_pci_default_filter(adev, mask);
 }
 
 /**
@@ -314,7 +261,6 @@ static void ali_set_dmamode(struct ata_port *ap, struct ata_device *adev)
 
 /**
  *     ali_lock_sectors        -       Keep older devices to 255 sector mode
- *     @ap: ATA port
  *     @adev: Device
  *
  *     Called during the bus probe for each device that is found. We use
@@ -324,7 +270,7 @@ static void ali_set_dmamode(struct ata_port *ap, struct ata_device *adev)
  *     slower PIO methods
  */
 
-static void ali_lock_sectors(struct ata_port *ap, struct ata_device *adev)
+static void ali_lock_sectors(struct ata_device *adev)
 {
        adev->max_sectors = 255;
 }
@@ -337,16 +283,18 @@ static struct scsi_host_template ali_sht = {
        .can_queue              = ATA_DEF_QUEUE,
        .this_id                = ATA_SHT_THIS_ID,
        .sg_tablesize           = LIBATA_MAX_PRD,
-       /* Keep LBA28 counts so large I/O's don't turn LBA48 and PIO
-          with older controllers. Not locked so will grow on C5 or later */
-       .max_sectors            = 255,
        .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,
        .bios_param             = ata_std_bios_param,
+#ifdef CONFIG_PM
+       .resume                 = ata_scsi_device_resume,
+       .suspend                = ata_scsi_device_suspend,
+#endif
 };
 
 /*
@@ -364,20 +312,21 @@ static struct ata_port_operations ali_early_port_ops = {
 
        .freeze         = ata_bmdma_freeze,
        .thaw           = ata_bmdma_thaw,
-       .error_handler  = ali_early_error_handler,
+       .error_handler  = ata_bmdma_error_handler,
        .post_internal_cmd = ata_bmdma_post_internal_cmd,
+       .cable_detect   = ata_cable_40wire,
 
        .qc_prep        = ata_qc_prep,
        .qc_issue       = ata_qc_issue_prot,
 
-       .data_xfer      = ata_pio_data_xfer,
+       .data_xfer      = ata_data_xfer,
 
        .irq_handler    = ata_interrupt,
        .irq_clear      = ata_bmdma_irq_clear,
+       .irq_on         = ata_irq_on,
+       .irq_ack        = ata_irq_ack,
 
        .port_start     = ata_port_start,
-       .port_stop      = ata_port_stop,
-       .host_stop      = ata_host_stop
 };
 
 /*
@@ -400,8 +349,9 @@ static struct ata_port_operations ali_20_port_ops = {
 
        .freeze         = ata_bmdma_freeze,
        .thaw           = ata_bmdma_thaw,
-       .error_handler  = ali_early_error_handler,
+       .error_handler  = ata_bmdma_error_handler,
        .post_internal_cmd = ata_bmdma_post_internal_cmd,
+       .cable_detect   = ata_cable_40wire,
 
        .bmdma_setup    = ata_bmdma_setup,
        .bmdma_start    = ata_bmdma_start,
@@ -411,14 +361,14 @@ static struct ata_port_operations ali_20_port_ops = {
        .qc_prep        = ata_qc_prep,
        .qc_issue       = ata_qc_issue_prot,
 
-       .data_xfer      = ata_pio_data_xfer,
+       .data_xfer      = ata_data_xfer,
 
        .irq_handler    = ata_interrupt,
        .irq_clear      = ata_bmdma_irq_clear,
+       .irq_on         = ata_irq_on,
+       .irq_ack        = ata_irq_ack,
 
        .port_start     = ata_port_start,
-       .port_stop      = ata_port_stop,
-       .host_stop      = ata_host_stop
 };
 
 /*
@@ -438,8 +388,9 @@ static struct ata_port_operations ali_c2_port_ops = {
 
        .freeze         = ata_bmdma_freeze,
        .thaw           = ata_bmdma_thaw,
-       .error_handler  = ali_c2_error_handler,
+       .error_handler  = ata_bmdma_error_handler,
        .post_internal_cmd = ata_bmdma_post_internal_cmd,
+       .cable_detect   = ali_c2_cable_detect,
 
        .bmdma_setup    = ata_bmdma_setup,
        .bmdma_start    = ata_bmdma_start,
@@ -449,14 +400,14 @@ static struct ata_port_operations ali_c2_port_ops = {
        .qc_prep        = ata_qc_prep,
        .qc_issue       = ata_qc_issue_prot,
 
-       .data_xfer      = ata_pio_data_xfer,
+       .data_xfer      = ata_data_xfer,
 
        .irq_handler    = ata_interrupt,
        .irq_clear      = ata_bmdma_irq_clear,
+       .irq_on         = ata_irq_on,
+       .irq_ack        = ata_irq_ack,
 
        .port_start     = ata_port_start,
-       .port_stop      = ata_port_stop,
-       .host_stop      = ata_host_stop
 };
 
 /*
@@ -475,8 +426,9 @@ static struct ata_port_operations ali_c5_port_ops = {
 
        .freeze         = ata_bmdma_freeze,
        .thaw           = ata_bmdma_thaw,
-       .error_handler  = ali_c2_error_handler,
+       .error_handler  = ata_bmdma_error_handler,
        .post_internal_cmd = ata_bmdma_post_internal_cmd,
+       .cable_detect   = ali_c2_cable_detect,
 
        .bmdma_setup    = ata_bmdma_setup,
        .bmdma_start    = ata_bmdma_start,
@@ -486,16 +438,79 @@ static struct ata_port_operations ali_c5_port_ops = {
        .qc_prep        = ata_qc_prep,
        .qc_issue       = ata_qc_issue_prot,
 
-       .data_xfer      = ata_pio_data_xfer,
+       .data_xfer      = ata_data_xfer,
 
        .irq_handler    = ata_interrupt,
        .irq_clear      = ata_bmdma_irq_clear,
+       .irq_on         = ata_irq_on,
+       .irq_ack        = ata_irq_ack,
 
        .port_start     = ata_port_start,
-       .port_stop      = ata_port_stop,
-       .host_stop      = ata_host_stop
 };
 
+
+/**
+ *     ali_init_chipset        -       chip setup function
+ *     @pdev: PCI device of ATA controller
+ *
+ *     Perform the setup on the device that must be done both at boot
+ *     and at resume time.
+ */
+
+static void ali_init_chipset(struct pci_dev *pdev)
+{
+       u8 rev, tmp;
+       struct pci_dev *north, *isa_bridge;
+
+       pci_read_config_byte(pdev, PCI_REVISION_ID, &rev);
+
+       /*
+        * The chipset revision selects the driver operations and
+        * mode data.
+        */
+
+       if (rev >= 0x20 && rev < 0xC2) {
+               /* 1543-E/F, 1543C-C, 1543C-D, 1543C-E */
+               pci_read_config_byte(pdev, 0x4B, &tmp);
+               /* Clear CD-ROM DMA write bit */
+               tmp &= 0x7F;
+               pci_write_config_byte(pdev, 0x4B, tmp);
+       } else if (rev >= 0xC2) {
+               /* Enable cable detection logic */
+               pci_read_config_byte(pdev, 0x4B, &tmp);
+               pci_write_config_byte(pdev, 0x4B, tmp | 0x08);
+       }
+       north = pci_get_bus_and_slot(0, PCI_DEVFN(0,0));
+       isa_bridge = pci_get_device(PCI_VENDOR_ID_AL, PCI_DEVICE_ID_AL_M1533, NULL);
+
+       if (north && north->vendor == PCI_VENDOR_ID_AL && isa_bridge) {
+               /* Configure the ALi bridge logic. For non ALi rely on BIOS.
+                  Set the south bridge enable bit */
+               pci_read_config_byte(isa_bridge, 0x79, &tmp);
+               if (rev == 0xC2)
+                       pci_write_config_byte(isa_bridge, 0x79, tmp | 0x04);
+               else if (rev > 0xC2 && rev < 0xC5)
+                       pci_write_config_byte(isa_bridge, 0x79, tmp | 0x02);
+       }
+       if (rev >= 0x20) {
+               /*
+                * CD_ROM DMA on (0x53 bit 0). Enable this even if we want
+                * to use PIO. 0x53 bit 1 (rev 20 only) - enable FIFO control
+                * via 0x54/55.
+                */
+               pci_read_config_byte(pdev, 0x53, &tmp);
+               if (rev <= 0x20)
+                       tmp &= ~0x02;
+               if (rev >= 0xc7)
+                       tmp |= 0x03;
+               else
+                       tmp |= 0x01;    /* CD_ROM enable for DMA */
+               pci_write_config_byte(pdev, 0x53, tmp);
+       }
+       pci_dev_put(isa_bridge);
+       pci_dev_put(north);
+       ata_pci_clear_simplex(pdev);
+}
 /**
  *     ali_init_one            -       discovery callback
  *     @pdev: PCI device ID
@@ -569,7 +584,7 @@ static int ali_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
 
        static struct ata_port_info *port_info[2];
        u8 rev, tmp;
-       struct pci_dev *north, *isa_bridge;
+       struct pci_dev *isa_bridge;
 
        pci_read_config_byte(pdev, PCI_REVISION_ID, &rev);
 
@@ -581,11 +596,6 @@ static int ali_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
        if (rev < 0x20) {
                port_info[0] = port_info[1] = &info_early;
        } else if (rev < 0xC2) {
-               /* 1543-E/F, 1543C-C, 1543C-D, 1543C-E */
-               pci_read_config_byte(pdev, 0x4B, &tmp);
-               /* Clear CD-ROM DMA write bit */
-               tmp &= 0x7F;
-               pci_write_config_byte(pdev, 0x4B, tmp);
                port_info[0] = port_info[1] = &info_20;
        } else if (rev == 0xC2) {
                port_info[0] = port_info[1] = &info_c2;
@@ -596,54 +606,27 @@ static int ali_init_one(struct pci_dev *pdev, const struct pci_device_id *id)
        } else
                port_info[0] = port_info[1] = &info_c5;
 
-       if (rev >= 0xC2) {
-               /* Enable cable detection logic */
-               pci_read_config_byte(pdev, 0x4B, &tmp);
-               pci_write_config_byte(pdev, 0x4B, tmp | 0x08);
-       }
+       ali_init_chipset(pdev);
 
-       north = pci_get_slot(pdev->bus, PCI_DEVFN(0,0));
        isa_bridge = pci_get_device(PCI_VENDOR_ID_AL, PCI_DEVICE_ID_AL_M1533, NULL);
-
-       if (north && north->vendor == PCI_VENDOR_ID_AL) {
-               /* Configure the ALi bridge logic. For non ALi rely on BIOS.
-                  Set the south bridge enable bit */
-               pci_read_config_byte(isa_bridge, 0x79, &tmp);
-               if (rev == 0xC2)
-                       pci_write_config_byte(isa_bridge, 0x79, tmp | 0x04);
-               else if (rev > 0xC2)
-                       pci_write_config_byte(isa_bridge, 0x79, tmp | 0x02);
+       if (isa_bridge && rev >= 0x20 && rev < 0xC2) {
+               /* Are we paired with a UDMA capable chip */
+               pci_read_config_byte(isa_bridge, 0x5E, &tmp);
+               if ((tmp & 0x1E) == 0x12)
+                       port_info[0] = port_info[1] = &info_20_udma;
+               pci_dev_put(isa_bridge);
        }
-
-       if (rev >= 0x20) {
-               if (rev < 0xC2) {
-                       /* Are we paired with a UDMA capable chip */
-                       pci_read_config_byte(isa_bridge, 0x5E, &tmp);
-                       if ((tmp & 0x1E) == 0x12)
-                               port_info[0] = port_info[1] = &info_20_udma;
-               }
-               /*
-                * CD_ROM DMA on (0x53 bit 0). Enable this even if we want
-                * to use PIO. 0x53 bit 1 (rev 20 only) - enable FIFO control
-                * via 0x54/55.
-                */
-               pci_read_config_byte(pdev, 0x53, &tmp);
-               if (rev <= 0x20)
-                       tmp &= ~0x02;
-               if (rev >= 0xc7)
-                       tmp |= 0x03;
-               else
-                       tmp |= 0x01;    /* CD_ROM enable for DMA */
-               pci_write_config_byte(pdev, 0x53, tmp);
-       }
-
-       pci_dev_put(isa_bridge);
-       pci_dev_put(north);
-
-       ata_pci_clear_simplex(pdev);
        return ata_pci_init_one(pdev, port_info, 2);
 }
 
+#ifdef CONFIG_PM
+static int ali_reinit_one(struct pci_dev *pdev)
+{
+       ali_init_chipset(pdev);
+       return ata_pci_device_resume(pdev);
+}
+#endif
+
 static const struct pci_device_id ali[] = {
        { PCI_VDEVICE(AL, PCI_DEVICE_ID_AL_M5228), },
        { PCI_VDEVICE(AL, PCI_DEVICE_ID_AL_M5229), },
@@ -655,7 +638,11 @@ static struct pci_driver ali_pci_driver = {
        .name           = DRV_NAME,
        .id_table       = ali,
        .probe          = ali_init_one,
-       .remove         = ata_pci_remove_one
+       .remove         = ata_pci_remove_one,
+#ifdef CONFIG_PM
+       .suspend        = ata_pci_device_suspend,
+       .resume         = ali_reinit_one,
+#endif
 };
 
 static int __init ali_init(void)