]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
staging: rtl8723bs: remove braces in single statement blocks
authorKang Minchul <tegongkang@gmail.com>
Tue, 13 Sep 2022 16:25:10 +0000 (01:25 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 24 Sep 2022 11:02:11 +0000 (13:02 +0200)
This commit cleans up checkpatch warning as follows:
WARNING: braces {} are not necessary for single statement blocks

Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Kang Minchul <tegongkang@gmail.com>
Link: https://lore.kernel.org/r/20220913162510.3134430-1-tegongkang@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_recv.c

index d8d394b67eebc7fad95051445406536e0adb1173..f360c7c631cb885b468e30f96d095995f75cef7f 100644 (file)
@@ -1374,9 +1374,8 @@ static signed int validate_80211w_mgmt(struct adapter *adapter, union recv_frame
                        /* actual management data frame body */
                        data_len = pattrib->pkt_len - pattrib->hdrlen - pattrib->iv_len - pattrib->icv_len;
                        mgmt_DATA = rtw_zmalloc(data_len);
-                       if (!mgmt_DATA) {
+                       if (!mgmt_DATA)
                                goto validate_80211w_fail;
-                       }
                        precv_frame = decryptor(adapter, precv_frame);
                        /* save actual management data frame body */
                        memcpy(mgmt_DATA, ptr+pattrib->hdrlen+pattrib->iv_len, data_len);
@@ -1385,9 +1384,8 @@ static signed int validate_80211w_mgmt(struct adapter *adapter, union recv_frame
                        /* remove the iv and icv length */
                        pattrib->pkt_len = pattrib->pkt_len - pattrib->iv_len - pattrib->icv_len;
                        kfree(mgmt_DATA);
-                       if (!precv_frame) {
+                       if (!precv_frame)
                                goto validate_80211w_fail;
-                       }
                } else if (IS_MCAST(GetAddr1Ptr(ptr)) &&
                        (subtype == WIFI_DEAUTH || subtype == WIFI_DISASSOC)) {
                        signed int BIP_ret = _SUCCESS;
@@ -1651,14 +1649,12 @@ static int check_indicate_seq(struct recv_reorder_ctrl *preorder_ctrl, u16 seq_n
        u16 wend = (preorder_ctrl->indicate_seq + wsize - 1) & 0xFFF;/*  4096; */
 
        /*  Rx Reorder initialize condition. */
-       if (preorder_ctrl->indicate_seq == 0xFFFF) {
+       if (preorder_ctrl->indicate_seq == 0xFFFF)
                preorder_ctrl->indicate_seq = seq_num;
-       }
 
        /*  Drop out the packet which SeqNum is smaller than WinStart */
-       if (SN_LESS(seq_num, preorder_ctrl->indicate_seq)) {
+       if (SN_LESS(seq_num, preorder_ctrl->indicate_seq))
                return false;
-       }
 
        /*  */
        /*  Sliding window manipulation. Conditions includes: */
@@ -2084,10 +2080,8 @@ s32 rtw_recv_entry(union recv_frame *precvframe)
        precvpriv = &padapter->recvpriv;
 
        ret = recv_func(padapter, precvframe);
-       if (ret == _FAIL) {
+       if (ret == _FAIL)
                goto _recv_entry_drop;
-       }
-
 
        precvpriv->rx_pkts++;