Several conditionals in rtw_wx_set_auth() contain a comment then a
single statement.  Move the comments to the top of the conditionals
so that braces can be removed from the statements, which saves space
and makes the code more readable.

Signed-off-by: Quytelda Kahja <quyte...@tamalin.org>
---
 drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
index 3e78fc28a8eb..1cacd7fff052 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
@@ -2151,13 +2151,11 @@ static int rtw_wx_set_auth(struct net_device *dev,
                 */
                break;
        case IW_AUTH_TKIP_COUNTERMEASURES:
-               if (param->value) {
-                       /*  wpa_supplicant is enabling the tkip countermeasure. 
*/
+               /* wpa_supplicant is setting the tkip countermeasure. */
+               if (param->value) /* enabling */
                        padapter->securitypriv.btkip_countermeasure = true;
-               } else {
-                       /*  wpa_supplicant is disabling the tkip 
countermeasure. */
+               else /* disabling */
                        padapter->securitypriv.btkip_countermeasure = false;
-               }
                break;
        case IW_AUTH_DROP_UNENCRYPTED:
                /* HACK:
@@ -2172,10 +2170,12 @@ static int rtw_wx_set_auth(struct net_device *dev,
                 * be set.
                 */
 
-               if (padapter->securitypriv.ndisencryptstatus == 
Ndis802_11Encryption1Enabled) {
-                       break;/* it means init value, or using wep, 
ndisencryptstatus = Ndis802_11Encryption1Enabled, */
-                       /*  then it needn't reset it; */
-               }
+               /*
+                * This means init value, or using wep, ndisencryptstatus =
+                * Ndis802_11Encryption1Enabled, then it needn't reset it;
+                */
+               if (padapter->securitypriv.ndisencryptstatus == 
Ndis802_11Encryption1Enabled)
+                       break;
 
                if (param->value) {
                        padapter->securitypriv.ndisencryptstatus = 
Ndis802_11EncryptionDisabled;
-- 
2.16.3

Reply via email to