Commit d356cf5a authored by Russell King - ARM Linux's avatar Russell King - ARM Linux Committed by Jason Cooper

Dove: Fix irq_to_pmu()

PMU interrupts start at IRQ_DOVE_PMU_START, not IRQ_DOVE_PMU_START + 1.
Fix the condition.  (It may have been less likely to occur had the code
been written "if (irq >= IRQ_DOVE_PMU_START" which imho is the easier
to understand notation, and matches the normal way of thinking about
these things.)
Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
Cc: stable@vger.kernel.org
Signed-off-by: default avatarJason Cooper <jason@lakedaemon.net>
parent 5d3df935
...@@ -63,7 +63,7 @@ static inline int pmu_to_irq(int pin) ...@@ -63,7 +63,7 @@ static inline int pmu_to_irq(int pin)
static inline int irq_to_pmu(int irq) static inline int irq_to_pmu(int irq)
{ {
if (IRQ_DOVE_PMU_START < irq && irq < NR_IRQS) if (IRQ_DOVE_PMU_START <= irq && irq < NR_IRQS)
return irq - IRQ_DOVE_PMU_START; return irq - IRQ_DOVE_PMU_START;
return -EINVAL; return -EINVAL;
......
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