staging: rtl8723bs: remove unnecessary braces from conditionals
authorShreyansh Chouhan <chouhan.shreyansh630@gmail.com>
Mon, 24 May 2021 13:51:04 +0000 (19:21 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 25 May 2021 16:25:21 +0000 (18:25 +0200)
Removed the braces from if else statements in core/rtw_wlan_util.c since
the previous commit (6a257dd6de516573) caused all conditional blocks to
have a single statement in the function check_assoc_AP.

Signed-off-by: Shreyansh Chouhan <chouhan.shreyansh630@gmail.com>
Link: https://lore.kernel.org/r/20210524135105.5550-3-chouhan.shreyansh630@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_wlan_util.c

index 36e515a7ab5c44f1b4ef465422beab42f4bd486c..dd965c810967b040cb7fc6de42b4547df3779c4f 100644 (file)
@@ -1532,25 +1532,24 @@ unsigned char check_assoc_AP(u8 *pframe, uint len)
 
                switch (pIE->ElementID) {
                case WLAN_EID_VENDOR_SPECIFIC:
-                       if ((!memcmp(pIE->data, ARTHEROS_OUI1, 3)) || (!memcmp(pIE->data, ARTHEROS_OUI2, 3))) {
+                       if ((!memcmp(pIE->data, ARTHEROS_OUI1, 3)) || (!memcmp(pIE->data, ARTHEROS_OUI2, 3)))
                                return HT_IOT_PEER_ATHEROS;
-                       } else if ((!memcmp(pIE->data, BROADCOM_OUI1, 3)) ||
-                                  (!memcmp(pIE->data, BROADCOM_OUI2, 3)) ||
-                                  (!memcmp(pIE->data, BROADCOM_OUI3, 3))) {
-                               return HT_IOT_PEER_BROADCOM;
-                       } else if (!memcmp(pIE->data, MARVELL_OUI, 3)) {
-                               return HT_IOT_PEER_MARVELL;
-                       } else if (!memcmp(pIE->data, RALINK_OUI, 3)) {
-                               return HT_IOT_PEER_RALINK;
-                       } else if (!memcmp(pIE->data, CISCO_OUI, 3)) {
-                               return HT_IOT_PEER_CISCO;
-                       } else if (!memcmp(pIE->data, REALTEK_OUI, 3)) {
-                               return get_realtek_assoc_AP_vender(pIE);
-                       } else if (!memcmp(pIE->data, AIRGOCAP_OUI, 3)) {
-                               return HT_IOT_PEER_AIRGO;
-                       } else {
-                               break;
-                       }
+                       else if ((!memcmp(pIE->data, BROADCOM_OUI1, 3)) ||
+                                (!memcmp(pIE->data, BROADCOM_OUI2, 3)) ||
+                                (!memcmp(pIE->data, BROADCOM_OUI3, 3)))
+                             return HT_IOT_PEER_BROADCOM;
+                       else if (!memcmp(pIE->data, MARVELL_OUI, 3))
+                             return HT_IOT_PEER_MARVELL;
+                       else if (!memcmp(pIE->data, RALINK_OUI, 3))
+                             return HT_IOT_PEER_RALINK;
+                       else if (!memcmp(pIE->data, CISCO_OUI, 3))
+                             return HT_IOT_PEER_CISCO;
+                       else if (!memcmp(pIE->data, REALTEK_OUI, 3))
+                             return get_realtek_assoc_AP_vender(pIE);
+                       else if (!memcmp(pIE->data, AIRGOCAP_OUI, 3))
+                             return HT_IOT_PEER_AIRGO;
+                       else
+                             break;
 
                default:
                        break;