local/net: bcm4358: fix misleading-indentation build warning 47/137947/1 accepted/tizen/4.0/unified/20170816.012918 accepted/tizen/unified/20170711.180931 submit/tizen/20170711.010212 submit/tizen_4.0/20170811.094300
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Mon, 10 Jul 2017 10:33:46 +0000 (19:33 +0900)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Mon, 10 Jul 2017 10:33:48 +0000 (19:33 +0900)
This patch fixes following build warning caused by wrong indent:

   drivers/net/wireless/bcmdhd4358/wl_cfg80211.c: In function 'wl_cfg80211_mgmt_tx':
   drivers/net/wireless/bcmdhd4358/wl_cfg80211.c:6345:4: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
       if ((dev == bcmcfg_to_prmry_ndev(cfg)) && cfg->p2p)
       ^~
   drivers/net/wireless/bcmdhd4358/wl_cfg80211.c:6347:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
        wl_cfgp2p_set_management_ie(cfg, dev, bssidx,
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Change-Id: I6ab0087a11b9a1a2be05211ba0ef751ff8f564b7
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
drivers/net/wireless/bcmdhd4358/wl_cfg80211.c

index 0cfe27c..ad3c16a 100644 (file)
@@ -6344,7 +6344,7 @@ wl_cfg80211_mgmt_tx(struct wiphy *wiphy, bcm_struct_cfgdev *cfgdev,
                        s32 ie_len = len - ie_offset;
                        if ((dev == bcmcfg_to_prmry_ndev(cfg)) && cfg->p2p)
                                bssidx = wl_to_p2p_bss_bssidx(cfg, P2PAPI_BSSCFG_DEVICE);
-                               wl_cfgp2p_set_management_ie(cfg, dev, bssidx,
+                       wl_cfgp2p_set_management_ie(cfg, dev, bssidx,
                                VNDR_IE_PRBRSP_FLAG, (u8 *)(buf + ie_offset), ie_len);
                        cfg80211_mgmt_tx_status(cfgdev, *cookie, buf, len, true, GFP_KERNEL);
                        goto exit;