Bluetooth: btmrvl: remove unnecessary wakeup interrupt number sanity check
authorXinming Hu <huxm@marvell.com>
Fri, 31 Mar 2017 06:32:32 +0000 (14:32 +0800)
committerMarcel Holtmann <marcel@holtmann.org>
Wed, 12 Apr 2017 20:02:40 +0000 (22:02 +0200)
Sanity check of interrupt number in interrupt handler is unnecessary and
confusion, remove it.

Reported-by: Guenter Roeck <groeck@chromium.org>
Signed-off-by: Xinming Hu <huxm@marvell.com>
Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/bluetooth/btmrvl_sdio.c

index 95e40ec27c0e3e0be151455aaabb2ced5773af56..eb794f08b2385bfcb2162531cee4fbaa62c818fd 100644 (file)
@@ -64,11 +64,9 @@ static irqreturn_t btmrvl_wake_irq_bt(int irq, void *priv)
        struct btmrvl_sdio_card *card = priv;
        struct btmrvl_plt_wake_cfg *cfg = card->plt_wake_cfg;
 
-       if (cfg->irq_bt >= 0) {
-               pr_info("%s: wake by bt", __func__);
-               cfg->wake_by_bt = true;
-               disable_irq_nosync(irq);
-       }
+       pr_info("%s: wake by bt", __func__);
+       cfg->wake_by_bt = true;
+       disable_irq_nosync(irq);
 
        pm_wakeup_event(&card->func->dev, 0);
        pm_system_wakeup();