Staging: rtl8723bs: core: rtw_cmd: Fixed multiple brace coding style issues
authorManuel Palenzuela <manuelpalenzuelamerino@gmail.com>
Wed, 28 Oct 2020 02:17:16 +0000 (02:17 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 28 Oct 2020 08:51:19 +0000 (09:51 +0100)
Fixed multiple brace coding style issues in rtw_cmd.c

Signed-off-by: Manuel Palenzuela <manuelpalenzuelamerino@gmail.com>
Link: https://lore.kernel.org/r/20201028021716.622-1-manuelpalenzuelamerino@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_cmd.c

index 3ac286bbfc4e6b72253a4eb98c260b51efba7087..4cf09d947d32620adb134129e0473f36de5d8720 100644 (file)
@@ -229,9 +229,8 @@ void _rtw_free_evt_priv(struct      evt_priv *pevtpriv)
 
        while (!rtw_cbuf_empty(pevtpriv->c2h_queue)) {
                void *c2h = rtw_cbuf_pop(pevtpriv->c2h_queue);
-               if (c2h && c2h != (void *)pevtpriv) {
+               if (c2h && c2h != (void *)pevtpriv)
                        kfree(c2h);
-               }
        }
        kfree(pevtpriv->c2h_queue);
 
@@ -339,9 +338,8 @@ int rtw_enqueue_cmd(struct cmd_priv *pcmdpriv, struct cmd_obj *cmd_obj)
        int res = _FAIL;
        struct adapter *padapter = pcmdpriv->padapter;
 
-       if (cmd_obj == NULL) {
+       if (cmd_obj == NULL)
                goto exit;
-       }
 
        cmd_obj->padapter = padapter;
 
@@ -543,9 +541,8 @@ post_process:
 
                if (pcmd->cmdcode == GEN_CMD_CODE(_Set_Drv_Extra)) {
                        extra_parm = (struct drvextra_cmd_parm *)pcmd->parmbuf;
-                       if (extra_parm->pbuf && extra_parm->size > 0) {
+                       if (extra_parm->pbuf && extra_parm->size > 0)
                                kfree(extra_parm->pbuf);
-                       }
                }
 
                rtw_free_cmd_obj(pcmd);
@@ -571,9 +568,8 @@ u8 rtw_sitesurvey_cmd(struct adapter  *padapter, struct ndis_802_11_ssid *ssid,
        struct cmd_priv         *pcmdpriv = &padapter->cmdpriv;
        struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
 
-       if (check_fwstate(pmlmepriv, _FW_LINKED) == true) {
+       if (check_fwstate(pmlmepriv, _FW_LINKED) == true)
                rtw_lps_ctrl_wk_cmd(padapter, LPS_CTRL_SCAN, 1);
-       }
 
        ph2c = rtw_zmalloc(sizeof(struct cmd_obj));
        if (ph2c == NULL)
@@ -826,9 +822,8 @@ u8 rtw_joinbss_cmd(struct adapter  *padapter, struct wlan_network *pnetwork)
        /*  If not,  we have to copy the connecting AP's MAC address to it so that */
        /*  the driver just has the bssid information for PMKIDList searching. */
 
-       if (pmlmepriv->assoc_by_bssid == false) {
+       if (pmlmepriv->assoc_by_bssid == false)
                memcpy(&pmlmepriv->assoc_bssid[0], &pnetwork->network.MacAddress[0], ETH_ALEN);
-       }
 
        psecnetwork->IELength = rtw_restruct_sec_ie(padapter, &pnetwork->network.IEs[0], &psecnetwork->IEs[0], pnetwork->network.IELength);
 
@@ -1349,9 +1344,8 @@ u8 traffic_status_watchdog(struct adapter *padapter, u8 from_timer)
 
                                /* DBG_871X("Set TrafficTransitionCount to %d\n", pmlmepriv->LinkDetectInfo.TrafficTransitionCount); */
 
-                               if (pmlmepriv->LinkDetectInfo.TrafficTransitionCount > 30/*TrafficTransitionLevel*/) {
+                               if (pmlmepriv->LinkDetectInfo.TrafficTransitionCount > 30/*TrafficTransitionLevel*/)
                                        pmlmepriv->LinkDetectInfo.TrafficTransitionCount = 30;
-                               }
                        }
                } else {
                        /* DBG_871X("(+)Tx = %d, Rx = %d\n", pmlmepriv->LinkDetectInfo.NumTxOkInPeriod, pmlmepriv->LinkDetectInfo.NumRxUnicastOkInPeriod); */
@@ -1405,9 +1399,8 @@ static void dynamic_chk_wk_hdl(struct adapter *padapter)
        struct mlme_priv *pmlmepriv;
        pmlmepriv = &(padapter->mlmepriv);
 
-       if (check_fwstate(pmlmepriv, WIFI_AP_STATE) == true) {
+       if (check_fwstate(pmlmepriv, WIFI_AP_STATE) == true)
                expire_timeout_chk(padapter);
-       }
 
        /* for debug purpose */
        _linked_info_dump(padapter);
@@ -1606,9 +1599,8 @@ static void rtw_lps_change_dtim_hdl(struct adapter *padapter, u8 dtim)
 
 static void rtw_dm_ra_mask_hdl(struct adapter *padapter, struct sta_info *psta)
 {
-       if (psta) {
+       if (psta)
                set_sta_rate(padapter, psta);
-       }
 }
 
 u8 rtw_dm_ra_mask_wk_cmd(struct adapter *padapter, u8 *psta)
@@ -1977,9 +1969,8 @@ u8 rtw_drvextra_cmd_hdl(struct adapter *padapter, unsigned char *pbuf)
                break;
        }
 
-       if (pdrvextra_cmd->pbuf && pdrvextra_cmd->size > 0) {
+       if (pdrvextra_cmd->pbuf && pdrvextra_cmd->size > 0)
                kfree(pdrvextra_cmd->pbuf);
-       }
 
        return H2C_SUCCESS;
 }