]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
staging: rtl8723bs: hal: simplify boolean comparisons
authorAndrei Khomenkov <khomenkov@mailbox.org>
Mon, 27 Apr 2026 17:58:40 +0000 (20:58 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 May 2026 14:52:06 +0000 (16:52 +0200)
Simplify boolean comparisons to improve readability
and conform to the Linux kernel coding style.

Signed-off-by: Andrei Khomenkov <khomenkov@mailbox.org>
Link: https://patch.msgid.link/20260427175846.23470-3-khomenkov@mailbox.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
13 files changed:
drivers/staging/rtl8723bs/hal/hal_btcoex.c
drivers/staging/rtl8723bs/hal/hal_com.c
drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
drivers/staging/rtl8723bs/hal/hal_intf.c
drivers/staging/rtl8723bs/hal/odm.c
drivers/staging/rtl8723bs/hal/odm_CfoTracking.c
drivers/staging/rtl8723bs/hal/odm_DIG.c
drivers/staging/rtl8723bs/hal/rtl8723b_cmd.c
drivers/staging/rtl8723bs/hal/rtl8723b_dm.c
drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c
drivers/staging/rtl8723bs/hal/rtl8723bs_recv.c
drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c
drivers/staging/rtl8723bs/hal/sdio_halinit.c

index 81d25a4404c101ad65c7ef76814652dc1ab7ea3a..735321624e1b644d2e65ac6eb9c65aa455ff5afa 100644 (file)
@@ -164,8 +164,8 @@ static u8 halbtcoutsrc_IsWifiBusy(struct adapter *padapter)
 
        pmlmepriv = &padapter->mlmepriv;
 
-       if (check_fwstate(pmlmepriv, WIFI_ASOC_STATE) == true) {
-               if (check_fwstate(pmlmepriv, WIFI_AP_STATE) == true)
+       if (check_fwstate(pmlmepriv, WIFI_ASOC_STATE)) {
+               if (check_fwstate(pmlmepriv, WIFI_AP_STATE))
                        return true;
                if (pmlmepriv->link_detect_info.busy_traffic)
                        return true;
@@ -185,8 +185,8 @@ static u32 _halbtcoutsrc_GetWifiLinkStatus(struct adapter *padapter)
        bp2p = false;
        portConnectedStatus = 0;
 
-       if (check_fwstate(pmlmepriv, WIFI_ASOC_STATE) == true) {
-               if (check_fwstate(pmlmepriv, WIFI_AP_STATE) == true) {
+       if (check_fwstate(pmlmepriv, WIFI_ASOC_STATE)) {
+               if (check_fwstate(pmlmepriv, WIFI_AP_STATE)) {
                        if (bp2p)
                                portConnectedStatus |= WIFI_P2P_GO_CONNECTED;
                        else
@@ -528,7 +528,7 @@ static u8 halbtcoutsrc_Set(void *pBtcContext, u8 setType, void *pInBuf)
        case BTC_SET_ACT_UPDATE_RAMASK:
                pBtCoexist->btInfo.raMask = *pU4Tmp;
 
-               if (check_fwstate(&padapter->mlmepriv, WIFI_ASOC_STATE) == true) {
+               if (check_fwstate(&padapter->mlmepriv, WIFI_ASOC_STATE)) {
                        struct sta_info *psta;
                        struct wlan_bssid_ex *cur_network;
 
index b4b135a9ca22675ba6e8103dc51ab9406a8d955a..7fd6e0249390b0b096ea78bda4a5a00751c29fd2 100644 (file)
@@ -94,7 +94,7 @@ bool HAL_IsLegalChannel(struct adapter *adapter, u32 Channel)
        bool bLegalChannel = true;
 
        if ((Channel <= 14) && (Channel >= 1)) {
-               if (is_supported_24g(adapter->registrypriv.wireless_mode) == false)
+               if (!is_supported_24g(adapter->registrypriv.wireless_mode))
                        bLegalChannel = false;
        } else {
                bLegalChannel = false;
@@ -563,7 +563,7 @@ void SetHwReg(struct adapter *adapter, u8 variable, u8 *val)
                odm->SupportAbility = *((u32 *)val);
                break;
        case HW_VAR_DM_FUNC_OP:
-               if (*((u8 *)val) == true) {
+               if (*((u8 *)val)) {
                        /* save dm flag */
                        odm->BK_SupportAbility = odm->SupportAbility;
                } else {
index 45dbe1782bae48f3bebdaa3c661241b23baa008b..1e5669b17d0f7ed7b957d69fb45f8fcfacc3ae84 100644 (file)
@@ -461,7 +461,7 @@ u8 PHY_GetTxPowerIndexBase(
        u8 txPower = 0;
        u8 chnlIdx = (Channel-1);
 
-       if (HAL_IsLegalChannel(padapter, Channel) == false)
+       if (!HAL_IsLegalChannel(padapter, Channel))
                chnlIdx = 0;
 
        if (IS_CCK_RATE(Rate))
@@ -489,7 +489,7 @@ s8 PHY_GetTxPowerTrackingOffset(struct adapter *padapter, u8 RFPath, u8 Rate)
        struct dm_odm_t *pDM_Odm = &pHalData->odmpriv;
        s8 offset = 0;
 
-       if (pDM_Odm->RFCalibrateInfo.TxPowerTrackControl  == false)
+       if (!pDM_Odm->RFCalibrateInfo.TxPowerTrackControl)
                return offset;
 
        if ((Rate == MGN_1M) || (Rate == MGN_2M) || (Rate == MGN_5_5M) || (Rate == MGN_11M))
index f75b8a8192b8b090e3cddd2d9a13067d148fff97..03f23b75d5329a207d497f864ffc0a8fe79037f7 100644 (file)
@@ -119,7 +119,7 @@ s32 rtw_hal_mgnt_xmit(struct adapter *padapter, struct xmit_frame *pmgntframe)
        /* pwlanhdr = (struct rtw_ieee80211_hdr *)pframe; */
        /* memcpy(pmgntframe->attrib.ra, pwlanhdr->addr1, ETH_ALEN); */
 
-       if (padapter->securitypriv.binstallBIPkey == true) {
+       if (padapter->securitypriv.binstallBIPkey) {
                if (is_multicast_ether_addr(pmgntframe->attrib.ra)) {
                        pmgntframe->attrib.encrypt = _BIP_;
                        /* pmgntframe->attrib.bswenc = true; */
@@ -165,7 +165,7 @@ void rtw_hal_update_ra_mask(struct sta_info *psta, u8 rssi_level)
 
        pmlmepriv = &(padapter->mlmepriv);
 
-       if (check_fwstate(pmlmepriv, WIFI_AP_STATE) == true)
+       if (check_fwstate(pmlmepriv, WIFI_AP_STATE))
                add_ratid(padapter, psta, rssi_level);
        else {
                UpdateHalRAMask8723B(padapter, psta->mac_id, rssi_level);
index 3dffa76207682b4d0c0276716c23237fbbd01bbb..b01e9016e30cbc9d1cba8bfd93b1b43ee4ea4caf 100644 (file)
@@ -381,7 +381,7 @@ static void FindMinimumRSSI(struct adapter *padapter)
        /* 1 1.Determine the minimum RSSI */
 
        if (
-               (pDM_Odm->bLinked != true) &&
+               !pDM_Odm->bLinked &&
                (pdmpriv->EntryMinUndecoratedSmoothedPWDB == 0)
        ) {
                pdmpriv->MinUndecoratedPWDBForDM = 0;
@@ -400,7 +400,7 @@ static void odm_RSSIMonitorCheckCE(struct dm_odm_t *pDM_Odm)
        u32 PWDB_rssi[NUM_STA] = {0};/* 0~15]:MACID, [16~31]:PWDB_rssi */
        struct ra_t *pRA_Table = &pDM_Odm->DM_RA_Table;
 
-       if (pDM_Odm->bLinked != true)
+       if (!pDM_Odm->bLinked)
                return;
 
        pRA_Table->firstconnect = pDM_Odm->bLinked;
@@ -431,7 +431,7 @@ static void odm_RSSIMonitorCheckCE(struct dm_odm_t *pDM_Odm)
 
                for (i = 0; i < sta_cnt; i++) {
                        if (PWDB_rssi[i] != (0)) {
-                               if (pHalData->fw_ractrl == true)/*  Report every sta's RSSI to FW */
+                               if (pHalData->fw_ractrl)/*  Report every sta's RSSI to FW */
                                        rtl8723b_set_rssi_cmd(Adapter, (u8 *)(&PWDB_rssi[i]));
                        }
                }
@@ -623,7 +623,7 @@ void ODM_DMWatchdog(struct dm_odm_t *pDM_Odm)
        }
        odm_CCKPacketDetectionThresh(pDM_Odm);
 
-       if (*(pDM_Odm->pbPowerSaving) == true)
+       if (*pDM_Odm->pbPowerSaving)
                return;
 
 
index 0eaedd8f6469716dc1180348f840a8c2b35e8c5b..7e6b49399abbfdd72cd3dfa1a5723ef0a8e748b8 100644 (file)
@@ -138,7 +138,7 @@ void ODM_CfoTracking(void *pDM_VOID)
                pCfoTrack->CFO_ave_pre = CFO_ave;
 
                /* 4 1.4 Dynamic Xtal threshold */
-               if (pCfoTrack->bAdjust == false) {
+               if (!pCfoTrack->bAdjust) {
                        if (CFO_ave > CFO_TH_XTAL_HIGH || CFO_ave < (-CFO_TH_XTAL_HIGH))
                                pCfoTrack->bAdjust = true;
                } else {
index a31c8368f9d990c0524467cb1592db5a01a6bdd1..426989986cc3775aca2b633f02057fbead3541e7 100644 (file)
@@ -82,7 +82,7 @@ void odm_NHMBB(void *pDM_VOID)
 
 
        if ((pDM_Odm->NHMCurTxOkcnt) + 1 > (u64)(pDM_Odm->NHMCurRxOkcnt<<2) + 1) { /* Tx > 4*Rx possible for adaptivity test */
-               if (pDM_Odm->NHM_cnt_0 >= 190 || pDM_Odm->adaptivity_flag == true) {
+               if (pDM_Odm->NHM_cnt_0 >= 190 || pDM_Odm->adaptivity_flag) {
                        /* Enable EDCCA since it is possible running Adaptivity testing */
                        /* test_status = 1; */
                        pDM_Odm->adaptivity_flag = true;
@@ -99,7 +99,7 @@ void odm_NHMBB(void *pDM_VOID)
                        }
                }
        } else { /*  TX<RX */
-               if (pDM_Odm->adaptivity_flag == true && pDM_Odm->NHM_cnt_0 <= 200) {
+               if (pDM_Odm->adaptivity_flag && pDM_Odm->NHM_cnt_0 <= 200) {
                        /* test_status = 2; */
                        pDM_Odm->tolerance_cnt = 0;
                } else {
@@ -189,7 +189,7 @@ void odm_AdaptivityInit(void *pDM_VOID)
 {
        struct dm_odm_t *pDM_Odm = (struct dm_odm_t *)pDM_VOID;
 
-       if (pDM_Odm->Carrier_Sense_enable == false)
+       if (!pDM_Odm->Carrier_Sense_enable)
                pDM_Odm->TH_L2H_ini = 0xf7; /*  -7 */
        else
                pDM_Odm->TH_L2H_ini = 0xa;
@@ -232,7 +232,7 @@ void odm_Adaptivity(void *pDM_VOID, u8 IGI)
        pDM_Odm->IGI_target = (u8) IGI_target;
 
        /* Search pwdB lower bound */
-       if (pDM_Odm->TxHangFlg == true) {
+       if (pDM_Odm->TxHangFlg) {
                PHY_SetBBReg(pDM_Odm->Adapter, ODM_REG_DBG_RPT_11N, bMaskDWord, 0x208);
                odm_SearchPwdBLowerBound(pDM_Odm, pDM_Odm->IGI_target);
        }
@@ -253,9 +253,9 @@ void odm_Adaptivity(void *pDM_VOID, u8 IGI)
 
        if (
                pDM_Odm->bLinked &&
-               pDM_Odm->Carrier_Sense_enable == false &&
-               pDM_Odm->NHM_disable == false &&
-               pDM_Odm->TxHangFlg == false
+               !pDM_Odm->Carrier_Sense_enable &&
+               !pDM_Odm->NHM_disable &&
+               !pDM_Odm->TxHangFlg
        )
                odm_NHMBB(pDM_Odm);
 
@@ -323,7 +323,7 @@ bool odm_DigAbort(void *pDM_VOID)
                return  true;
 
        /* add by Neil Chen to avoid PSD is processing */
-       if (pDM_Odm->bDMInitialGainEnable == false)
+       if (!pDM_Odm->bDMInitialGainEnable)
                return  true;
 
        return  false;
@@ -387,14 +387,14 @@ void odm_DIG(void *pDM_VOID)
        if (odm_DigAbort(pDM_Odm))
                return;
 
-       if (pDM_Odm->adaptivity_flag == true)
+       if (pDM_Odm->adaptivity_flag)
                Adap_IGI_Upper = pDM_Odm->Adaptivity_IGI_upper;
 
 
        /* 1 Update status */
        DIG_Dynamic_MIN = pDM_DigTable->DIG_Dynamic_MIN_0;
-       FirstConnect = (pDM_Odm->bLinked) && (pDM_DigTable->bMediaConnect_0 == false);
-       FirstDisConnect = (!pDM_Odm->bLinked) && (pDM_DigTable->bMediaConnect_0 == true);
+       FirstConnect = (pDM_Odm->bLinked) && !pDM_DigTable->bMediaConnect_0;
+       FirstDisConnect = (!pDM_Odm->bLinked) && pDM_DigTable->bMediaConnect_0;
 
        /* 1 Boundary Decision */
        /* 2 For WIN\CE */
@@ -529,7 +529,7 @@ void odm_DIG(void *pDM_VOID)
        /* 1 Force upper bound and lower bound for adaptivity */
        if (
                pDM_Odm->SupportAbility & ODM_BB_ADAPTIVITY &&
-               pDM_Odm->adaptivity_flag == true
+               pDM_Odm->adaptivity_flag
        ) {
                if (CurrentIGI > Adap_IGI_Upper)
                        CurrentIGI = Adap_IGI_Upper;
index 4bdc8e314015b89d4b70758e0602a4f963111ea1..6c93371796457edc86e99dc7504aabbaa4c9e097 100644 (file)
@@ -359,7 +359,7 @@ void rtl8723b_set_FwPwrMode_cmd(struct adapter *padapter, u8 psmode)
        }
 
        if (psmode > 0) {
-               if (hal_btcoex_IsBtControlLps(padapter) == true) {
+               if (hal_btcoex_IsBtControlLps(padapter)) {
                        PowerState = hal_btcoex_RpwmVal(padapter);
                        byte5 = hal_btcoex_LpsVal(padapter);
 
index 928226679ab4591f46dee6d37a9ce6202aa9644e..5cb13a10efd62775d961d33df60190be648a20f9 100644 (file)
@@ -131,21 +131,21 @@ void rtl8723b_HalDmWatchDog(struct adapter *Adapter)
 
        hw_init_completed = Adapter->hw_init_completed;
 
-       if (hw_init_completed == false)
+       if (!hw_init_completed)
                goto skip_dm;
 
        fw_current_in_ps_mode = adapter_to_pwrctl(Adapter)->fw_current_in_ps_mode;
        rtw_hal_get_hwreg(Adapter, HW_VAR_FWLPS_RF_ON, (u8 *)(&bFwPSAwake));
 
        if (
-               (hw_init_completed == true) &&
-               ((!fw_current_in_ps_mode) && bFwPSAwake)
+               hw_init_completed &&
+               (!fw_current_in_ps_mode && bFwPSAwake)
        ) {
                rtw_hal_check_rxfifo_full(Adapter);
        }
 
        /* ODM */
-       if (hw_init_completed == true) {
+       if (hw_init_completed) {
                u8 bLinked = false;
                u8 bsta_state = false;
                bool bBtDisabled = true;
@@ -207,7 +207,7 @@ void rtl8723b_HalDmWatchDog_in_LPS(struct adapter *Adapter)
        struct sta_priv *pstapriv = &Adapter->stapriv;
        struct sta_info *psta = NULL;
 
-       if (Adapter->hw_init_completed == false)
+       if (!Adapter->hw_init_completed)
                goto skip_lps_dm;
 
 
@@ -216,7 +216,7 @@ void rtl8723b_HalDmWatchDog_in_LPS(struct adapter *Adapter)
 
        ODM_CmnInfoUpdate(&pHalData->odmpriv, ODM_CMNINFO_LINK, bLinked);
 
-       if (bLinked == false)
+       if (!bLinked)
                goto skip_lps_dm;
 
        if (!(pDM_Odm->SupportAbility & ODM_BB_RSSI_MONITOR))
index 9a91fd0a96eb2724586c4cf1c4390c931c7151f9..aa469fe57ba84596f693022d8188fcb7c4982513 100644 (file)
@@ -738,7 +738,7 @@ static void hal_ReadEFuse_BT(
        Hal_GetEfuseDefinition(padapter, EFUSE_BT, TYPE_AVAILABLE_EFUSE_BYTES_BANK, &total);
 
        for (bank = 1; bank < 3; bank++) { /*  8723b Max bake 0~2 */
-               if (hal_EfuseSwitchToBank(padapter, bank) == false)
+               if (!hal_EfuseSwitchToBank(padapter, bank))
                        goto exit;
 
                eFuse_Addr = 0;
@@ -882,7 +882,7 @@ void rtl8723b_InitBeaconParameters(struct adapter *padapter)
        rtw_write16(padapter, REG_TBTT_PROHIBIT, 0x6404);/*  ms */
        /*  Firmware will control REG_DRVERLYINT when power saving is enable, */
        /*  so don't set this register on STA mode. */
-       if (check_fwstate(&padapter->mlmepriv, WIFI_STATION_STATE) == false)
+       if (!check_fwstate(&padapter->mlmepriv, WIFI_STATION_STATE))
                rtw_write8(padapter, REG_DRVERLYINT, DRIVER_EARLY_INT_TIME_8723B); /*  5ms */
        rtw_write8(padapter, REG_BCNDMATIM, BCN_DMA_ATIME_INT_TIME_8723B); /*  2ms */
 
@@ -998,7 +998,7 @@ void rtl8723b_SetBeaconRelatedRegisters(struct adapter *padapter)
        rtw_write32(padapter, REG_TCR, value32);
 
        /*  NOTE: Fix test chip's bug (about contention windows's randomness) */
-       if (check_fwstate(&padapter->mlmepriv, WIFI_ADHOC_STATE|WIFI_ADHOC_MASTER_STATE|WIFI_AP_STATE) == true) {
+       if (check_fwstate(&padapter->mlmepriv, WIFI_ADHOC_STATE|WIFI_ADHOC_MASTER_STATE|WIFI_AP_STATE)) {
                rtw_write8(padapter, REG_RXTSF_OFFSET_CCK, 0x50);
                rtw_write8(padapter, REG_RXTSF_OFFSET_OFDM, 0x50);
        }
@@ -2119,7 +2119,7 @@ static void hw_var_set_mlme_sitesurvey(struct adapter *padapter, u8 variable, u8
        /*  config RCR to receive different BSSID & not to receive data frame */
        value_rxfltmap2 = 0;
 
-       if ((check_fwstate(pmlmepriv, WIFI_AP_STATE) == true))
+       if (check_fwstate(pmlmepriv, WIFI_AP_STATE))
                rcr_clear_bit = RCR_CBSSID_BCN;
 
        value_rcr = rtw_read32(padapter, REG_RCR);
@@ -2186,7 +2186,7 @@ static void hw_var_set_mlme_join(struct adapter *padapter, u8 variable, u8 *val)
                        val32 |= RCR_CBSSID_DATA|RCR_CBSSID_BCN;
                rtw_write32(padapter, REG_RCR, val32);
 
-               if (check_fwstate(pmlmepriv, WIFI_STATION_STATE) == true)
+               if (check_fwstate(pmlmepriv, WIFI_STATION_STATE))
                        RetryLimit = (pEEPROM->CustomerID == RT_CID_CCX) ? 7 : 48;
                else /*  Ad-hoc Mode */
                        RetryLimit = 0x7;
@@ -2724,7 +2724,7 @@ void SetHwReg8723B(struct adapter *padapter, u8 variable, u8 *val)
                break;
 
        case HW_VAR_DL_RSVD_PAGE:
-               if (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == true)
+               if (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE))
                        rtl8723b_download_BTCoex_AP_mode_rsvd_page(padapter);
                else
                        rtl8723b_download_rsvd_page(padapter, RT_MEDIA_CONNECT);
index cab4707091e2b8d3e2d74fc3501ed0648a6e0db5..888029e21dcb70b22b37b6082cd03db186822fda 100644 (file)
@@ -135,7 +135,7 @@ static void update_recvframe_phyinfo(union recv_frame *precvframe,
        precvframe->u.hdr.psta = NULL;
        if (
                pkt_info.bssid_match &&
-               (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == true)
+               (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE))
        ) {
                if (psta) {
                        precvframe->u.hdr.psta = psta;
index ff39077deb698c975d50981dc6f0a2fc83074e26..705becfe60439141b08e452e472cd8f72b7f8c38 100644 (file)
@@ -93,7 +93,7 @@ query_free_page:
        )
                goto free_xmitbuf;
 
-       if (rtw_sdio_wait_enough_TxOQT_space(padapter, pxmitbuf->agg_num) == false)
+       if (!rtw_sdio_wait_enough_TxOQT_space(padapter, pxmitbuf->agg_num))
                goto free_xmitbuf;
 
        traffic_check_for_leave_lps(padapter, true, pxmitbuf->agg_num);
@@ -225,7 +225,7 @@ static s32 xmit_xmitframes(struct adapter *padapter, struct xmit_priv *pxmitpriv
 
                        frame_phead = get_list_head(pframe_queue);
 
-                       while (list_empty(frame_phead) == false) {
+                       while (!list_empty(frame_phead)) {
                                frame_plist = get_next(frame_phead);
                                pxmitframe = container_of(frame_plist, struct xmit_frame, list);
 
@@ -269,7 +269,7 @@ static s32 xmit_xmitframes(struct adapter *padapter, struct xmit_priv *pxmitpriv
                                }
 
                                /*  ok to send, remove frame from queue */
-                               if (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE) == true)
+                               if (check_fwstate(&padapter->mlmepriv, WIFI_AP_STATE))
                                        if (
                                                (pxmitframe->attrib.psta->state & WIFI_SLEEP_STATE) &&
                                                (pxmitframe->attrib.triggered == 0)
@@ -566,7 +566,7 @@ void rtl8723bs_free_xmit_priv(struct adapter *padapter)
        spin_unlock_bh(&pqueue->lock);
 
        phead = &tmplist;
-       while (list_empty(phead) == false) {
+       while (!list_empty(phead)) {
                plist = get_next(phead);
                list_del_init(plist);
 
index b3366830edbb37c852e972525b5f8dffd2ec015a..f27970ef54c8f797d569b211748642e8139a65c6 100644 (file)
@@ -584,7 +584,7 @@ u32 rtl8723bs_hal_init(struct adapter *padapter)
        pwrctrlpriv = adapter_to_pwrctl(padapter);
 
        if (
-               adapter_to_pwrctl(padapter)->bips_processing == true &&
+               adapter_to_pwrctl(padapter)->bips_processing &&
                adapter_to_pwrctl(padapter)->pre_ips_type == 0
        ) {
                unsigned long start_time;