[PATCH v1 0/4] staging: rtl8723bs: Fix missing spaces around operators

[PATCH v1 0/4] staging: rtl8723bs: Fix missing spaces around operators

[Moksh Panicker]

This patch series fixes missing spaces around arithmetic and bitwise
operators ('+',' -', '&') in the rtl8723bs staging driver. These were
flagged by checkpatch.pl with:

CHECK: spaces preferred around that 'X' (ctx:VxV)

Each patch addresses one file independently.

Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>

Moksh Panicker (4):
  staging: rtl8723bs: Fix missing space around '+' operator in
    hal_sdio.c
  staging: rtl8723bs: Fix missing space around '-' operator in
    hal_com_phycfg.c
  staging: rtl8723bs: Fix missing spaces around operators in
    HalPwrSeqCmd.c
  staging: rtl8723bs: Fix missing spaces around '-' operator in
    HalPhyRf.c

 drivers/staging/rtl8723bs/hal/HalPhyRf.c       | 8 ++++----
 drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c   | 6 +++---
 drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 2 +-
 drivers/staging/rtl8723bs/hal/hal_sdio.c       | 2 +-
 4 files changed, 9 insertions(+), 9 deletions(-)

-- 
2.34.1

[PATCH v1 1/4] staging: rtl8723bs: Fix missing space around '+' operator in hal_sdio.c

[Moksh Panicker]

Add missing space around the '+' operator in HalQueryTxBufferStatus().
This fixes the following checkpatch.pl warning:

CHECK: spaces preferred around that '+' (ctx:VxV)
Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
---
 drivers/staging/rtl8723bs/hal/hal_sdio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8723bs/hal/hal_sdio.c b/drivers/staging/rtl8723bs/hal/hal_sdio.c
