Merge git://git.kernel.org/pub/scm/linux/kernel/git/bart/ide-2.6
[safe/jmp/linux-2.6] / drivers / ide / setup-pci.c
index f187889..02d14bf 100644 (file)
@@ -147,15 +147,15 @@ static int ide_setup_pci_baseregs (struct pci_dev *dev, const char *name)
 #ifdef CONFIG_BLK_DEV_IDEDMA_PCI
 /**
  *     ide_get_or_set_dma_base         -       setup BMIBA
- *     @d: IDE pci device data
- *     @hwif: Interface
+ *     @d: IDE port info
+ *     @hwif: IDE interface
  *
  *     Fetch the DMA Bus-Master-I/O-Base-Address (BMIBA) from PCI space.
  *     Where a device has a partner that is already in DMA mode we check
  *     and enforce IDE simplex rules.
  */
 
-static unsigned long ide_get_or_set_dma_base(ide_pci_device_t *d, ide_hwif_t *hwif)
+static unsigned long ide_get_or_set_dma_base(const struct ide_port_info *d, ide_hwif_t *hwif)
 {
        unsigned long   dma_base = 0;
        struct pci_dev  *dev = hwif->pci_dev;
@@ -185,9 +185,9 @@ static unsigned long ide_get_or_set_dma_base(ide_pci_device_t *d, ide_hwif_t *hw
                        case PCI_DEVICE_ID_CMD_643:
                        case PCI_DEVICE_ID_SERVERWORKS_CSB5IDE:
                        case PCI_DEVICE_ID_REVOLUTION:
-                               simplex_stat = hwif->INB(dma_base + 2);
-                               hwif->OUTB((simplex_stat&0x60),(dma_base + 2));
-                               simplex_stat = hwif->INB(dma_base + 2);
+                               simplex_stat = inb(dma_base + 2);
+                               outb(simplex_stat & 0x60, dma_base + 2);
+                               simplex_stat = inb(dma_base + 2);
                                if (simplex_stat & 0x80) {
                                        printk(KERN_INFO "%s: simplex device: "
                                                         "DMA forced\n",
@@ -225,10 +225,11 @@ static unsigned long ide_get_or_set_dma_base(ide_pci_device_t *d, ide_hwif_t *hw
 }
 #endif /* CONFIG_BLK_DEV_IDEDMA_PCI */
 
-void ide_setup_pci_noise (struct pci_dev *dev, ide_pci_device_t *d)
+void ide_setup_pci_noise(struct pci_dev *dev, const struct ide_port_info *d)
 {
-       printk(KERN_INFO "%s: IDE controller at PCI slot %s\n",
-                        d->name, pci_name(dev));
+       printk(KERN_INFO "%s: IDE controller (0x%04x:0x%04x rev 0x%02x) at "
+                        " PCI slot %s\n", d->name, dev->vendor, dev->device,
+                        dev->revision, pci_name(dev));
 }
 
 EXPORT_SYMBOL_GPL(ide_setup_pci_noise);
@@ -237,15 +238,15 @@ EXPORT_SYMBOL_GPL(ide_setup_pci_noise);
 /**
  *     ide_pci_enable  -       do PCI enables
  *     @dev: PCI device
- *     @d: IDE pci device data
+ *     @d: IDE port info
  *
  *     Enable the IDE PCI device. We attempt to enable the device in full
  *     but if that fails then we only need BAR4 so we will enable that.
  *     
  *     Returns zero on success or an error code
  */
-static int ide_pci_enable(struct pci_dev *dev, ide_pci_device_t *d)
+
+static int ide_pci_enable(struct pci_dev *dev, const struct ide_port_info *d)
 {
        int ret;
 
@@ -260,9 +261,9 @@ static int ide_pci_enable(struct pci_dev *dev, ide_pci_device_t *d)
        }
 
        /*
-        * assume all devices can do 32-bit dma for now. we can add a
-        * dma mask field to the ide_pci_device_t if we need it (or let
-        * lower level driver set the dma mask)
+        * assume all devices can do 32-bit DMA for now, we can add
+        * a DMA mask field to the struct ide_port_info if we need it
+        * (or let lower level driver set the DMA mask)
         */
        ret = pci_set_dma_mask(dev, DMA_32BIT_MASK);
        if (ret < 0) {
@@ -284,13 +285,13 @@ out:
 /**
  *     ide_pci_configure       -       configure an unconfigured device
  *     @dev: PCI device
- *     @d: IDE pci device data
+ *     @d: IDE port info
  *
  *     Enable and configure the PCI device we have been passed.
  *     Returns zero on success or an error code.
  */
-static int ide_pci_configure(struct pci_dev *dev, ide_pci_device_t *d)
+
+static int ide_pci_configure(struct pci_dev *dev, const struct ide_port_info *d)
 {
        u16 pcicmd = 0;
        /*
@@ -318,15 +319,15 @@ static int ide_pci_configure(struct pci_dev *dev, ide_pci_device_t *d)
 
 /**
  *     ide_pci_check_iomem     -       check a register is I/O
- *     @dev: pci device
- *     @d: ide_pci_device
- *     @bar: bar number
+ *     @dev: PCI device
+ *     @d: IDE port info
+ *     @bar: BAR number
  *
  *     Checks if a BAR is configured and points to MMIO space. If so
  *     print an error and return an error code. Otherwise return 0
  */
-static int ide_pci_check_iomem(struct pci_dev *dev, ide_pci_device_t *d, int bar)
+
+static int ide_pci_check_iomem(struct pci_dev *dev, const struct ide_port_info *d, int bar)
 {
        ulong flags = pci_resource_flags(dev, bar);
        
@@ -348,7 +349,7 @@ static int ide_pci_check_iomem(struct pci_dev *dev, ide_pci_device_t *d, int bar
 /**
  *     ide_hwif_configure      -       configure an IDE interface
  *     @dev: PCI device holding interface
- *     @d: IDE pci data
+ *     @d: IDE port info
  *     @mate: Paired interface if any
  *
  *     Perform the initial set up for the hardware interface structure. This
@@ -357,8 +358,8 @@ static int ide_pci_check_iomem(struct pci_dev *dev, ide_pci_device_t *d, int bar
  *
  *     Returns the new hardware interface structure, or NULL on a failure
  */
-static ide_hwif_t *ide_hwif_configure(struct pci_dev *dev, ide_pci_device_t *d, ide_hwif_t *mate, int port, int irq)
+
+static ide_hwif_t *ide_hwif_configure(struct pci_dev *dev, const struct ide_port_info *d, ide_hwif_t *mate, int port, int irq)
 {
        unsigned long ctl = 0, base = 0;
        ide_hwif_t *hwif;
@@ -387,19 +388,20 @@ static ide_hwif_t *ide_hwif_configure(struct pci_dev *dev, ide_pci_device_t *d,
                return NULL;    /* no room in ide_hwifs[] */
        if (hwif->io_ports[IDE_DATA_OFFSET] != base ||
            hwif->io_ports[IDE_CONTROL_OFFSET] != (ctl | 2)) {
-               memset(&hwif->hw, 0, sizeof(hwif->hw));
-#ifndef IDE_ARCH_OBSOLETE_INIT
-               ide_std_init_ports(&hwif->hw, base, (ctl | 2));
-               hwif->hw.io_ports[IDE_IRQ_OFFSET] = 0;
+               hw_regs_t hw;
+
+               memset(&hw, 0, sizeof(hw));
+#ifndef CONFIG_IDE_ARCH_OBSOLETE_INIT
+               ide_std_init_ports(&hw, base, ctl | 2);
 #else
-               ide_init_hwif_ports(&hwif->hw, base, (ctl | 2), NULL);
+               ide_init_hwif_ports(&hw, base, ctl | 2, NULL);
 #endif
-               memcpy(hwif->io_ports, hwif->hw.io_ports, sizeof(hwif->io_ports));
+               memcpy(hwif->io_ports, hw.io_ports, sizeof(hwif->io_ports));
                hwif->noprobe = !hwif->io_ports[IDE_DATA_OFFSET];
        }
-       hwif->chipset = ide_pci;
+       hwif->chipset = d->chipset ? d->chipset : ide_pci;
        hwif->pci_dev = dev;
-       hwif->cds = (struct ide_pci_device_s *) d;
+       hwif->cds = d;
        hwif->channel = port;
 
        if (!hwif->irq)
@@ -414,21 +416,17 @@ static ide_hwif_t *ide_hwif_configure(struct pci_dev *dev, ide_pci_device_t *d,
 /**
  *     ide_hwif_setup_dma      -       configure DMA interface
  *     @dev: PCI device
- *     @d: IDE pci data
- *     @hwif: Hardware interface we are configuring
+ *     @d: IDE port info
+ *     @hwif: IDE interface
  *
  *     Set up the DMA base for the interface. Enable the master bits as
  *     necessary and attempt to bring the device DMA into a ready to use
  *     state
  */
-#ifndef CONFIG_BLK_DEV_IDEDMA_PCI
-static void ide_hwif_setup_dma(struct pci_dev *dev, ide_pci_device_t *d, ide_hwif_t *hwif)
-{
-}
-#else
-static void ide_hwif_setup_dma(struct pci_dev *dev, ide_pci_device_t *d, ide_hwif_t *hwif)
+
+static void ide_hwif_setup_dma(struct pci_dev *dev, const struct ide_port_info *d, ide_hwif_t *hwif)
 {
+#ifdef CONFIG_BLK_DEV_IDEDMA_PCI
        u16 pcicmd;
 
        pci_read_config_word(dev, PCI_COMMAND, &pcicmd);
@@ -460,13 +458,13 @@ static void ide_hwif_setup_dma(struct pci_dev *dev, ide_pci_device_t *d, ide_hwi
                                "(BIOS)\n", hwif->name, d->name);
                }
        }
-}
 #endif /* CONFIG_BLK_DEV_IDEDMA_PCI*/
+}
 
 /**
  *     ide_setup_pci_controller        -       set up IDE PCI
  *     @dev: PCI device
- *     @d: IDE PCI data
+ *     @d: IDE port info
  *     @noisy: verbose flag
  *     @config: returned as 1 if we configured the hardware
  *
@@ -474,8 +472,8 @@ static void ide_hwif_setup_dma(struct pci_dev *dev, ide_pci_device_t *d, ide_hwi
  *     up the PCI side of the device, checks that the device is enabled
  *     and enables it if need be
  */
-static int ide_setup_pci_controller(struct pci_dev *dev, ide_pci_device_t *d, int noisy, int *config)
+
+static int ide_setup_pci_controller(struct pci_dev *dev, const struct ide_port_info *d, int noisy, int *config)
 {
        int ret;
        u16 pcicmd;
@@ -500,9 +498,6 @@ static int ide_setup_pci_controller(struct pci_dev *dev, ide_pci_device_t *d, in
                printk(KERN_INFO "%s: device enabled (Linux)\n", d->name);
        }
 
-       if (noisy)
-               printk(KERN_INFO "%s: chipset revision %d\n",
-                                d->name, dev->revision);
 out:
        return ret;
 }
@@ -510,9 +505,9 @@ out:
 /**
  *     ide_pci_setup_ports     -       configure ports/devices on PCI IDE
  *     @dev: PCI device
- *     @d: IDE pci device info
+ *     @d: IDE port info
  *     @pciirq: IRQ line
- *     @index: ata index to update
+ *     @idx: ATA index table to update
  *
  *     Scan the interfaces attached to this device and do any
  *     necessary per port setup. Attach the devices and ask the
@@ -522,26 +517,25 @@ out:
  *     but is also used directly as a helper function by some controllers
  *     where the chipset setup is not the default PCI IDE one.
  */
-void ide_pci_setup_ports(struct pci_dev *dev, ide_pci_device_t *d, int pciirq, ata_index_t *index)
+
+void ide_pci_setup_ports(struct pci_dev *dev, const struct ide_port_info *d, int pciirq, u8 *idx)
 {
        int channels = (d->host_flags & IDE_HFLAG_SINGLE) ? 1 : 2, port;
-       int at_least_one_hwif_enabled = 0;
        ide_hwif_t *hwif, *mate = NULL;
        u8 tmp;
 
-       index->all = 0xf0f0;
-
        /*
         * Set up the IDE ports
         */
-        
+
        for (port = 0; port < channels; ++port) {
-               ide_pci_enablebit_t *e = &(d->enablebits[port]);
-       
+               const ide_pci_enablebit_t *e = &(d->enablebits[port]);
+
                if (e->reg && (pci_read_config_byte(dev, e->reg, &tmp) ||
-                   (tmp & e->mask) != e->val))
+                   (tmp & e->mask) != e->val)) {
+                       printk(KERN_INFO "%s: IDE port disabled\n", d->name);
                        continue;       /* port not enabled */
+               }
 
                if ((hwif = ide_hwif_configure(dev, d, mate, port, pciirq)) == NULL)
                        continue;
@@ -549,11 +543,7 @@ void ide_pci_setup_ports(struct pci_dev *dev, ide_pci_device_t *d, int pciirq, a
                /* setup proper ancestral information */
                hwif->gendev.parent = &dev->dev;
 
-               if (hwif->channel) {
-                       index->b.high = hwif->index;
-               } else {
-                       index->b.low = hwif->index;
-               }
+               *(idx + port) = hwif->index;
 
                
                if (d->init_iops)
@@ -562,15 +552,40 @@ void ide_pci_setup_ports(struct pci_dev *dev, ide_pci_device_t *d, int pciirq, a
                if ((d->host_flags & IDE_HFLAG_NO_DMA) == 0)
                        ide_hwif_setup_dma(dev, d, hwif);
 
+               if ((!hwif->irq && (d->host_flags & IDE_HFLAG_LEGACY_IRQS)) ||
+                   (d->host_flags & IDE_HFLAG_FORCE_LEGACY_IRQS))
+                       hwif->irq = port ? 15 : 14;
+
+               hwif->fixup = d->fixup;
+
                hwif->host_flags = d->host_flags;
                hwif->pio_mask = d->pio_mask;
 
+               if ((d->host_flags & IDE_HFLAG_SERIALIZE) && hwif->mate)
+                       hwif->mate->serialized = hwif->serialized = 1;
+
+               if (d->host_flags & IDE_HFLAG_IO_32BIT) {
+                       hwif->drives[0].io_32bit = 1;
+                       hwif->drives[1].io_32bit = 1;
+               }
+
+               if (d->host_flags & IDE_HFLAG_UNMASK_IRQS) {
+                       hwif->drives[0].unmask = 1;
+                       hwif->drives[1].unmask = 1;
+               }
+
                if (hwif->dma_base) {
                        hwif->swdma_mask = d->swdma_mask;
                        hwif->mwdma_mask = d->mwdma_mask;
                        hwif->ultra_mask = d->udma_mask;
                }
 
+               hwif->drives[0].autotune = 1;
+               hwif->drives[1].autotune = 1;
+
+               if (d->host_flags & IDE_HFLAG_RQSIZE_256)
+                       hwif->rqsize = 256;
+
                if (d->init_hwif)
                        /* Call chipset-specific routine
                         * for each enabled hwif
@@ -578,10 +593,7 @@ void ide_pci_setup_ports(struct pci_dev *dev, ide_pci_device_t *d, int pciirq, a
                        d->init_hwif(hwif);
 
                mate = hwif;
-               at_least_one_hwif_enabled = 1;
        }
-       if (!at_least_one_hwif_enabled)
-               printk(KERN_INFO "%s: neither IDE port enabled (BIOS)\n", d->name);
 }
 
 EXPORT_SYMBOL_GPL(ide_pci_setup_ports);
@@ -593,13 +605,13 @@ EXPORT_SYMBOL_GPL(ide_pci_setup_ports);
  *
  * One thing that is not standardized is the location of the
  * primary/secondary interface "enable/disable" bits.  For chipsets that
- * we "know" about, this information is in the ide_pci_device_t struct;
+ * we "know" about, this information is in the struct ide_port_info;
  * for all other chipsets, we just assume both interfaces are enabled.
  */
-static int do_ide_setup_pci_device(struct pci_dev *dev, ide_pci_device_t *d,
-                                  ata_index_t *index, u8 noisy)
+static int do_ide_setup_pci_device(struct pci_dev *dev,
+                                  const struct ide_port_info *d,
+                                  u8 *idx, u8 noisy)
 {
-       static ata_index_t ata_index = { .b = { .low = 0xff, .high = 0xff } };
        int tried_config = 0;
        int pciirq, ret;
 
@@ -649,51 +661,35 @@ static int do_ide_setup_pci_device(struct pci_dev *dev, ide_pci_device_t *d,
 
        /* FIXME: silent failure can happen */
 
-       *index = ata_index;
-       ide_pci_setup_ports(dev, d, pciirq, index);
+       ide_pci_setup_ports(dev, d, pciirq, idx);
 out:
        return ret;
 }
 
-int ide_setup_pci_device(struct pci_dev *dev, ide_pci_device_t *d)
+int ide_setup_pci_device(struct pci_dev *dev, const struct ide_port_info *d)
 {
-       ide_hwif_t *hwif = NULL, *mate = NULL;
-       ata_index_t index_list;
+       u8 idx[4] = { 0xff, 0xff, 0xff, 0xff };
        int ret;
 
-       ret = do_ide_setup_pci_device(dev, d, &index_list, 1);
-       if (ret < 0)
-               goto out;
-
-       if ((index_list.b.low & 0xf0) != 0xf0)
-               hwif = &ide_hwifs[index_list.b.low];
-       if ((index_list.b.high & 0xf0) != 0xf0)
-               mate = &ide_hwifs[index_list.b.high];
+       ret = do_ide_setup_pci_device(dev, d, &idx[0], 1);
 
-       if (hwif)
-               probe_hwif_init_with_fixup(hwif, d->fixup);
-       if (mate)
-               probe_hwif_init_with_fixup(mate, d->fixup);
+       if (ret >= 0)
+               ide_device_add(idx);
 
-       if (hwif)
-               ide_proc_register_port(hwif);
-       if (mate)
-               ide_proc_register_port(mate);
-out:
        return ret;
 }
 
 EXPORT_SYMBOL_GPL(ide_setup_pci_device);
 
 int ide_setup_pci_devices(struct pci_dev *dev1, struct pci_dev *dev2,
-                         ide_pci_device_t *d)
+                         const struct ide_port_info *d)
 {
        struct pci_dev *pdev[] = { dev1, dev2 };
-       ata_index_t index_list[2];
        int ret, i;
+       u8 idx[4] = { 0xff, 0xff, 0xff, 0xff };
 
        for (i = 0; i < 2; i++) {
-               ret = do_ide_setup_pci_device(pdev[i], d, index_list + i, !i);
+               ret = do_ide_setup_pci_device(pdev[i], d, &idx[i*2], !i);
                /*
                 * FIXME: Mom, mom, they stole me the helper function to undo
                 * do_ide_setup_pci_device() on the first device!
@@ -702,25 +698,7 @@ int ide_setup_pci_devices(struct pci_dev *dev1, struct pci_dev *dev2,
                        goto out;
        }
 
-       for (i = 0; i < 2; i++) {
-               u8 idx[2] = { index_list[i].b.low, index_list[i].b.high };
-               int j;
-
-               for (j = 0; j < 2; j++) {
-                       if ((idx[j] & 0xf0) != 0xf0)
-                               probe_hwif_init(ide_hwifs + idx[j]);
-               }
-       }
-
-       for (i = 0; i < 2; i++) {
-               u8 idx[2] = { index_list[i].b.low, index_list[i].b.high };
-               int j;
-
-               for (j = 0; j < 2; j++) {
-                       if ((idx[j] & 0xf0) != 0xf0)
-                               ide_proc_register_port(ide_hwifs + idx[j]);
-               }
-       }
+       ide_device_add(idx);
 out:
        return ret;
 }
@@ -745,9 +723,6 @@ static LIST_HEAD(ide_pci_drivers);
  *     hands the controllers off to the core PCI code to do the rest of
  *     the work.
  *
- *     The driver_data of the driver table must point to an ide_pci_device_t
- *     describing the interface.
- *
  *     Returns are the same as for pci_register_driver
  */