staging: rtl8188eu: core: rtw_xmit: Remove extra braces
authorGeorgiana Rodica Chelu <georgiana.chelu93@gmail.com>
Tue, 20 Sep 2016 09:13:06 +0000 (12:13 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 20 Sep 2016 11:38:00 +0000 (13:38 +0200)
Remove the braces {} in the following cases:
- when there is a single branch with a single statement.
- when both branches have a single statement

The modifications improve the coding style.
Issues found by checkpatch.pl script.

Signed-off-by: Georgiana Rodica Chelu <georgiana.chelu93@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_xmit.c

index 56c6604401d4bf372d5bd89d60713dcaf7677b48..1d2817b3a5df1345de9397b4ec9f2aba15ba15a0 100644 (file)
@@ -142,9 +142,8 @@ s32 _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
                if (res == _FAIL) {
                        msleep(10);
                        res = rtw_os_xmit_resource_alloc(padapter, pxmitbuf, (MAX_XMITBUF_SZ + XMITBUF_ALIGN_SZ));
-                       if (res == _FAIL) {
+                       if (res == _FAIL)
                                goto exit;
-                       }
                }
 
                pxmitbuf->flags = XMIT_VO_QUEUE;
@@ -248,9 +247,8 @@ void _rtw_free_xmit_priv(struct xmit_priv *pxmitpriv)
                pxmitbuf++;
        }
 
-       if (pxmitpriv->pallocated_xmit_extbuf) {
+       if (pxmitpriv->pallocated_xmit_extbuf)
                vfree(pxmitpriv->pallocated_xmit_extbuf);
-       }
 
        rtw_free_hwxmits(padapter);
 
@@ -779,11 +777,10 @@ s32 rtw_make_wlanhdr(struct adapter *padapter, u8 *hdr, struct pkt_attrib *pattr
        if (pattrib->psta) {
                psta = pattrib->psta;
        } else {
-               if (bmcst) {
+               if (bmcst)
                        psta = rtw_get_bcmc_stainfo(padapter);
-               } else {
+               else
                        psta = rtw_get_stainfo(&padapter->stapriv, pattrib->ra);
-               }
        }
 
        memset(hdr, 0, WLANHDR_OFFSET);
@@ -1048,9 +1045,8 @@ s32 rtw_xmitframe_coalesce(struct adapter *padapter, struct sk_buff *pkt, struct
                        mpdu_len -= llc_sz;
                }
 
-               if ((pattrib->icv_len > 0) && (pattrib->bswenc)) {
+               if ((pattrib->icv_len > 0) && (pattrib->bswenc))
                        mpdu_len -= pattrib->icv_len;
-               }
 
                if (bmcst) {
                        /*  don't do fragment to broadcat/multicast packets */
@@ -1554,11 +1550,10 @@ s32 rtw_xmit_classifier(struct adapter *padapter, struct xmit_frame *pxmitframe)
        int res = _SUCCESS;
 
 
-       if (pattrib->psta) {
+       if (pattrib->psta)
                psta = pattrib->psta;
-       } else {
+       else
                psta = rtw_get_stainfo(pstapriv, pattrib->ra);
-       }
 
        if (psta == NULL) {
                res = _FAIL;