staging: wilc1000: Removed braces from single block statements
authorRoger H. Newell <newell.roger@gmail.com>
Sat, 19 Mar 2016 15:19:21 +0000 (12:49 -0230)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 28 Mar 2016 14:30:36 +0000 (07:30 -0700)
This patch corrects warnings generated by checkpatch.pl by
removing braces from single block statements.

Signed-ff-by: Roger H. Newell <newell.roger@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
drivers/staging/wilc1000/wilc_wlan_cfg.c

index 448a5c8c451460756b7cfbd4dbff757fa48e03a1..70625f8a2e3c35f2d1f766e02461744f7960f9c5 100644 (file)
@@ -558,11 +558,11 @@ static void CfgConnectResult(enum conn_event enuConnDisconnEvent,
 
                if (!pstrWFIDrv->p2p_connect)
                        wlan_channel = INVALID_CHANNEL;
-               if ((pstrWFIDrv->IFC_UP) && (dev == wl->vif[1]->ndev)) {
+               if ((pstrWFIDrv->IFC_UP) && (dev == wl->vif[1]->ndev))
                        pstrDisconnectNotifInfo->reason = 3;
-               } else if ((!pstrWFIDrv->IFC_UP) && (dev == wl->vif[1]->ndev)) {
+               else if ((!pstrWFIDrv->IFC_UP) && (dev == wl->vif[1]->ndev))
                        pstrDisconnectNotifInfo->reason = 1;
-               }
+
                cfg80211_disconnected(dev, pstrDisconnectNotifInfo->reason, pstrDisconnectNotifInfo->ie,
                                      pstrDisconnectNotifInfo->ie_len, false,
                                      GFP_KERNEL);
@@ -739,18 +739,15 @@ static int connect(struct wiphy *wiphy, struct net_device *dev,
                        wilc_add_wep_key_bss_sta(vif, sme->key, sme->key_len,
                                                 sme->key_idx);
                } else if (sme->crypto.wpa_versions & NL80211_WPA_VERSION_2)   {
-                       if (sme->crypto.cipher_group == WLAN_CIPHER_SUITE_TKIP) {
+                       if (sme->crypto.cipher_group == WLAN_CIPHER_SUITE_TKIP)
                                u8security = ENCRYPT_ENABLED | WPA2 | TKIP;
-                       } else {
+                       else
                                u8security = ENCRYPT_ENABLED | WPA2 | AES;
-                       }
                } else if (sme->crypto.wpa_versions & NL80211_WPA_VERSION_1)   {
-                       if (sme->crypto.cipher_group == WLAN_CIPHER_SUITE_TKIP) {
+                       if (sme->crypto.cipher_group == WLAN_CIPHER_SUITE_TKIP)
                                u8security = ENCRYPT_ENABLED | WPA | TKIP;
-                       } else {
+                       else
                                u8security = ENCRYPT_ENABLED | WPA | AES;
-                       }
-
                } else {
                        s32Error = -ENOTSUPP;
                        netdev_err(dev, "Not supported cipher\n");
@@ -762,11 +759,10 @@ static int connect(struct wiphy *wiphy, struct net_device *dev,
        if ((sme->crypto.wpa_versions & NL80211_WPA_VERSION_1)
            || (sme->crypto.wpa_versions & NL80211_WPA_VERSION_2)) {
                for (i = 0; i < sme->crypto.n_ciphers_pairwise; i++) {
-                       if (sme->crypto.ciphers_pairwise[i] == WLAN_CIPHER_SUITE_TKIP) {
+                       if (sme->crypto.ciphers_pairwise[i] == WLAN_CIPHER_SUITE_TKIP)
                                u8security = u8security | TKIP;
-                       } else {
+                       else
                                u8security = u8security | AES;
-                       }
                }
        }
 
@@ -1355,9 +1351,8 @@ static void WILC_WFI_CfgParseRxAction(u8 *buf, u32 len)
        u8 channel_list_attr_index = 0;
 
        while (index < len) {
-               if (buf[index] == GO_INTENT_ATTR_ID) {
+               if (buf[index] == GO_INTENT_ATTR_ID)
                        buf[index + 3] = (buf[index + 3]  & 0x01) | (0x00 << 1);
-               }
 
                if (buf[index] ==  CHANLIST_ATTR_ID)
                        channel_list_attr_index = index;
@@ -1369,9 +1364,8 @@ static void WILC_WFI_CfgParseRxAction(u8 *buf, u32 len)
                if (channel_list_attr_index) {
                        for (i = channel_list_attr_index + 3; i < ((channel_list_attr_index + 3) + buf[channel_list_attr_index + 1]); i++) {
                                if (buf[i] == 0x51) {
-                                       for (j = i + 2; j < ((i + 2) + buf[i + 1]); j++) {
+                                       for (j = i + 2; j < ((i + 2) + buf[i + 1]); j++)
                                                buf[j] = wlan_channel;
-                                       }
                                        break;
                                }
                        }
@@ -1409,9 +1403,8 @@ static void WILC_WFI_CfgParseTxAction(u8 *buf, u32 len, bool bOperChan, u8 iftyp
                if (channel_list_attr_index) {
                        for (i = channel_list_attr_index + 3; i < ((channel_list_attr_index + 3) + buf[channel_list_attr_index + 1]); i++) {
                                if (buf[i] == 0x51) {
-                                       for (j = i + 2; j < ((i + 2) + buf[i + 1]); j++) {
+                                       for (j = i + 2; j < ((i + 2) + buf[i + 1]); j++)
                                                buf[j] = wlan_channel;
-                                       }
                                        break;
                                }
                        }
index b3425b9cec946623556acfaad9f99757d5ec253b..b9711caac0da4e40e890bbeeeda2fd828063d433 100644 (file)
@@ -253,9 +253,8 @@ static int wilc_wlan_cfg_set_bin(u8 *frame, u32 offset, u16 id, u8 *b, u32 size)
 
        if ((b != NULL) && (size != 0)) {
                memcpy(&buf[4], b, size);
-               for (i = 0; i < size; i++) {
+               for (i = 0; i < size; i++)
                        checksum += buf[i + 4];
-               }
        }
 
        buf[size + 4] = checksum;