From: "Luka Gejak" <luka.gejak@linux.dev>
To: "Pranav Desai" <contact.pranavdesai@gmail.com>,
<gregkh@linuxfoundation.org>
Cc: <linux-staging@lists.linux.dev>, <straube.linux@gmail.com>,
<b9788213@gmail.com>, <bera@yuzu.li>
Subject: Re: [PATCH 2/2] staging: rtl8723bs: fix comment style in HalPhyRf_8723B.c
Date: Thu, 09 Apr 2026 16:03:55 +0200 [thread overview]
Message-ID: <DHOOIGOHBSIA.122G87AINVY3G@linux.dev> (raw)
In-Reply-To: <20260409034859.42356-2-contact.pranavdesai@gmail.com>
On Thu Apr 9, 2026 at 5:48 AM CEST, Pranav Desai wrote:
> removed extra spaces for comments and formated them to be more readable
>
> Signed-off-by: Pranav Desai <contact.pranavdesai@gmail.com>
> ---
> .../staging/rtl8723bs/hal/HalPhyRf_8723B.c | 120 +++++++++---------
> 1 file changed, 60 insertions(+), 60 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c b/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c
> index 63c848ebd661..d4e8ec172658 100644
> --- a/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c
> +++ b/drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c
> @@ -8,8 +8,8 @@
> #include <drv_types.h>
> #include "odm_precomp.h"
>
> -/* MACRO definition for pRFCalibrateInfo->TxIQC_8723B[0] */
> -#define PATH_S0 1 /* RF_PATH_B */
> +/* MACRO definition for pRFCalibrateInfo->TxIQC_8723B[0] */
> +#define PATH_S0 1 /* RF_PATH_B */
> #define IDX_0xC94 0
> #define IDX_0xC80 1
> #define IDX_0xC14 0
> @@ -17,8 +17,8 @@
> #define KEY 0
> #define VAL 1
>
> -/* MACRO definition for pRFCalibrateInfo->TxIQC_8723B[1] */
> -#define PATH_S1 0 /* RF_PATH_A */
> +/* MACRO definition for pRFCalibrateInfo->TxIQC_8723B[1] */
> +#define PATH_S1 0 /* RF_PATH_A */
> #define IDX_0xC4C 2
>
> /*---------------------------Define Local Constant---------------------------*/
> @@ -210,7 +210,7 @@ void ODM_TxPwrTrackSetPwr_8723B(
> Final_OFDM_Swing_Index = pDM_Odm->DefaultOfdmIndex + pDM_Odm->Absolute_OFDMSwingIdx[RFPath];
> Final_CCK_Swing_Index = pDM_Odm->DefaultCckIndex + pDM_Odm->Absolute_OFDMSwingIdx[RFPath];
>
> - /* Adjust BB swing by OFDM IQ matrix */
> + /* Adjust BB swing by OFDM IQ matrix */
> if (Final_OFDM_Swing_Index >= PwrTrackingLimit_OFDM)
> Final_OFDM_Swing_Index = PwrTrackingLimit_OFDM;
> else if (Final_OFDM_Swing_Index <= 0)
> @@ -269,7 +269,7 @@ void ODM_TxPwrTrackSetPwr_8723B(
> setCCKFilterCoefficient(pDM_Odm, PwrTrackingLimit_CCK);
> pDM_Odm->Modify_TxAGC_Flag_PathA_CCK = true;
> PHY_SetTxPowerIndexByRateSection(Adapter, RFPath, pHalData->CurrentChannel, CCK);
> - } else if (Final_CCK_Swing_Index <= 0) { /* Lowest CCK Index = 0 */
> + } else if (Final_CCK_Swing_Index <= 0) { /* Lowest CCK Index = 0 */
> pDM_Odm->Remnant_CCKSwingIdx = Final_CCK_Swing_Index;
> setCCKFilterCoefficient(pDM_Odm, 0);
> pDM_Odm->Modify_TxAGC_Flag_PathA_CCK = true;
> @@ -284,7 +284,7 @@ void ODM_TxPwrTrackSetPwr_8723B(
> }
> }
> } else
> - return; /* This method is not supported. */
> + return; /* This method is not supported. */
> }
>
> static void GetDeltaSwingTable_8723B(
> @@ -350,17 +350,18 @@ static u8 phy_PathA_IQK_8723B(
> struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
> struct dm_odm_t *pDM_Odm = &pHalData->odmpriv;
>
> - /* Save RF Path */
> + /* Save RF Path */
> Path_SEL_BB = PHY_QueryBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord);
>
> /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* enable path A PA in TXIQK mode */
> + /* enable path A PA in TXIQK mode */
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_WE_LUT, 0x80000, 0x1);
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_RCK_OS, bRFRegOffsetMask, 0x18000);
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_TXPA_G1, bRFRegOffsetMask, 0x0003f);
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_TXPA_G2, bRFRegOffsetMask, 0xc7f87);
> + /* disable path B PA in TXIQK mode */
The code that actually disabled Path B was the dead code removed in
patch 1. By adding this comment back in patch 2 at the end of the
block, you leave a dangling, highly confusing comment that implies the
preceding RF_PATH_A code is disabling PATH_B. This is dangerous for
future maintainers trying to understand the hardware configuration. In
this patch, ensure you are only reformatting existing, valid comments.
Do not introduce dangling comments for code that no longer exists.
Best regards,
Luka Gejak
>
> /* 1 Tx IQK */
> /* IQK setting */
> @@ -384,10 +385,10 @@ static u8 phy_PathA_IQK_8723B(
>
> /* Ant switch */
> if (configPathB || (RF_Path == 0))
> - /* wifi switch to S1 */
> + /* wifi switch to S1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000000);
> else
> - /* wifi switch to S0 */
> + /* wifi switch to S0 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000280);
>
> /* GNT_BT = 0 */
> @@ -408,7 +409,7 @@ static u8 phy_PathA_IQK_8723B(
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regE94 = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_Before_IQK_A, bMaskDWord);
> regE9C = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_After_IQK_A, bMaskDWord);
> @@ -444,7 +445,7 @@ static u8 phy_PathA_RxIQK8723B(
> struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
> struct dm_odm_t *pDM_Odm = &pHalData->odmpriv;
>
> - /* Save RF Path */
> + /* Save RF Path */
> Path_SEL_BB = PHY_QueryBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord);
>
> /* leave IQK mode */
> @@ -481,10 +482,10 @@ static u8 phy_PathA_RxIQK8723B(
>
> /* Ant switch */
> if (configPathB || (RF_Path == 0))
> - /* wifi switch to S1 */
> + /* wifi switch to S1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000000);
> else
> - /* wifi switch to S0 */
> + /* wifi switch to S0 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000280);
>
> /* GNT_BT = 0 */
> @@ -504,7 +505,7 @@ static u8 phy_PathA_RxIQK8723B(
> /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regE94 = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_Before_IQK_A, bMaskDWord);
> regE9C = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_After_IQK_A, bMaskDWord);
> @@ -564,10 +565,10 @@ static u8 phy_PathA_RxIQK8723B(
>
> /* Ant switch */
> if (configPathB || (RF_Path == 0))
> - /* wifi switch to S1 */
> + /* wifi switch to S1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000000);
> else
> - /* wifi switch to S0 */
> + /* wifi switch to S0 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord, 0x00000280);
>
> /* GNT_BT = 0 */
> @@ -584,14 +585,14 @@ static u8 phy_PathA_RxIQK8723B(
> /* GNT_BT = 1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x764, bMaskDWord, 0x00001800);
>
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regEA4 = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_Before_IQK_A_2, bMaskDWord);
>
> - /* PA/PAD controlled by 0x0 */
> + /* PA/PAD controlled by 0x0 */
> /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xdf, bRFRegOffsetMask, 0x780);
> @@ -622,10 +623,10 @@ static u8 phy_PathB_IQK_8723B(struct adapter *padapter)
> struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
> struct dm_odm_t *pDM_Odm = &pHalData->odmpriv;
>
> - /* Save RF Path */
> + /* Save RF Path */
> Path_SEL_BB = PHY_QueryBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord);
>
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xed, 0x20, 0x1);
> @@ -671,10 +672,10 @@ static u8 phy_PathB_IQK_8723B(struct adapter *padapter)
> /* GNT_BT = 1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x764, bMaskDWord, 0x00001800);
>
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regE94 = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_Before_IQK_A, bMaskDWord);
> regE9C = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_After_IQK_A, bMaskDWord);
> @@ -705,9 +706,9 @@ static u8 phy_PathB_RxIQK8723B(struct adapter *padapter, bool configPathB)
> struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
> struct dm_odm_t *pDM_Odm = &pHalData->odmpriv;
>
> - /* Save RF Path */
> + /* Save RF Path */
> Path_SEL_BB = PHY_QueryBBReg(pDM_Odm->Adapter, 0x948, bMaskDWord);
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> /* switch to path B */
> @@ -741,7 +742,7 @@ static u8 phy_PathB_RxIQK8723B(struct adapter *padapter, bool configPathB)
> /* LO calibration setting */
> PHY_SetBBReg(pDM_Odm->Adapter, rIQK_AGC_Rsp, bMaskDWord, 0x0046a911);
>
> - /* enter IQK mode */
> + /* enter IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x808000);
>
> /* switch to path B */
> @@ -762,10 +763,10 @@ static u8 phy_PathB_RxIQK8723B(struct adapter *padapter, bool configPathB)
> /* GNT_BT = 1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x764, bMaskDWord, 0x00001800);
>
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regE94 = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_Before_IQK_A, bMaskDWord);
> regE9C = PHY_QueryBBReg(pDM_Odm->Adapter, rTx_Power_After_IQK_A, bMaskDWord);
> @@ -819,7 +820,7 @@ static u8 phy_PathB_RxIQK8723B(struct adapter *padapter, bool configPathB)
> /* LO calibration setting */
> PHY_SetBBReg(pDM_Odm->Adapter, rIQK_AGC_Rsp, bMaskDWord, 0x0046a8d1);
>
> - /* enter IQK mode */
> + /* enter IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x808000);
>
> /* switch to path B */
> @@ -839,10 +840,10 @@ static u8 phy_PathB_RxIQK8723B(struct adapter *padapter, bool configPathB)
> /* GNT_BT = 1 */
> PHY_SetBBReg(pDM_Odm->Adapter, 0x764, bMaskDWord, 0x00001800);
>
> - /* leave IQK mode */
> + /* leave IQK mode */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
>
> - /* Check failed */
> + /* Check failed */
> regEAC = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_After_IQK_A_2, bMaskDWord);
> regEA4 = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_Before_IQK_A_2, bMaskDWord);
>
> @@ -913,7 +914,7 @@ static void _PHY_PathAFillIQKMatrix8723B(
> pRFCalibrateInfo->TxIQC_8723B[PATH_S1][IDX_0xC4C][VAL] = PHY_QueryBBReg(pDM_Odm->Adapter, rOFDM0_ECCAThreshold, bMaskDWord);
>
> if (bTxOnly) {
> - /* <20130226, Kordan> Saving RxIQC, otherwise not initialized. */
> + /* <20130226, Kordan> Saving RxIQC, otherwise not initialized. */
> pRFCalibrateInfo->RxIQC_8723B[PATH_S1][IDX_0xCA0][KEY] = rOFDM0_RxIQExtAnta;
> pRFCalibrateInfo->RxIQC_8723B[PATH_S1][IDX_0xCA0][VAL] = 0xfffffff & PHY_QueryBBReg(pDM_Odm->Adapter, rOFDM0_RxIQExtAnta, bMaskDWord);
> pRFCalibrateInfo->RxIQC_8723B[PATH_S1][IDX_0xC14][KEY] = rOFDM0_XARxIQImbalance;
> @@ -1009,10 +1010,10 @@ static void _PHY_PathBFillIQKMatrix8723B(
> }
> }
>
> -/* */
> -/* 2011/07/26 MH Add an API for testing IQK fail case. */
> -/* */
> -/* MP Already declare in odm.c */
> +/* */
> +/* 2011/07/26 MH Add an API for testing IQK fail case. */
> +/* */
> +/* MP Already declare in odm.c */
>
> void ODM_SetIQCbyRFpath(struct dm_odm_t *pDM_Odm, u32 RFpath)
> {
> @@ -1290,14 +1291,14 @@ static void phy_IQCalibrate_8723B(
> };
> const u32 retryCount = 2;
>
> - /* Note: IQ calibration must be performed after loading */
> - /* PHY_REG.txt , and radio_a, radio_b.txt */
> + /* Note: IQ calibration must be performed after loading */
> + /* PHY_REG.txt , and radio_a, radio_b.txt */
>
> /* u32 bbvalue; */
>
> if (t == 0) {
>
> - /* Save ADDA parameters, turn Path A ADDA on */
> + /* Save ADDA parameters, turn Path A ADDA on */
> _PHY_SaveADDARegisters8723B(padapter, ADDA_REG, pDM_Odm->RFCalibrateInfo.ADDA_backup, IQK_ADDA_REG_NUM);
> _PHY_SaveMACRegisters8723B(padapter, IQK_MAC_REG, pDM_Odm->RFCalibrateInfo.IQK_MAC_backup);
> _PHY_SaveADDARegisters8723B(padapter, IQK_BB_REG_92C, pDM_Odm->RFCalibrateInfo.IQK_BB_backup, IQK_BB_REG_NUM);
> @@ -1325,11 +1326,11 @@ static void phy_IQCalibrate_8723B(
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xed, 0x20, 0x1);
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0x43, bRFRegOffsetMask, 0x60fbd);
>
> -/* path A TX IQK */
> + /* path A TX IQK */
> for (i = 0 ; i < retryCount ; i++) {
> PathAOK = phy_PathA_IQK_8723B(padapter, is2T, RF_Path);
> if (PathAOK == 0x01) {
> - /* Path A Tx IQK Success */
> + /* Path A Tx IQK Success */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
> pDM_Odm->RFCalibrateInfo.TxLOK[RF_PATH_A] = PHY_QueryRFReg(pDM_Odm->Adapter, RF_PATH_A, 0x8, bRFRegOffsetMask);
>
> @@ -1361,7 +1362,7 @@ static void phy_IQCalibrate_8723B(
> for (i = 0 ; i < retryCount ; i++) {
> PathBOK = phy_PathB_IQK_8723B(padapter);
> if (PathBOK == 0x01) {
> - /* Path B Tx IQK Success */
> + /* Path B Tx IQK Success */
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0x000000);
> pDM_Odm->RFCalibrateInfo.TxLOK[RF_PATH_B] = PHY_QueryRFReg(pDM_Odm->Adapter, RF_PATH_B, 0x8, bRFRegOffsetMask);
>
> @@ -1388,17 +1389,16 @@ static void phy_IQCalibrate_8723B(
> PHY_SetBBReg(pDM_Odm->Adapter, rFPGA0_IQK, bMaskH3Bytes, 0);
>
> if (t != 0) {
> - /* Reload ADDA power saving parameters */
> + /* Reload ADDA power saving parameters */
> _PHY_ReloadADDARegisters8723B(padapter, ADDA_REG, pDM_Odm->RFCalibrateInfo.ADDA_backup, IQK_ADDA_REG_NUM);
>
> - /* Reload MAC parameters */
> + /* Reload MAC parameters */
> _PHY_ReloadMACRegisters8723B(padapter, IQK_MAC_REG, pDM_Odm->RFCalibrateInfo.IQK_MAC_backup);
>
> _PHY_ReloadADDARegisters8723B(padapter, IQK_BB_REG_92C, pDM_Odm->RFCalibrateInfo.IQK_BB_backup, IQK_BB_REG_NUM);
>
> -
> /* Allen initial gain 0xc50 */
> - /* Restore RX initial gain */
> + /* Restore RX initial gain */
> PHY_SetBBReg(pDM_Odm->Adapter, 0xc50, bMaskByte0, 0x50);
> PHY_SetBBReg(pDM_Odm->Adapter, 0xc50, bMaskByte0, tmp0xc50);
> if (is2T) {
> @@ -1426,8 +1426,8 @@ static void phy_LCCalibrate_8723B(struct dm_odm_t *pDM_Odm, bool is2T)
>
> if ((tmpReg&0x70) != 0) /* Deal with contisuous TX case */
> rtw_write8(pDM_Odm->Adapter, 0xd03, tmpReg&0x8F); /* disable all continuous TX */
> - else /* Deal with Packet TX case */
> - rtw_write8(pDM_Odm->Adapter, REG_TXPAUSE, 0xFF); /* block all queues */
> + else /* Deal with Packet TX case */
> + rtw_write8(pDM_Odm->Adapter, REG_TXPAUSE, 0xFF); /* block all queues */
>
> if ((tmpReg&0x70) != 0) {
> /* 1. Read original RF mode */
> @@ -1451,14 +1451,14 @@ static void phy_LCCalibrate_8723B(struct dm_odm_t *pDM_Odm, bool is2T)
> LC_Cal = PHY_QueryRFReg(padapter, RF_PATH_A, RF_CHNLBW, bMask12Bits);
>
> /* 4. Set LC calibration begin bit15 */
> - PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xB0, bRFRegOffsetMask, 0xDFBE0); /* LDO ON */
> + PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xB0, bRFRegOffsetMask, 0xDFBE0); /* LDO ON */
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_CHNLBW, bMask12Bits, LC_Cal|0x08000);
>
> mdelay(100);
>
> - PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xB0, bRFRegOffsetMask, 0xDFFE0); /* LDO OFF */
> + PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, 0xB0, bRFRegOffsetMask, 0xDFFE0); /* LDO OFF */
>
> - /* Channel 10 LC calibration issue for 8723bs with 26M xtal */
> + /* Channel 10 LC calibration issue for 8723bs with 26M xtal */
> if (pDM_Odm->SupportInterface == ODM_ITRF_SDIO && pDM_Odm->PackageType >= 0x2) {
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_A, RF_CHNLBW, bMask12Bits, LC_Cal);
> }
> @@ -1472,7 +1472,7 @@ static void phy_LCCalibrate_8723B(struct dm_odm_t *pDM_Odm, bool is2T)
> /* Path-B */
> if (is2T)
> PHY_SetRFReg(pDM_Odm->Adapter, RF_PATH_B, RF_AC, bMask12Bits, RF_Bmode);
> - } else /* Deal with Packet TX case */
> + } else /* Deal with Packet TX case */
> rtw_write8(pDM_Odm->Adapter, REG_TXPAUSE, 0x00);
> }
>
> @@ -1516,7 +1516,7 @@ void PHY_IQCalibrate_8723B(
> if (!(pDM_Odm->SupportAbility & ODM_RF_CALIBRATION))
> return;
>
> - /* 20120213<Kordan> Turn on when continuous Tx to pass lab testing. (required by Edlu) */
> + /* 20120213<Kordan> Turn on when continuous Tx to pass lab testing. (required by Edlu) */
> if (bSingleTone || bCarrierSuppression)
> return;
>
> @@ -1533,7 +1533,7 @@ void PHY_IQCalibrate_8723B(
>
> path = (PHY_QueryBBReg(pDM_Odm->Adapter, rS0S1_PathSwitch, bMaskByte0) == 0x00) ? RF_PATH_A : RF_PATH_B;
>
> - /* Restore TX IQK */
> + /* Restore TX IQK */
> for (i = 0; i < 3; ++i) {
> offset = pRFCalibrateInfo->TxIQC_8723B[path][i][0];
> data = pRFCalibrateInfo->TxIQC_8723B[path][i][1];
> @@ -1544,7 +1544,7 @@ void PHY_IQCalibrate_8723B(
> PHY_SetBBReg(pDM_Odm->Adapter, offset, bMaskDWord, data);
> }
>
> - /* Restore RX IQK */
> + /* Restore RX IQK */
> for (i = 0; i < 2; ++i) {
> offset = pRFCalibrateInfo->RxIQC_8723B[path][i][0];
> data = pRFCalibrateInfo->RxIQC_8723B[path][i][1];
> @@ -1654,8 +1654,8 @@ void PHY_IQCalibrate_8723B(
> _PHY_PathBFillIQKMatrix8723B(padapter, bPathBOK, result, final_candidate, (RegEC4 == 0));
> }
>
> -/* To Fix BSOD when final_candidate is 0xff */
> -/* by sherry 20120321 */
> + /* To Fix BSOD when final_candidate is 0xff */
> + /* by sherry 20120321 */
> if (final_candidate < 4) {
> for (i = 0; i < IQK_MATRIX_REG_NUM; i++)
> pDM_Odm->RFCalibrateInfo.iqk_matrix_regs_setting_value[0][i] = result[final_candidate][i];
> @@ -1692,7 +1692,7 @@ void PHY_LCCalibrate_8723B(struct dm_odm_t *pDM_Odm)
> if (!(pDM_Odm->SupportAbility & ODM_RF_CALIBRATION))
> return;
>
> - /* 20120213<Kordan> Turn on when continuous Tx to pass lab testing. (required by Edlu) */
> + /* 20120213<Kordan> Turn on when continuous Tx to pass lab testing. (required by Edlu) */
> if (bSingleTone || bCarrierSuppression)
> return;
>
next prev parent reply other threads:[~2026-04-09 14:04 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-09 3:48 [PATCH 1/2] staging: rtl8723bs: remove commented-out dead code in HalPhyRf_8723B.c Pranav Desai
2026-04-09 3:48 ` [PATCH 2/2] staging: rtl8723bs: fix comment style " Pranav Desai
2026-04-09 14:03 ` Luka Gejak [this message]
2026-04-09 13:14 ` [PATCH 1/2] staging: rtl8723bs: remove commented-out dead code " Bera Yüzlü
2026-04-09 13:57 ` Luka Gejak
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DHOOIGOHBSIA.122G87AINVY3G@linux.dev \
--to=luka.gejak@linux.dev \
--cc=b9788213@gmail.com \
--cc=bera@yuzu.li \
--cc=contact.pranavdesai@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-staging@lists.linux.dev \
--cc=straube.linux@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox