Commit 7f94226f authored by Thomas Gleixner's avatar Thomas Gleixner

genirq: Move wakeup state to irq_data

Some irq_chips need to know the state of wakeup mode for
setting the trigger type etc. Reflect it in irq_data state.
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent d4d5e089
...@@ -168,6 +168,8 @@ struct irq_data { ...@@ -168,6 +168,8 @@ struct irq_data {
* IRQD_PER_CPU - Interrupt is per cpu * IRQD_PER_CPU - Interrupt is per cpu
* IRQD_AFFINITY_SET - Interrupt affinity was set * IRQD_AFFINITY_SET - Interrupt affinity was set
* IRQD_LEVEL - Interrupt is level triggered * IRQD_LEVEL - Interrupt is level triggered
* IRQD_WAKEUP_STATE - Interrupt is configured for wakeup
* from suspend
*/ */
enum { enum {
IRQD_TRIGGER_MASK = 0xf, IRQD_TRIGGER_MASK = 0xf,
...@@ -176,6 +178,7 @@ enum { ...@@ -176,6 +178,7 @@ enum {
IRQD_PER_CPU = (1 << 11), IRQD_PER_CPU = (1 << 11),
IRQD_AFFINITY_SET = (1 << 12), IRQD_AFFINITY_SET = (1 << 12),
IRQD_LEVEL = (1 << 13), IRQD_LEVEL = (1 << 13),
IRQD_WAKEUP_STATE = (1 << 14),
}; };
static inline bool irqd_is_setaffinity_pending(struct irq_data *d) static inline bool irqd_is_setaffinity_pending(struct irq_data *d)
...@@ -217,6 +220,11 @@ static inline bool irqd_is_level_type(struct irq_data *d) ...@@ -217,6 +220,11 @@ static inline bool irqd_is_level_type(struct irq_data *d)
return d->state_use_accessors & IRQD_LEVEL; return d->state_use_accessors & IRQD_LEVEL;
} }
static inline bool irqd_is_wakeup_set(struct irq_data *d)
{
return d->state_use_accessors & IRQD_WAKEUP_STATE;
}
/** /**
* struct irq_chip - hardware interrupt chip descriptor * struct irq_chip - hardware interrupt chip descriptor
* *
......
...@@ -46,7 +46,6 @@ enum { ...@@ -46,7 +46,6 @@ enum {
* IRQS_PENDING - irq is pending and replayed later * IRQS_PENDING - irq is pending and replayed later
* IRQS_MASKED - irq is masked * IRQS_MASKED - irq is masked
* IRQS_SUSPENDED - irq is suspended * IRQS_SUSPENDED - irq is suspended
* IRQS_WAKEUP - irq triggers system wakeup from suspend
*/ */
enum { enum {
IRQS_AUTODETECT = 0x00000001, IRQS_AUTODETECT = 0x00000001,
...@@ -60,7 +59,6 @@ enum { ...@@ -60,7 +59,6 @@ enum {
IRQS_PENDING = 0x00000200, IRQS_PENDING = 0x00000200,
IRQS_MASKED = 0x00000400, IRQS_MASKED = 0x00000400,
IRQS_SUSPENDED = 0x00000800, IRQS_SUSPENDED = 0x00000800,
IRQS_WAKEUP = 0x00001000,
}; };
#include "compat.h" #include "compat.h"
......
...@@ -492,7 +492,7 @@ int irq_set_irq_wake(unsigned int irq, unsigned int on) ...@@ -492,7 +492,7 @@ int irq_set_irq_wake(unsigned int irq, unsigned int on)
if (ret) if (ret)
desc->wake_depth = 0; desc->wake_depth = 0;
else else
desc->istate |= IRQS_WAKEUP; irqd_set(&desc->irq_data, IRQD_WAKEUP_STATE);
} }
} else { } else {
if (desc->wake_depth == 0) { if (desc->wake_depth == 0) {
...@@ -502,7 +502,7 @@ int irq_set_irq_wake(unsigned int irq, unsigned int on) ...@@ -502,7 +502,7 @@ int irq_set_irq_wake(unsigned int irq, unsigned int on)
if (ret) if (ret)
desc->wake_depth = 1; desc->wake_depth = 1;
else else
desc->istate &= ~IRQS_WAKEUP; irqd_clear(&desc->irq_data, IRQD_WAKEUP_STATE);
} }
} }
......
...@@ -69,7 +69,7 @@ int check_wakeup_irqs(void) ...@@ -69,7 +69,7 @@ int check_wakeup_irqs(void)
int irq; int irq;
for_each_irq_desc(irq, desc) for_each_irq_desc(irq, desc)
if ((desc->istate & IRQS_WAKEUP) && if (irqd_is_wakeup_set(&desc->irq_data) &&
(desc->istate & IRQS_PENDING)) (desc->istate & IRQS_PENDING))
return -EBUSY; return -EBUSY;
......
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