diff options
| author | Srinivas Girigowda <sgirigow@codeaurora.org> | 2017-08-03 11:28:32 -0700 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-08-04 11:26:31 -0700 |
| commit | fc1340066e7f76ded31c33fcedf76c4a6f45ef69 (patch) | |
| tree | efb5d7952b3d915a70972a1bb3280cf297c2cb4e | |
| parent | 480fa2891a421e4ed1fd138ff28e920a29aecb4c (diff) | |
qcacld-3.0: clang: Remove extraneous parentheses
clang is reporting these compiler warnings.
Remove extraneous parentheses.
Change-Id: I3dd6490cc1bf2a038e72e1630ba9b0c2b5f0ebed
CRs-Fixed: 2087229
| -rw-r--r-- | core/mac/src/pe/lim/lim_process_action_frame.c | 3 | ||||
| -rw-r--r-- | core/mac/src/pe/lim/lim_session.c | 9 | ||||
| -rw-r--r-- | core/mac/src/pe/rrm/rrm_api.c | 2 | ||||
| -rw-r--r-- | core/mac/src/pe/sch/sch_beacon_gen.c | 2 | ||||
| -rw-r--r-- | core/sap/dfs/src/dfs_bindetects.c | 4 | ||||
| -rw-r--r-- | core/sap/src/sap_fsm.c | 4 | ||||
| -rw-r--r-- | core/sme/src/csr/csr_api_roam.c | 4 | ||||
| -rw-r--r-- | core/sme/src/csr/csr_api_scan.c | 2 |
8 files changed, 15 insertions, 15 deletions
diff --git a/core/mac/src/pe/lim/lim_process_action_frame.c b/core/mac/src/pe/lim/lim_process_action_frame.c index 7f2dce7adaa5..ba37f4a82966 100644 --- a/core/mac/src/pe/lim/lim_process_action_frame.c +++ b/core/mac/src/pe/lim/lim_process_action_frame.c @@ -981,8 +981,7 @@ static void __lim_process_del_ts_req(tpAniSirGlobal mac_ctx, tsinfo = delts.wmeTspecPresent ? &delts.tspec.tsinfo : &delts.tsinfo; /* if no Admit Control, ignore the request */ - if ((tsinfo->traffic.accessPolicy == SIR_MAC_ACCESSPOLICY_EDCA)) { - + if (tsinfo->traffic.accessPolicy == SIR_MAC_ACCESSPOLICY_EDCA) { if (upToAc(tsinfo->traffic.userPrio) >= MAX_NUM_AC) { pe_warn("DelTs with UP: %d has no AC - ignoring req", tsinfo->traffic.userPrio); diff --git a/core/mac/src/pe/lim/lim_session.c b/core/mac/src/pe/lim/lim_session.c index f3dbf51492b2..a1c7acc571b4 100644 --- a/core/mac/src/pe/lim/lim_session.c +++ b/core/mac/src/pe/lim/lim_session.c @@ -618,17 +618,18 @@ tpPESession pe_find_session_by_bss_idx(tpAniSirGlobal pMac, uint8_t bssIdx) \sa --------------------------------------------------------------------------*/ -tpPESession pe_find_session_by_session_id(tpAniSirGlobal pMac, uint8_t sessionId) +tpPESession pe_find_session_by_session_id(tpAniSirGlobal pMac, + uint8_t sessionId) { if (sessionId >= pMac->lim.maxBssId) { pe_err("Invalid sessionId: %d", sessionId); return NULL; } - if ((pMac->lim.gpSession[sessionId].valid == true)) { + + if (pMac->lim.gpSession[sessionId].valid) return &pMac->lim.gpSession[sessionId]; - } - return NULL; + return NULL; } /** diff --git a/core/mac/src/pe/rrm/rrm_api.c b/core/mac/src/pe/rrm/rrm_api.c index 3480f682f2d9..22ed28638979 100644 --- a/core/mac/src/pe/rrm/rrm_api.c +++ b/core/mac/src/pe/rrm/rrm_api.c @@ -210,7 +210,7 @@ tSirRetStatus rrm_set_max_tx_power_rsp(tpAniSirGlobal pMac, tpSirMsgQ limMsgQ) if (qdf_is_macaddr_broadcast(&pMaxTxParams->bssId)) { for (i = 0; i < pMac->lim.maxBssId; i++) { - if ((pMac->lim.gpSession[i].valid == true)) { + if (pMac->lim.gpSession[i].valid == true) { pSessionEntry = &pMac->lim.gpSession[i]; rrm_cache_mgmt_tx_power(pMac, pMaxTxParams->power, pSessionEntry); diff --git a/core/mac/src/pe/sch/sch_beacon_gen.c b/core/mac/src/pe/sch/sch_beacon_gen.c index 3df29b6e15b3..60e594d705cd 100644 --- a/core/mac/src/pe/sch/sch_beacon_gen.c +++ b/core/mac/src/pe/sch/sch_beacon_gen.c @@ -439,7 +439,7 @@ sch_set_fixed_beacon_fields(tpAniSirGlobal mac_ctx, tpPESession session) } } - if ((LIM_IS_AP_ROLE(session))) { + if (LIM_IS_AP_ROLE(session)) { /* * Can be efficiently updated whenever new IE added in Probe * response in future diff --git a/core/sap/dfs/src/dfs_bindetects.c b/core/sap/dfs/src/dfs_bindetects.c index dc3acf6dabd6..2c3b799ddbcb 100644 --- a/core/sap/dfs/src/dfs_bindetects.c +++ b/core/sap/dfs/src/dfs_bindetects.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2002-2014 The Linux Foundation. All rights reserved. + * Copyright (c) 2002-2014, 2017 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -458,7 +458,7 @@ dfs_bin_pri_check(struct ath_dfs *dfs, struct dfs_filter *rf, } if (primatch && (deltadur < durmargin)) { - if ((numpulses == 1)) { + if (numpulses == 1) { numpulses++; } else { delta_time_stamps = diff --git a/core/sap/src/sap_fsm.c b/core/sap/src/sap_fsm.c index 1bbfc760e78d..dec8eaff1545 100644 --- a/core/sap/src/sap_fsm.c +++ b/core/sap/src/sap_fsm.c @@ -4754,8 +4754,8 @@ static QDF_STATUS sap_get_channel_list(ptSapContext sap_ctx, #ifdef FEATURE_WLAN_CH_AVOID for (i = 0; i < NUM_CHANNELS; i++) { - if ((safe_channels[i].channelNumber == - CDS_CHANNEL_NUM(loop_count))) { + if (safe_channels[i].channelNumber == + CDS_CHANNEL_NUM(loop_count)) { /* Check if channel is safe */ if (true == safe_channels[i].isSafe) { #endif diff --git a/core/sme/src/csr/csr_api_roam.c b/core/sme/src/csr/csr_api_roam.c index c40d48175bbc..2d4d60891114 100644 --- a/core/sme/src/csr/csr_api_roam.c +++ b/core/sme/src/csr/csr_api_roam.c @@ -20966,8 +20966,8 @@ static QDF_STATUS csr_process_roam_sync_callback(tpAniSirGlobal mac_ctx, * with AP after this point and sends new keys to the driver. * Driver starts wait_for_key timer for that purpose. */ - if ((roam_synch_data->authStatus - == CSR_ROAM_AUTH_STATUS_AUTHENTICATED)) { + if (roam_synch_data->authStatus + == CSR_ROAM_AUTH_STATUS_AUTHENTICATED) { QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_DEBUG, FL("LFR3:Don't start waitforkey timer")); diff --git a/core/sme/src/csr/csr_api_scan.c b/core/sme/src/csr/csr_api_scan.c index 98efd5ecf4dd..70fa782438d2 100644 --- a/core/sme/src/csr/csr_api_scan.c +++ b/core/sme/src/csr/csr_api_scan.c @@ -5687,7 +5687,7 @@ static bool csr_scan_filter_given_chnl_band(tpAniSirGlobal mac_ctx, * In case if no-concurrent IBSS session exist then scan * full band */ - if ((dst_req->ChannelInfo.numOfChannels == 0)) { + if (dst_req->ChannelInfo.numOfChannels == 0) { csr_get_cfg_valid_channels(mac_ctx, valid_chnl_list, &valid_chnl_len); } else { |
