sata_via: fix resource-managed iomap conversion
Conversion to resource-managed iomap was buggy causing init failures on both vt6420 and 6421 - BAR5 wasn't mapped for both controllers while on vt6420 sata_via tried to map BAR0-4 twice. Fix it. Signed-off-by: Tejun Heo <htejun@gmail.com> Signed-off-by: Jeff Garzik <jeff@garzik.org>
This commit is contained in:
parent
c3c70c443c
commit
e1be5d73e0
|
@ -423,16 +423,21 @@ static struct ata_probe_ent *vt6420_init_probe_ent(struct pci_dev *pdev)
|
|||
{
|
||||
struct ata_probe_ent *probe_ent;
|
||||
struct ata_port_info *ppi[2];
|
||||
void __iomem * const *iomap;
|
||||
void __iomem *bar5;
|
||||
|
||||
ppi[0] = ppi[1] = &vt6420_port_info;
|
||||
probe_ent = ata_pci_init_native_mode(pdev, ppi, ATA_PORT_PRIMARY | ATA_PORT_SECONDARY);
|
||||
if (!probe_ent)
|
||||
return NULL;
|
||||
|
||||
iomap = pcim_iomap_table(pdev);
|
||||
probe_ent->port[0].scr_addr = svia_scr_addr(iomap[5], 0);
|
||||
probe_ent->port[1].scr_addr = svia_scr_addr(iomap[5], 1);
|
||||
bar5 = pcim_iomap(pdev, 5, 0);
|
||||
if (!bar5) {
|
||||
dev_printk(KERN_ERR, &pdev->dev, "failed to iomap PCI BAR 5\n");
|
||||
return NULL;
|
||||
}
|
||||
|
||||
probe_ent->port[0].scr_addr = svia_scr_addr(bar5, 0);
|
||||
probe_ent->port[1].scr_addr = svia_scr_addr(bar5, 1);
|
||||
|
||||
return probe_ent;
|
||||
}
|
||||
|
@ -460,6 +465,13 @@ static struct ata_probe_ent *vt6421_init_probe_ent(struct pci_dev *pdev)
|
|||
probe_ent->mwdma_mask = 0x07;
|
||||
probe_ent->udma_mask = 0x7f;
|
||||
|
||||
for (i = 0; i < 6; i++)
|
||||
if (!pcim_iomap(pdev, i, 0)) {
|
||||
dev_printk(KERN_ERR, &pdev->dev,
|
||||
"failed to iomap PCI BAR %d\n", i);
|
||||
return NULL;
|
||||
}
|
||||
|
||||
for (i = 0; i < N_PORTS; i++)
|
||||
vt6421_init_addrs(probe_ent, pcim_iomap_table(pdev), i);
|
||||
|
||||
|
@ -522,7 +534,7 @@ static int svia_init_one (struct pci_dev *pdev, const struct pci_device_id *ent)
|
|||
if (rc)
|
||||
return rc;
|
||||
|
||||
rc = pcim_iomap_regions(pdev, 0x1f, DRV_NAME);
|
||||
rc = pci_request_regions(pdev, DRV_NAME);
|
||||
if (rc) {
|
||||
pcim_pin_device(pdev);
|
||||
return rc;
|
||||
|
|
Loading…
Reference in New Issue