From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754506AbbINMxI (ORCPT ); Mon, 14 Sep 2015 08:53:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47194 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753096AbbINMww (ORCPT ); Mon, 14 Sep 2015 08:52:52 -0400 From: Jes Sorensen To: Shraddha Barke Cc: Greg Kroah-Hartman , Julia Lawall , linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/3] Staging: rtl8723au: core: Bool tests don't need comparisons References: <1442161641-7598-1-git-send-email-shraddha.6596@gmail.com> <1442161641-7598-2-git-send-email-shraddha.6596@gmail.com> Date: Mon, 14 Sep 2015 08:52:51 -0400 In-Reply-To: <1442161641-7598-2-git-send-email-shraddha.6596@gmail.com> (Shraddha Barke's message of "Sun, 13 Sep 2015 21:57:20 +0530") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Shraddha Barke writes: > This patch removes comparisons to true/false values on bool variables. > Please do not send patches for multiple different drivers in the same patch set. In addition, if you post more than one patch, you need to include a cover letter. Jes > > > Signed-off-by: Shraddha Barke > --- > Changes in v4- > No change > > drivers/staging/rtl8723au/core/rtw_ap.c | 10 +++++----- > drivers/staging/rtl8723au/core/rtw_mlme_ext.c | 28 +++++++++++++-------------- > 2 files changed, 19 insertions(+), 19 deletions(-) > > diff --git a/drivers/staging/rtl8723au/core/rtw_ap.c b/drivers/staging/rtl8723au/core/rtw_ap.c > index b96e7b6..7f46b7a 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 != > @@ -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,7 +1882,7 @@ 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); > 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;