genirq: pm: Fix the enable ordering in resume

In suspend interrupts are disabled from 0 to NR_IRQ, in resume interrupts
should be enabled in reverse order.

Enabling parent or summary interrupts before enabling child interrupts
causes the handler of the child interrupt to run even before it is
enabled. Usually the genirq handler does the correct thing of masking
the interrupt and additionally marking the interrupt IRQ_PENDING if its
an edge triggered interrupt. However the nested handler
(handle_nested_irq()) simply ignores the interrupt causing a loss of it.

Not calling the action of an interrupt, especially if it marked wakeup,
causes the system to incorrectly go back to suspend immediately.

Change-Id: Ica30c10a975a4a7b41b97b4f21250dac80335b2b
Signed-off-by: Abhijeet Dharmapurikar <adharmap@codeaurora.org>
(cherry picked from commit 6dfcdc120d05d041e38668d15fd041fb7803986d)
This commit is contained in:
Abhijeet Dharmapurikar 2010-12-19 16:23:48 -08:00 committed by Rohit Vaswani
parent ee004b5221
commit 445abc7500
1 changed files with 1 additions and 1 deletions

View File

@ -45,7 +45,7 @@ static void resume_irqs(bool want_early)
struct irq_desc *desc;
int irq;
for_each_irq_desc(irq, desc) {
for_each_irq_desc_reverse(irq, desc) {
unsigned long flags;
bool is_early = desc->action &&
desc->action->flags & IRQF_EARLY_RESUME;