From: Xinming Hu <h...@marvell.com>

If wakeup interrupt handler is called, we know that the wakeup
interrupt number is valid, there is no need to check it.

Signed-off-by: Xinming Hu <h...@marvell.com>
Signed-off-by: Cathy Luo <c...@marvell.com>
Reviewed-by: Dmitry Torokhov <d...@chromium.org>
Reviewed-by: Brian Norris <briannor...@chromium.org>
---
v2: modify description(Dimtry)
v4: same as v2, v3
---
 drivers/net/wireless/marvell/mwifiex/main.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/net/wireless/marvell/mwifiex/main.c 
b/drivers/net/wireless/marvell/mwifiex/main.c
index 0dfbac8..98fd491 100644
--- a/drivers/net/wireless/marvell/mwifiex/main.c
+++ b/drivers/net/wireless/marvell/mwifiex/main.c
@@ -1513,11 +1513,9 @@ static irqreturn_t mwifiex_irq_wakeup_handler(int irq, 
void *priv)
 {
        struct mwifiex_adapter *adapter = priv;
 
-       if (adapter->irq_wakeup >= 0) {
-               dev_dbg(adapter->dev, "%s: wake by wifi", __func__);
-               adapter->wake_by_wifi = true;
-               disable_irq_nosync(irq);
-       }
+       dev_dbg(adapter->dev, "%s: wake by wifi", __func__);
+       adapter->wake_by_wifi = true;
+       disable_irq_nosync(irq);
 
        /* Notify PM core we are wakeup source */
        pm_wakeup_event(adapter->dev, 0);
-- 
1.8.1.4

Reply via email to