staging: rtl8723bs: remove redundant statements
authorKaixu Xia <kaixuxia@tencent.com>
Sat, 21 Nov 2020 12:17:21 +0000 (20:17 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 23 Nov 2020 16:59:52 +0000 (17:59 +0100)
The bool variable is2T is true, so the if statement is redundant.
we can directly set the variable bound to 8 and remove the if
statement.

Reported-by: Tosk Robot <tencent_os_robot@tencent.com>
Signed-off-by: Kaixu Xia <kaixuxia@tencent.com>
Link: https://lore.kernel.org/r/1605961041-12875-1-git-send-email-kaixuxia@tencent.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c

index 85ea535dd6e93cc1ac5023b309544360cba57608..645f4f2a835ee60a07aab4262df4061a1cf6fd42 100644 (file)
@@ -1407,14 +1407,9 @@ static bool phy_SimularityCompare_8723B(
        u32 i, j, diff, SimularityBitMap, bound = 0;
        u8 final_candidate[2] = {0xFF, 0xFF}; /* for path A and path B */
        bool bResult = true;
-       bool is2T = true;
        s32 tmp1 = 0, tmp2 = 0;
 
-       if (is2T)
-               bound = 8;
-       else
-               bound = 4;
-
+       bound = 8;
        SimularityBitMap = 0;
 
        for (i = 0; i < bound; i++) {