index 665c85ecc..1d29245da 100644
--- a/drivers/staging/rtl8723bs/hal/hal_sdio.c
+++ b/drivers/staging/rtl8723bs/hal/hal_sdio.c
@@ -24,7 +24,7 @@ u8 rtw_hal_sdio_query_tx_freepage(
 {
 	struct hal_com_data	*pHalData = GET_HAL_DATA(padapter);
 
-	if ((pHalData->SdioTxFIFOFreePage[PageIdx]+pHalData->SdioTxFIFOFreePage[PUBLIC_QUEUE_IDX]) >= (RequiredPageNum))
+	if ((pHalData->SdioTxFIFOFreePage[PageIdx] + pHalData->SdioTxFIFOFreePage[PUBLIC_QUEUE_IDX]) >= (RequiredPageNum))
 		return true;
 	else
 		return false;
-- 
2.34.1

Re: [PATCH v1 1/4] staging: rtl8723bs: Fix missing space around '+' operator in hal_sdio.c

[Dan Carpenter]

On Sun, Jun 14, 2026 at 12:21:52PM +0000, Moksh Panicker wrote:
> Add missing space around the '+' operator in HalQueryTxBufferStatus().
> This fixes the following checkpatch.pl warning:
> 
> CHECK: spaces preferred around that '+' (ctx:VxV)
> Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>

There needs to be a blank line before Signing block.

regards,
dan carpenter

[PATCH v1 2/4] staging: rtl8723bs: Fix missing space around '-' operator in hal_com_phycfg.c

[Moksh Panicker]

Add missing space around the '-' operator in channel index calculation.
This fixes the following checkpatch.pl warning:

CHECK: spaces preferred around that '-' (ctx:VxV)
Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
---
 drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
index bdd595a99..5e2f4131c 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
@@ -459,7 +459,7 @@ u8 PHY_GetTxPowerIndexBase(
 {
 	struct hal_com_data *pHalData = GET_HAL_DATA(padapter);
 	u8 txPower = 0;
-	u8 chnlIdx = (Channel-1);
+	u8 chnlIdx = (Channel - 1);
 
 	if (HAL_IsLegalChannel(padapter, Channel) == false)
 		chnlIdx = 0;
-- 
2.34.1

[PATCH v1 3/4] staging: rtl8723bs: Fix missing spaces around operators in HalPwrSeqCmd.c

[Moksh Panicker]

Add missing spaces around the '&' operator in HalPwrSeqCmdParsing().
This fixes the following checkpatch.pl warnings:

CHECK: spaces preferred around that '&' (ctx:ExV)
CHECK: spaces preferred around that '&' (ctx:WxV)
Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
---
 drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
index 86404b5e6..98a3d7695 100644
--- a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
+++ b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c
@@ -88,7 +88,7 @@ u8 HalPwrSeqCmdParsing(
 					value &= (~(GET_PWR_CFG_MASK(PwrCfgCmd)));
 					value |= (
 						GET_PWR_CFG_VALUE(PwrCfgCmd)
-						&GET_PWR_CFG_MASK(PwrCfgCmd)
+						 & GET_PWR_CFG_MASK(PwrCfgCmd)
 					);
 
 					/*  Write the value back to system register */
@@ -106,7 +106,7 @@ u8 HalPwrSeqCmdParsing(
 					else
 						value = rtw_read8(padapter, offset);
 
-					value = value&GET_PWR_CFG_MASK(PwrCfgCmd);
+					value = value & GET_PWR_CFG_MASK(PwrCfgCmd);
 					if (
 						value == (GET_PWR_CFG_VALUE(PwrCfgCmd) &
 						GET_PWR_CFG_MASK(PwrCfgCmd))
@@ -126,7 +126,7 @@ u8 HalPwrSeqCmdParsing(
 				if (GET_PWR_CFG_VALUE(PwrCfgCmd) == PWRSEQ_DELAY_US)
 					udelay(GET_PWR_CFG_OFFSET(PwrCfgCmd));
 				else
-					udelay(GET_PWR_CFG_OFFSET(PwrCfgCmd)*1000);
+					udelay(GET_PWR_CFG_OFFSET(PwrCfgCmd) * 1000);
 				break;
 
 			case PWR_CMD_END:
-- 
2.34.1

[PATCH v1 4/4] staging: rtl8723bs: Fix missing spaces around '-' operator in HalPhyRf.c

[Moksh Panicker]

Add missing spaces around the '-' operator in swing table index
boundary checks. This fixes the following checkpatch.pl warnings:

CHECK: spaces preferred around that '-' (ctx:VxV)
Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
---
 drivers/staging/rtl8723bs/hal/HalPhyRf.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/HalPhyRf.c b/drivers/staging/rtl8723bs/hal/HalPhyRf.c
index 7bef05a9a..254bfef54 100644
--- a/drivers/staging/rtl8723bs/hal/HalPhyRf.c
+++ b/drivers/staging/rtl8723bs/hal/HalPhyRf.c
@@ -220,13 +220,13 @@ void ODM_TXPowerTrackingCallback_ThermalMeter(struct adapter *Adapter)
 				pDM_Odm->RFCalibrateInfo.OFDM_index[p];
 
 			/* 4 7.1 Handle boundary conditions of index. */
-			if (pDM_Odm->RFCalibrateInfo.OFDM_index[p] > c.SwingTableSize_OFDM-1)
-				pDM_Odm->RFCalibrateInfo.OFDM_index[p] = c.SwingTableSize_OFDM-1;
+			if (pDM_Odm->RFCalibrateInfo.OFDM_index[p] > c.SwingTableSize_OFDM - 1)
+				pDM_Odm->RFCalibrateInfo.OFDM_index[p] = c.SwingTableSize_OFDM - 1;
 			else if (pDM_Odm->RFCalibrateInfo.OFDM_index[p] < OFDM_min_index)
 				pDM_Odm->RFCalibrateInfo.OFDM_index[p] = OFDM_min_index;
 		}
-		if (pDM_Odm->RFCalibrateInfo.CCK_index > c.SwingTableSize_CCK-1)
-			pDM_Odm->RFCalibrateInfo.CCK_index = c.SwingTableSize_CCK-1;
+		if (pDM_Odm->RFCalibrateInfo.CCK_index > c.SwingTableSize_CCK - 1)
+			pDM_Odm->RFCalibrateInfo.CCK_index = c.SwingTableSize_CCK - 1;
 		/* else if (pDM_Odm->RFCalibrateInfo.CCK_index < 0) */
 			/* pDM_Odm->RFCalibrateInfo.CCK_index = 0; */
 	} else {
-- 
2.34.1

Re: [PATCH v1 0/4] staging: rtl8723bs: Fix missing spaces around operators

[Dan Carpenter]

On Sun, Jun 14, 2026 at 12:21:51PM +0000, Moksh Panicker wrote:
> This patch series fixes missing spaces around arithmetic and bitwise
> operators ('+',' -', '&') in the rtl8723bs staging driver. These were
> flagged by checkpatch.pl with:
> 
> CHECK: spaces preferred around that 'X' (ctx:VxV)
> 
> Each patch addresses one file independently.
> 
> Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
> 
> Moksh Panicker (4):
>   staging: rtl8723bs: Fix missing space around '+' operator in
>     hal_sdio.c
>   staging: rtl8723bs: Fix missing space around '-' operator in
>     hal_com_phycfg.c
>   staging: rtl8723bs: Fix missing spaces around operators in
>     HalPwrSeqCmd.c
>   staging: rtl8723bs: Fix missing spaces around '-' operator in
>     HalPhyRf.c
> 
>  drivers/staging/rtl8723bs/hal/HalPhyRf.c       | 8 ++++----
>  drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c   | 6 +++---
>  drivers/staging/rtl8723bs/hal/hal_com_phycfg.c | 2 +-
>  drivers/staging/rtl8723bs/hal/hal_sdio.c       | 2 +-
>  4 files changed, 9 insertions(+), 9 deletions(-)

Just combine all 4 patches into one patch.

regards,
dan carpenter