From f37bdc4b800cf9f895c498caa92ee4a7e5d80c9d Mon Sep 17 00:00:00 2001 From: Moksh Panicker Date: Mon, 15 Jun 2026 08:25:38 +0000 Subject: [PATCH] staging: rtl8723bs: Fix missing spaces around operators Add missing spaces around arithmetic and bitwise operators ('+', '-', '&') in the rtl8723bs staging driver across multiple files. These were flagged by checkpatch.pl with: CHECK: spaces preferred around that 'X' (ctx:VxV) Signed-off-by: Moksh Panicker Reviewed-by: Dan Carpenter Link: https://patch.msgid.link/20260615082538.12007-1-mokshpanicker.7@gmail.com Signed-off-by: Greg Kroah-Hartman --- 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(-) diff --git a/drivers/staging/rtl8723bs/hal/HalPhyRf.c b/drivers/staging/rtl8723bs/hal/HalPhyRf.c index d0dcd8a56b27..bf44c1050be5 100644 --- a/drivers/staging/rtl8723bs/hal/HalPhyRf.c +++ b/drivers/staging/rtl8723bs/hal/HalPhyRf.c @@ -219,13 +219,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 { diff --git a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c index 7527ff051865..c254e3481cc2 100644 --- a/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c +++ b/drivers/staging/rtl8723bs/hal/HalPwrSeqCmd.c @@ -87,7 +87,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 */ @@ -105,7 +105,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)) @@ -125,7 +125,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: diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c index b16456d5b6bb..f886699104e5 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)) chnlIdx = 0; diff --git a/drivers/staging/rtl8723bs/hal/hal_sdio.c b/drivers/staging/rtl8723bs/hal/hal_sdio.c index 1a865d4de8ac..1bd165a4573f 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;