From: Justin Chen <[email protected]>

Utilize the Broadcom interrupt controller standard property
"brcm,irq-can-wake" to flag whether this particular interrupt controller
instance is wake-up capable.

Since we do not know what type of parent interrupt controller we are
interfaced with, ensure that enable_irq_wake() is called early on.

Signed-off-by: Justin Chen <[email protected]>
Signed-off-by: Florian Fainelli <[email protected]>
---
 drivers/irqchip/irq-bcm7120-l2.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/irqchip/irq-bcm7120-l2.c b/drivers/irqchip/irq-bcm7120-l2.c
index 586df3587be0..c7c9e976acbb 100644
--- a/drivers/irqchip/irq-bcm7120-l2.c
+++ b/drivers/irqchip/irq-bcm7120-l2.c
@@ -143,6 +143,9 @@ static int bcm7120_l2_intc_init_one(struct device_node *dn,
 
        irq_set_chained_handler_and_data(parent_irq,
                                         bcm7120_l2_intc_irq_handle, l1_data);
+       if (data->can_wake)
+               enable_irq_wake(parent_irq);
+
        return 0;
 }
 
@@ -247,6 +250,8 @@ static int __init bcm7120_l2_intc_probe(struct device_node 
*dn,
        if (ret < 0)
                goto out_free_l1_data;
 
+       data->can_wake = of_property_read_bool(dn, "brcm,irq-can-wake");
+
        for (irq = 0; irq < data->num_parent_irqs; irq++) {
                ret = bcm7120_l2_intc_init_one(dn, data, irq, valid_mask);
                if (ret)
@@ -274,9 +279,6 @@ static int __init bcm7120_l2_intc_probe(struct device_node 
*dn,
                goto out_free_domain;
        }
 
-       if (of_property_read_bool(dn, "brcm,irq-can-wake"))
-               data->can_wake = true;
-
        for (idx = 0; idx < data->n_words; idx++) {
                irq = idx * IRQS_PER_WORD;
                gc = irq_get_domain_generic_chip(data->domain, irq);
-- 
2.17.1

Reply via email to