Re: [RFT PATCH v3 5/5] staging: rtl8723bs: remove duplicate rate checks in PHY_GetTxPowerIndexBase()

[Ethan Tidmore]

On Sun Apr 5, 2026 at 7:57 AM CDT, Luka Gejak wrote:
> On April 5, 2026 1:41:32 PM GMT+02:00, Prithvi Tambewagh <activprithvi@xxxxxxxxx> wrote:
>>The code previously checked (Rate >= MGN_MCS0 && Rate <= MGN_MCS7)
>>condition twice - once for the (BandWidth == CHANNEL_WIDTH_20) check and
>>once for the (BandWidth == CHANNEL_WIDTH_40) check. Fix if statement
>>formatting to move that if check as an outer if check to improve code
>>formatting.
>>
>>Signed-off-by: Prithvi Tambewagh <activprithvi@xxxxxxxxx>
>>---
>> drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 7 +++----
>> 1 file changed, 3 insertions(+), 4 deletions(-)
>>
>>diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
>>index 9e523491a008..efd1c76f2953 100644
>>--- a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
>>+++ b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
>>@@ -469,11 +469,10 @@ u8 PHY_GetTxPowerIndexBase(
>> 	if ((Rate >= MGN_6M && Rate <= MGN_54M) && !IS_CCK_RATE(Rate))
>> 		txPower += pHalData->OFDM_24G_Diff[RFPath][TX_1S];
>> 
>>-	if (BandWidth == CHANNEL_WIDTH_20) { /*  BW20-1S, BW20-2S */
>>-		if (MGN_MCS0 <= Rate && Rate <= MGN_MCS7)
>>+	if (Rate >= MGN_MCS0 && Rate <= MGN_MCS7) {
>>+		if (BandWidth == CHANNEL_WIDTH_20) /*  BW20-1S, BW20-2S */
>> 			txPower += pHalData->BW20_24G_Diff[RFPath][TX_1S];
>>-	} else if (BandWidth == CHANNEL_WIDTH_40) { /*  BW40-1S, BW40-2S */
>>-		if (MGN_MCS0 <= Rate && Rate <= MGN_MCS7)
>>+		else if (BandWidth == CHANNEL_WIDTH_40) /*  BW40-1S, BW40-2S */
>> 			txPower += pHalData->BW40_24G_Diff[RFPath][TX_1S];
>> 	}
>> 
>
> LGTM,
> Reviewed-by: Luka Gejak <luka.gejak@xxxxxxxxx>

Doesn't apply to staging-next too.

Thanks,

ET