[PATCH v3 2/4] staging: rtl8723bs: Remove unnecessary braces for single statements

[Wambui Karuga]

Clean up multiple unnecessary braces around single statement blocks in
drivers/staging/rtl8723bs/core/rtw_mlme.c
Issues reported by checkpatch.pl as:
WARNING: braces {} are not necessary for single statement blocks or
WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Wambui Karuga <wambui.karu...@gmail.com>
---
 drivers/staging/rtl8723bs/core/rtw_mlme.c | 37 ++++++++---------------
 1 file changed, 12 insertions(+), 25 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c 
b/drivers/staging/rtl8723bs/core/rtw_mlme.c
index b0018fe7bae3..52f490d5ebfb 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
@@ -112,9 +112,8 @@ void _rtw_free_mlme_priv(struct mlme_priv *pmlmepriv)
 {
        if (pmlmepriv) {
                rtw_free_mlme_priv_ie_data(pmlmepriv);
-               if (pmlmepriv->free_bss_buf) {
+               if (pmlmepriv->free_bss_buf)
                        vfree(pmlmepriv->free_bss_buf);
-               }
        }
 }
 
@@ -753,11 +752,10 @@ int rtw_is_desired_network(struct adapter *adapter, 
struct wlan_network *pnetwor
 
                if (psecuritypriv->ndisauthtype == Ndis802_11AuthModeWPA2PSK) {
                        p = rtw_get_ie(pnetwork->network.IEs + 
_BEACON_IE_OFFSET_, _RSN_IE_2_, &ie_len, (pnetwork->network.IELength - 
_BEACON_IE_OFFSET_));
-                       if (p && ie_len > 0) {
+                       if (p && ie_len > 0)
                                bselected = true;
-                       } else {
+                       else
                                bselected = false;
-                       }
                }
        }
 
@@ -822,9 +820,8 @@ void rtw_survey_event_callback(struct adapter       
*adapter, u8 *pbuf)
 
        /*  lock pmlmepriv->lock when you accessing network_q */
        if ((check_fwstate(pmlmepriv, _FW_UNDER_LINKING)) == false) {
-               if (pnetwork->Ssid.Ssid[0] == 0) {
+               if (pnetwork->Ssid.Ssid[0] == 0)
                        pnetwork->Ssid.SsidLength = 0;
-               }
                rtw_add_network(adapter, pnetwork);
        }
 
@@ -893,9 +890,8 @@ void rtw_surveydone_event_callback(struct adapter   
*adapter, u8 *pbuf)
 
                                        pmlmepriv->fw_state = 
WIFI_ADHOC_MASTER_STATE;
 
-                                       if (rtw_createbss_cmd(adapter) != 
_SUCCESS) {
-                                       RT_TRACE(_module_rtl871x_mlme_c_, 
_drv_err_, ("Error =>rtw_createbss_cmd status FAIL\n"));
-                                       }
+                                       if (rtw_createbss_cmd(adapter) != 
_SUCCESS)
+                                               
RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("Error =>rtw_createbss_cmd status 
FAIL\n"));
 
                                        pmlmepriv->to_join = false;
                                }
@@ -1166,9 +1162,8 @@ static struct sta_info *rtw_joinbss_update_stainfo(struct 
adapter *padapter, str
        struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
 
        psta = rtw_get_stainfo(pstapriv, pnetwork->network.MacAddress);
-       if (!psta) {
+       if (!psta)
                psta = rtw_alloc_stainfo(pstapriv, 
pnetwork->network.MacAddress);
-       }
 
        if (psta) { /* update ptarget_sta */
 
@@ -1347,11 +1342,10 @@ void rtw_joinbss_event_prehandle(struct adapter 
*adapter, u8 *pbuf)
        rtw_get_encrypt_decrypt_from_registrypriv(adapter);
 
 
-       if (pmlmepriv->assoc_ssid.SsidLength == 0) {
+       if (pmlmepriv->assoc_ssid.SsidLength == 0)
                RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("@@@@@   joinbss 
event call back  for Any SSid\n"));
-       } else {
+       else
                RT_TRACE(_module_rtl871x_mlme_c_, _drv_err_, ("@@@@@   
rtw_joinbss_event_callback for SSid:%s\n", pmlmepriv->assoc_ssid.Ssid));
-       }
 
        the_same_macaddr = !memcmp(pnetwork->network.MacAddress, 
cur_network->network.MacAddress, ETH_ALEN);
 
@@ -1723,13 +1717,8 @@ void rtw_stadel_event_callback(struct adapter *adapter, 
u8 *pbuf)
                                _clr_fwstate_(pmlmepriv, WIFI_ADHOC_STATE);
                        }
 
-                       if (rtw_createbss_cmd(adapter) != _SUCCESS) {
-
+                       if (rtw_createbss_cmd(adapter) != _SUCCESS)
                                RT_TRACE(_module_rtl871x_ioctl_set_c_, 
_drv_err_, ("***Error =>stadel_event_callback: rtw_createbss_cmd status 
FAIL***\n "));
-
-                       }
-
-
                }
 
        }
@@ -1902,9 +1891,8 @@ void rtw_dynamic_check_timer_handler(struct adapter 
*adapter)
                }
 
        } else {
-               if (is_primary_adapter(adapter)) {
+               if (is_primary_adapter(adapter))
                        rtw_dynamic_chk_wk_cmd(adapter);
-               }
        }
 
        /* auto site survey */
@@ -2988,9 +2976,8 @@ void rtw_append_exented_cap(struct adapter *padapter, u8 
*out_ie, uint *pout_len
        u8 *pframe;
 
 
-       if (phtpriv->bss_coexist) {
+       if (phtpriv->bss_coexist)
                SET_EXT_CAPABILITY_ELE_BSS_COEXIST(cap_content, 1);
-       }
 
        pframe = rtw_set_ie(out_ie + *pout_len, EID_EXTCapability, 8, 
cap_content, pout_len);
 }
-- 
2.23.0

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel