brcmfmac: fix curly brace mistake in brcmf_pcie_handle_mb_data()
authorArend van Spriel <arend@broadcom.com>
Thu, 7 Aug 2014 12:45:08 +0000 (14:45 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 11 Aug 2014 18:43:51 +0000 (14:43 -0400)
Running coccicheck on brcm80211 drivers resulted in following report:

$ make coccicheck MODE=report M=drivers/net/wireless/brcm80211

  drivers/net/wireless/brcm80211/brcmfmac/pcie.c:595:2-43:
    code aligned with following code on line 596

It revealed that due to a merge failure a block statement lost its
curly braces where it should not.

Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/pcie.c

index bc972c0..e5101b2 100644 (file)
@@ -591,12 +591,13 @@ static void brcmf_pcie_handle_mb_data(struct brcmf_pciedev_info *devinfo)
        }
        if (dtoh_mb_data & BRCMF_D2H_DEV_DS_EXIT_NOTE)
                brcmf_dbg(PCIE, "D2H_MB_DATA: DEEP SLEEP EXIT\n");
-       if (dtoh_mb_data & BRCMF_D2H_DEV_D3_ACK)
+       if (dtoh_mb_data & BRCMF_D2H_DEV_D3_ACK) {
                brcmf_dbg(PCIE, "D2H_MB_DATA: D3 ACK\n");
                if (waitqueue_active(&devinfo->mbdata_resp_wait)) {
                        devinfo->mbdata_completed = true;
                        wake_up(&devinfo->mbdata_resp_wait);
                }
+       }
 }