On 2017-05-31 22:43, Stephen Boyd wrote:
On 05/30, Kiran Gunda wrote:
Currently the SPMI interrupt will not wake the device. Enable this
interrupt as a wakeup source.

Signed-off-by: Nicholas Troast <[email protected]>
Signed-off-by: Kiran Gunda <[email protected]>
---
 drivers/spmi/spmi-pmic-arb.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/spmi/spmi-pmic-arb.c b/drivers/spmi/spmi-pmic-arb.c
index 0deac33..2afe359 100644
--- a/drivers/spmi/spmi-pmic-arb.c
+++ b/drivers/spmi/spmi-pmic-arb.c
@@ -1140,6 +1140,7 @@ static int spmi_pmic_arb_probe(struct platform_device *pdev)
        }

        irq_set_chained_handler_and_data(pa->irq, pmic_arb_chained_irq, pa);
+       enable_irq_wake(pa->irq);

Why don't we do this through an irq_set_wake callback in the
irqchip? That way, we don't mark this irq as wakeup if any child
irqs aren't marked as wakeup.
Yes. This looks cleaner. Will change it in the subsequent patch.

Reply via email to