[PATCH] mfd: Flag WM831x /IRQ as a wake source

[PATCH] mfd: Flag WM831x /IRQ as a wake source

[Mark Brown]

The WM831x can generate wake events, some unconditionally, so flag
the primary IRQ as a wake source in order to help the CPU treat the
/IRQ signal appropriately.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
---
 drivers/mfd/wm831x-irq.c |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/drivers/mfd/wm831x-irq.c b/drivers/mfd/wm831x-irq.c
index ee88f6a..f7192d4 100644
--- a/drivers/mfd/wm831x-irq.c
+++ b/drivers/mfd/wm831x-irq.c
@@ -517,6 +517,17 @@ int wm831x_irq_init(struct wm831x *wm831x, int irq)
 		return 0;
 	}
 
+	/* Try to flag /IRQ as a wake source; there are a number of
+	 * unconditional wake sources in the PMIC so this isn't
+	 * conditional but we don't actually care *too* much if it
+	 * fails.
+	 */
+	ret = enable_irq_wake(irq);
+	if (ret != 0) {
+		dev_warn(wm831x->dev, "Can't enable IRQ as wake source: %d\n",
+			 ret);
+	}
+
 	wm831x->irq = irq;
 	wm831x->irq_base = pdata->irq_base;
 
-- 
1.7.2.3

Re: [PATCH] mfd: Flag WM831x /IRQ as a wake source

[Samuel Ortiz]

Hi Mark,

On Wed, Jan 05, 2011 at 05:56:01PM +0000, Mark Brown wrote:
> The WM831x can generate wake events, some unconditionally, so flag
> the primary IRQ as a wake source in order to help the CPU treat the
> /IRQ signal appropriately.
>
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
I applied this patch as well, thanks.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/