Commit 9a343b9e authored by Marc Zyngier's avatar Marc Zyngier Committed by Arnd Bergmann

ARM: tegra: irq: fix buggy usage of irq_data irq field

The crazy gic_arch_extn thing that Tegra uses contains multiple
references to the irq field in struct irq_data, and uses this
to directly poke hardware register.

But irq is the *virtual* irq number, something that has nothing
to do with the actual HW irq (stored in the hwirq field). And once
we put the stacked domain code in action, the whole thing explodes,
as these two values are *very* different:

root@bacon-fat:~# cat /proc/interrupts
            CPU0       CPU1
 16:      25801       2075       GIC  29  twd
 17:          0          0       GIC  73  timer0
112:          0          0      GPIO  58  c8000600.sdhci cd
123:          0          0      GPIO  69  c8000200.sdhci cd
279:       1126          0       GIC 122  serial
281:          0          0       GIC  70  7000c000.i2c
282:          0          0       GIC 116  7000c400.i2c
283:          0          0       GIC 124  7000c500.i2c
284:        300          0       GIC  85  7000d000.i2c
[...]

Just replacing all instances of irq with hwirq fixes the issue.
Signed-off-by: default avatarMarc Zyngier <marc.zyngier@arm.com>
Acked-by: default avatarThierry Reding <treding@nvidia.com>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
parent 5d01410f
...@@ -99,42 +99,42 @@ static inline void tegra_irq_write_mask(unsigned int irq, unsigned long reg) ...@@ -99,42 +99,42 @@ static inline void tegra_irq_write_mask(unsigned int irq, unsigned long reg)
static void tegra_mask(struct irq_data *d) static void tegra_mask(struct irq_data *d)
{ {
if (d->irq < FIRST_LEGACY_IRQ) if (d->hwirq < FIRST_LEGACY_IRQ)
return; return;
tegra_irq_write_mask(d->irq, ICTLR_CPU_IER_CLR); tegra_irq_write_mask(d->hwirq, ICTLR_CPU_IER_CLR);
} }
static void tegra_unmask(struct irq_data *d) static void tegra_unmask(struct irq_data *d)
{ {
if (d->irq < FIRST_LEGACY_IRQ) if (d->hwirq < FIRST_LEGACY_IRQ)
return; return;
tegra_irq_write_mask(d->irq, ICTLR_CPU_IER_SET); tegra_irq_write_mask(d->hwirq, ICTLR_CPU_IER_SET);
} }
static void tegra_ack(struct irq_data *d) static void tegra_ack(struct irq_data *d)
{ {
if (d->irq < FIRST_LEGACY_IRQ) if (d->hwirq < FIRST_LEGACY_IRQ)
return; return;
tegra_irq_write_mask(d->irq, ICTLR_CPU_IEP_FIR_CLR); tegra_irq_write_mask(d->hwirq, ICTLR_CPU_IEP_FIR_CLR);
} }
static void tegra_eoi(struct irq_data *d) static void tegra_eoi(struct irq_data *d)
{ {
if (d->irq < FIRST_LEGACY_IRQ) if (d->hwirq < FIRST_LEGACY_IRQ)
return; return;
tegra_irq_write_mask(d->irq, ICTLR_CPU_IEP_FIR_CLR); tegra_irq_write_mask(d->hwirq, ICTLR_CPU_IEP_FIR_CLR);
} }
static int tegra_retrigger(struct irq_data *d) static int tegra_retrigger(struct irq_data *d)
{ {
if (d->irq < FIRST_LEGACY_IRQ) if (d->hwirq < FIRST_LEGACY_IRQ)
return 0; return 0;
tegra_irq_write_mask(d->irq, ICTLR_CPU_IEP_FIR_SET); tegra_irq_write_mask(d->hwirq, ICTLR_CPU_IEP_FIR_SET);
return 1; return 1;
} }
...@@ -142,7 +142,7 @@ static int tegra_retrigger(struct irq_data *d) ...@@ -142,7 +142,7 @@ static int tegra_retrigger(struct irq_data *d)
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
static int tegra_set_wake(struct irq_data *d, unsigned int enable) static int tegra_set_wake(struct irq_data *d, unsigned int enable)
{ {
u32 irq = d->irq; u32 irq = d->hwirq;
u32 index, mask; u32 index, mask;
if (irq < FIRST_LEGACY_IRQ || if (irq < FIRST_LEGACY_IRQ ||
......
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