diff options
| author | David S. Miller <davem@sunset.davemloft.net> | 2006-07-13 02:59:53 +0400 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2006-07-13 12:50:08 +0400 |
| commit | a23c3a86e9952a8badb49a6bb1de455c1f5bad6d (patch) | |
| tree | 1c8a9b6df274b82cfb08c1e29ad34c1f6db71f28 /arch/sparc64 | |
| parent | 17556fe861d8f76ed13e975ceb6eae450671aba1 (diff) | |
| download | linux-a23c3a86e9952a8badb49a6bb1de455c1f5bad6d.tar.xz | |
[SPARC64]: Fix 2 bugs in sabre_irq_build()
When installing the IRQ pre-handler, we were not setting up the second
argument correctly. It should be a pointer to the sabre_irq_data, not
the config space PIO address.
Furthermore, we only need this pre-handler installed if the device
sits behind a PCI bridge that is not Sabre or Simba/APB.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch/sparc64')
| -rw-r--r-- | arch/sparc64/kernel/prom.c | 30 |
1 files changed, 26 insertions, 4 deletions
diff --git a/arch/sparc64/kernel/prom.c b/arch/sparc64/kernel/prom.c index 99daeee4209d..86cdbd4dbad5 100644 --- a/arch/sparc64/kernel/prom.c +++ b/arch/sparc64/kernel/prom.c @@ -539,6 +539,25 @@ static unsigned long __sabre_onboard_imap_off[] = { ((ino & 0x20) ? (SABRE_ICLR_SCSI + (((ino) & 0x1f) << 3)) : \ (SABRE_ICLR_A_SLOT0 + (((ino) & 0x1f)<<3))) +static int parent_is_sabre_or_simba(struct device_node *dp) +{ + char *parent_model, *parent_compat; + + parent_model = of_get_property(dp->parent, "model", NULL); + if (parent_model && + (!strcmp(parent_model, "SUNW,sabre") || + !strcmp(parent_model, "SUNW,simba"))) + return 1; + + parent_compat = of_get_property(dp->parent, "compatible", NULL); + if (parent_compat && + (!strcmp(parent_compat, "pci108e,a000") || + !strcmp(parent_compat, "pci108e,a001"))) + return 1; + + return 0; +} + static unsigned int sabre_irq_build(struct device_node *dp, unsigned int ino, void *_data) @@ -577,15 +596,18 @@ static unsigned int sabre_irq_build(struct device_node *dp, virt_irq = build_irq(inofixup, iclr, imap); + /* If the parent device is a PCI<->PCI bridge other than + * APB, we have to install a pre-handler to ensure that + * all pending DMA is drained before the interrupt handler + * is run. + */ regs = of_get_property(dp, "reg", NULL); if (regs && - ((regs->phys_hi >> 16) & 0xff) != irq_data->pci_first_busno) { + !parent_is_sabre_or_simba(dp)) { irq_install_pre_handler(virt_irq, sabre_wsync_handler, (void *) (long) regs->phys_hi, - (void *) - controller_regs + - SABRE_WRSYNC); + (void *) irq_data); } return virt_irq; |