Staging: wilc1000: Remove unnecessary else after return
authorAybuke Ozdemir <aybuke.147@gmail.com>
Sat, 17 Oct 2015 12:25:34 +0000 (15:25 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 17 Oct 2015 16:03:16 +0000 (09:03 -0700)
Problem found using checkpatch.pl:
WARNING: else is not generally useful after a break or return

Signed-off-by: Aybuke Ozdemir <aybuke.147@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/coreconfigurator.c

index 93f3c75..cea7c6f 100644 (file)
@@ -307,8 +307,7 @@ u8 *get_tim_elm(u8 *pu8msa, u16 u16RxLen, u16 u16TagParamOffset)
        while (u16index < (u16RxLen - FCS_LEN)) {
                if (pu8msa[u16index] == ITIM)
                        return &pu8msa[u16index];
-               else
-                       u16index += (IE_HDR_LEN + pu8msa[u16index + 1]);
+               u16index += (IE_HDR_LEN + pu8msa[u16index + 1]);
        }
 
        return NULL;
@@ -324,9 +323,8 @@ u8 get_current_channel_802_11n(u8 *pu8msa, u16 u16RxLen)
        while (index < (u16RxLen - FCS_LEN)) {
                if (pu8msa[index] == IDSPARMS)
                        return pu8msa[index + 2];
-               else
-                       /* Increment index by length information and header */
-                       index += pu8msa[index + 1] + IE_HDR_LEN;
+               /* Increment index by length information and header */
+               index += pu8msa[index + 1] + IE_HDR_LEN;
        }
 
        /* Return current channel information from the MIB, if beacon/probe  */