[POWERPC] Do not write virq back to PCI config space
authorMichal Ostrowski <mostrows@watson.ibm.com>
Thu, 28 Dec 2006 04:14:43 +0000 (22:14 -0600)
committerPaul Mackerras <paulus@samba.org>
Wed, 24 Jan 2007 10:13:57 +0000 (21:13 +1100)
- Drivers will not rely on the PCI config space value, as they've
  already been conditioned to rely on the irq field in "struct pci_dev".

- The virq value may not be < 256 as it has been remapped.

- The PCI config space should reflect the hardware configuration, which
  is not being changed.  We are only creating a virtual irq mapping that
  exists in the kernel only. One would never expect the PCI hardware to
  generate the "virq" interrupt.

Signed-off-by: Michal Ostrowski <mostrows@watson.ibm.com>
Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/kernel/pci_32.c
arch/powerpc/kernel/pci_64.c

index c54f3639c5ade452a20af537b14e840647f24dfe..d8ef2e1005051c3408b5105a80b376c2f43d4d66 100644 (file)
@@ -1450,7 +1450,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev)
                return -1;
        }
        pci_dev->irq = virq;
-       pci_write_config_byte(pci_dev, PCI_INTERRUPT_LINE, virq);
 
        return 0;
 }
index 73c59ec4912059d5693f8026a95ff946c4701e9d..b6d08738180b997832890ffbd450f3c1e4df059e 100644 (file)
@@ -1325,7 +1325,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev)
        DBG(" -> mapped to linux irq %d\n", virq);
 
        pci_dev->irq = virq;
-       pci_write_config_byte(pci_dev, PCI_INTERRUPT_LINE, virq);
 
        return 0;
 }