Messages in this thread Patch in this message | | | From | xiakaixu1987@gmail ... | Subject | [PATCH] staging: rtl8723bs: remove redundant statements | Date | Sat, 21 Nov 2020 20:17:21 +0800 |
| |
From: Kaixu Xia <kaixuxia@tencent.com>
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> --- drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c b/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c index 85ea535dd6e9..645f4f2a835e 100644 --- a/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c +++ b/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c @@ -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++) { -- 2.20.0
| |