This patch removes comparisons to true/false values on bool variables. Changes in v3- Made fix manually
Signed-off-by: Shraddha Barke <shraddha.6...@gmail.com> --- drivers/staging/rtl8723au/core/rtw_ap.c | 13 ++++++------- drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 28 +++++++++++++-------------- 2 files changed, 20 insertions(+), 21 deletions(-) diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging/rtl8723au/core/rtw_ap.c index 65b209a..b6bfbbf 100644 --- a/drivers/staging/rtl8723au/core/rtw_ap.c +++ b/drivers/staging/rtl8723au/core/rtw_ap.c @@ -409,7 +409,7 @@ void add_RATid23a(struct rtw_adapter *padapter, struct sta_info *psta, u8 rssi_l arg |= BIT(7);/* support entry 2~31 */ - if (shortGIrate == true) + if (shortGIrate) arg |= BIT(5); tx_ra_bitmap |= ((raid<<28)&0xf0000000); @@ -424,7 +424,7 @@ void add_RATid23a(struct rtw_adapter *padapter, struct sta_info *psta, u8 rssi_l /* arg[5] = Short GI */ rtl8723a_add_rateatid(padapter, tx_ra_bitmap, arg, rssi_level); - if (shortGIrate == true) + if (shortGIrate) init_rate |= BIT(6); /* set ra_id, init_rate */ @@ -662,7 +662,7 @@ static void start_bss_network(struct rtw_adapter *padapter, u8 *pbuf) update_hw_ht_param(padapter); } - if (pmlmepriv->cur_network.join_res != true) { + if (!pmlmepriv->cur_network.join_res) { /* setting only at first time */ /* WEP Key will be set before this function, do not clear CAM. */ if (psecuritypriv->dot11PrivacyAlgrthm != @@ -1061,7 +1061,7 @@ int rtw_acl_add_sta23a(struct rtw_adapter *padapter, u8 *addr) paclnode = container_of(plist, struct rtw_wlan_acl_node, list); if (!memcmp(paclnode->addr, addr, ETH_ALEN)) { - if (paclnode->valid == true) { + if (paclnode->valid) { added = true; DBG_8723A("%s, sta has been added\n", __func__); break; @@ -1370,7 +1370,7 @@ static int rtw_ht_operation_update(struct rtw_adapter *padapter) void associated_clients_update23a(struct rtw_adapter *padapter, u8 updated) { /* update associated stations cap. */ - if (updated == true) { + if (updated) { struct list_head *phead, *plist, *ptmp; struct sta_info *psta; struct sta_priv *pstapriv = &padapter->stapriv; @@ -1882,9 +1882,8 @@ void stop_ap_mode23a(struct rtw_adapter *padapter) list_for_each_safe(plist, ptmp, phead) { paclnode = container_of(plist, struct rtw_wlan_acl_node, list); - if (paclnode->valid == true) { + if (paclnode->valid) { paclnode->valid = false; - list_del_init(&paclnode->list); pacl_list->num--; diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c index be9a3d5..65ef4a4 100644 --- a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c +++ b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c @@ -602,7 +602,7 @@ void free_mlme_ext_priv23a (struct mlme_ext_priv *pmlmeext) if (!padapter) return; - if (padapter->bDriverStopped == true) { + if (padapter->bDriverStopped) { del_timer_sync(&pmlmeext->survey_timer); del_timer_sync(&pmlmeext->link_timer); /* del_timer_sync(&pmlmeext->ADDBA_timer); */ @@ -959,7 +959,7 @@ OnAuth23a(struct rtw_adapter *padapter, struct recv_frame *precv_frame) goto auth_fail; } - if (rtw_access_ctrl23a(padapter, sa) == false) { + if (!rtw_access_ctrl23a(padapter, sa)) { status = WLAN_STATUS_AP_UNABLE_TO_HANDLE_NEW_STA; goto auth_fail; } @@ -2049,7 +2049,7 @@ static int OnAction23a_back23a(struct rtw_adapter *padapter, sizeof(struct ADDBA_request)); process_addba_req23a(padapter, (u8 *)&pmlmeinfo->ADDBA_req, addr); - if (pmlmeinfo->bAcceptAddbaReq == true) + if (pmlmeinfo->bAcceptAddbaReq) issue_action_BA23a(padapter, addr, WLAN_ACTION_ADDBA_RESP, 0); else { @@ -2253,8 +2253,8 @@ void update_mgntframe_attrib23a(struct rtw_adapter *padapter, void dump_mgntframe23a(struct rtw_adapter *padapter, struct xmit_frame *pmgntframe) { - if (padapter->bSurpriseRemoved == true || - padapter->bDriverStopped == true) + if (padapter->bSurpriseRemoved || + padapter->bDriverStopped) return; rtl8723au_mgnt_xmit(padapter, pmgntframe); @@ -2269,8 +2269,8 @@ int dump_mgntframe23a_and_wait(struct rtw_adapter *padapter, struct xmit_buf *pxmitbuf = pmgntframe->pxmitbuf; struct submit_ctx sctx; - if (padapter->bSurpriseRemoved == true || - padapter->bDriverStopped == true) + if (padapter->bSurpriseRemoved || + padapter->bDriverStopped) return ret; rtw_sctx_init23a(&sctx, timeout_ms); @@ -2295,8 +2295,8 @@ int dump_mgntframe23a_and_wait_ack23a(struct rtw_adapter *padapter, u32 timeout_ms = 500;/* 500ms */ struct xmit_priv *pxmitpriv = &padapter->xmitpriv; - if (padapter->bSurpriseRemoved == true || - padapter->bDriverStopped == true) + if (padapter->bSurpriseRemoved || + padapter->bDriverStopped) return _FAIL; mutex_lock(&pxmitpriv->ack_tx_mutex); @@ -3942,7 +3942,7 @@ int send_delba23a(struct rtw_adapter *padapter, u8 initiator, u8 *addr) if (initiator == 0) { /* recipient */ for (tid = 0; tid < MAXTID; tid++) { - if (psta->recvreorder_ctrl[tid].enable == true) { + if (psta->recvreorder_ctrl[tid].enable) { DBG_8723A("rx agg disable tid(%d)\n", tid); issue_action_BA23a(padapter, addr, WLAN_ACTION_DELBA, (((tid <<1) |initiator)&0x1F)); psta->recvreorder_ctrl[tid].enable = false; @@ -4092,7 +4092,7 @@ static void rtw_site_survey(struct rtw_adapter *padapter) /* turn on dynamic functions */ rtl8723a_odm_support_ability_restore(padapter); - if (is_client_associated_to_ap23a(padapter) == true) + if (is_client_associated_to_ap23a(padapter)) issue_nulldata23a(padapter, NULL, 0, 3, 500); rtl8723a_mlme_sitesurvey(padapter, 0); @@ -5195,7 +5195,7 @@ void linked_status_chk23a(struct rtw_adapter *padapter) if (psta) { bool is_p2p_enable = false; - if (chk_ap_is_alive(padapter, psta) == false) + if (!chk_ap_is_alive(padapter, psta)) rx_chk = _FAIL; if (pxmitpriv->last_tx_pkts == pxmitpriv->tx_pkts) @@ -5319,7 +5319,7 @@ static void survey_timer_hdl(unsigned long data) if (pmlmeext->sitesurvey_res.state == SCAN_PROCESS) pmlmeext->sitesurvey_res.channel_idx++; - if (pmlmeext->scan_abort == true) { + if (pmlmeext->scan_abort) { pmlmeext->sitesurvey_res.channel_idx = pmlmeext->sitesurvey_res.ch_num; DBG_8723A("%s idx:%d\n", __func__, @@ -5823,7 +5823,7 @@ int sitesurvey_cmd_hdl23a(struct rtw_adapter *padapter, const u8 *pbuf) /* config the initial gain under scanning, need to write the BB registers */ - if (wdev_to_priv(padapter->rtw_wdev)->p2p_enabled == true) + if (wdev_to_priv(padapter->rtw_wdev)->p2p_enabled) initialgain = 0x30; else initialgain = 0x1E; -- 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/