| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <wrfjlhc9yxgc.fsf@redhat.com>
Date: Mon, 14 Sep 2015 08:52:51 -0400
From: Jes Sorensen <Jes.Sorensen@...hat.com>
To: Shraddha Barke <shraddha.6596@...il.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Julia Lawall <julia.lawall@...6.fr>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 1/3] Staging: rtl8723au: core: Bool tests don't need comparisons
Shraddha Barke <shraddha.6596@...il.com> 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 <shraddha.6596@...il.com>
> ---
> 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;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists