| Message ID | 1482863397-11400-1-git-send-email-stefan.wahren@i2se.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Tue, Dec 27, 2016 at 4:29 PM, Stefan Wahren <stefan.wahren@i2se.com> wrote: > The ICOLL controller doesn't provide any facility to configure the > wakeup sources. That's the reason why this implementation lacks > the irq_set_wake implementation. But this prevent us from properly > entering power management states like "suspend to idle". > > So enable the flags IRQCHIP_SKIP_SET_WAKE and > IRQCHIP_MASK_ON_SUSPEND to let the irqchip core allows and handles > the power management. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com> -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Stefan, On Tue, Dec 27, 2016 at 06:29:57PM +0000, Stefan Wahren wrote: > The ICOLL controller doesn't provide any facility to configure the > wakeup sources. That's the reason why this implementation lacks > the irq_set_wake implementation. But this prevent us from properly > entering power management states like "suspend to idle". > > So enable the flags IRQCHIP_SKIP_SET_WAKE and > IRQCHIP_MASK_ON_SUSPEND to let the irqchip core allows and handles > the power management. > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > --- > drivers/irqchip/irq-mxs.c | 4 ++++ > 1 file changed, 4 insertions(+) Applied to irqchip/urgent with Fabio's Reviewed-by. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/irqchip/irq-mxs.c b/drivers/irqchip/irq-mxs.c index 1730470..05fa9f7 100644 --- a/drivers/irqchip/irq-mxs.c +++ b/drivers/irqchip/irq-mxs.c @@ -131,12 +131,16 @@ static void asm9260_unmask_irq(struct irq_data *d) .irq_ack = icoll_ack_irq, .irq_mask = icoll_mask_irq, .irq_unmask = icoll_unmask_irq, + .flags = IRQCHIP_MASK_ON_SUSPEND | + IRQCHIP_SKIP_SET_WAKE, }; static struct irq_chip asm9260_icoll_chip = { .irq_ack = icoll_ack_irq, .irq_mask = asm9260_mask_irq, .irq_unmask = asm9260_unmask_irq, + .flags = IRQCHIP_MASK_ON_SUSPEND | + IRQCHIP_SKIP_SET_WAKE, }; asmlinkage void __exception_irq_entry icoll_handle_irq(struct pt_regs *regs)
The ICOLL controller doesn't provide any facility to configure the wakeup sources. That's the reason why this implementation lacks the irq_set_wake implementation. But this prevent us from properly entering power management states like "suspend to idle". So enable the flags IRQCHIP_SKIP_SET_WAKE and IRQCHIP_MASK_ON_SUSPEND to let the irqchip core allows and handles the power management. Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> --- drivers/irqchip/irq-mxs.c | 4 ++++ 1 file changed, 4 insertions(+)