Commit a2894cfb authored by Michal Ostrowski's avatar Michal Ostrowski Committed by Paul Mackerras

[POWERPC] Do not write virq back to PCI config space

- 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: default avatarMichal Ostrowski <mostrows@watson.ibm.com>
Acked-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 7e60d1b4
...@@ -1450,7 +1450,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev) ...@@ -1450,7 +1450,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev)
return -1; return -1;
} }
pci_dev->irq = virq; pci_dev->irq = virq;
pci_write_config_byte(pci_dev, PCI_INTERRUPT_LINE, virq);
return 0; return 0;
} }
......
...@@ -1325,7 +1325,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev) ...@@ -1325,7 +1325,6 @@ int pci_read_irq_line(struct pci_dev *pci_dev)
DBG(" -> mapped to linux irq %d\n", virq); DBG(" -> mapped to linux irq %d\n", virq);
pci_dev->irq = virq; pci_dev->irq = virq;
pci_write_config_byte(pci_dev, PCI_INTERRUPT_LINE, virq);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment