qcacld-3.0: clang: Remove extraneous parentheses
clang is reporting these compiler warnings. Remove extraneous parentheses. Change-Id: I3dd6490cc1bf2a038e72e1630ba9b0c2b5f0ebed CRs-Fixed: 2087229
This commit is contained in:

committed by
snandini

parent
0cd4be2953
commit
f326dfe224
@@ -984,8 +984,7 @@ static void __lim_process_del_ts_req(tpAniSirGlobal mac_ctx,
|
|||||||
tsinfo = delts.wmeTspecPresent ? &delts.tspec.tsinfo : &delts.tsinfo;
|
tsinfo = delts.wmeTspecPresent ? &delts.tspec.tsinfo : &delts.tsinfo;
|
||||||
|
|
||||||
/* if no Admit Control, ignore the request */
|
/* 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) {
|
if (upToAc(tsinfo->traffic.userPrio) >= MAX_NUM_AC) {
|
||||||
pe_warn("DelTs with UP: %d has no AC - ignoring req",
|
pe_warn("DelTs with UP: %d has no AC - ignoring req",
|
||||||
tsinfo->traffic.userPrio);
|
tsinfo->traffic.userPrio);
|
||||||
|
@@ -532,17 +532,18 @@ tpPESession pe_find_session_by_bss_idx(tpAniSirGlobal pMac, uint8_t bssIdx)
|
|||||||
|
|
||||||
\sa
|
\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) {
|
if (sessionId >= pMac->lim.maxBssId) {
|
||||||
pe_err("Invalid sessionId: %d", sessionId);
|
pe_err("Invalid sessionId: %d", sessionId);
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
if ((pMac->lim.gpSession[sessionId].valid == true)) {
|
|
||||||
return &pMac->lim.gpSession[sessionId];
|
|
||||||
}
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
|
if (pMac->lim.gpSession[sessionId].valid)
|
||||||
|
return &pMac->lim.gpSession[sessionId];
|
||||||
|
|
||||||
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@@ -211,7 +211,7 @@ tSirRetStatus rrm_set_max_tx_power_rsp(tpAniSirGlobal pMac,
|
|||||||
|
|
||||||
if (qdf_is_macaddr_broadcast(&pMaxTxParams->bssId)) {
|
if (qdf_is_macaddr_broadcast(&pMaxTxParams->bssId)) {
|
||||||
for (i = 0; i < pMac->lim.maxBssId; i++) {
|
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];
|
pSessionEntry = &pMac->lim.gpSession[i];
|
||||||
rrm_cache_mgmt_tx_power(pMac, pMaxTxParams->power,
|
rrm_cache_mgmt_tx_power(pMac, pMaxTxParams->power,
|
||||||
pSessionEntry);
|
pSessionEntry);
|
||||||
|
@@ -437,7 +437,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
|
* Can be efficiently updated whenever new IE added in Probe
|
||||||
* response in future
|
* response in future
|
||||||
|
@@ -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.
|
* 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 (primatch && (deltadur < durmargin)) {
|
||||||
if ((numpulses == 1)) {
|
if (numpulses == 1) {
|
||||||
numpulses++;
|
numpulses++;
|
||||||
} else {
|
} else {
|
||||||
delta_time_stamps =
|
delta_time_stamps =
|
||||||
|
@@ -4110,8 +4110,8 @@ static QDF_STATUS sap_get_channel_list(ptSapContext sap_ctx,
|
|||||||
|
|
||||||
#ifdef FEATURE_WLAN_CH_AVOID
|
#ifdef FEATURE_WLAN_CH_AVOID
|
||||||
for (i = 0; i < NUM_CHANNELS; i++) {
|
for (i = 0; i < NUM_CHANNELS; i++) {
|
||||||
if ((safe_channels[i].channelNumber ==
|
if (safe_channels[i].channelNumber ==
|
||||||
WLAN_REG_CH_NUM(loop_count))) {
|
WLAN_REG_CH_NUM(loop_count)) {
|
||||||
/* Check if channel is safe */
|
/* Check if channel is safe */
|
||||||
if (true == safe_channels[i].isSafe) {
|
if (true == safe_channels[i].isSafe) {
|
||||||
#endif
|
#endif
|
||||||
|
@@ -20083,8 +20083,8 @@ static QDF_STATUS csr_process_roam_sync_callback(tpAniSirGlobal mac_ctx,
|
|||||||
* with AP after this point and sends new keys to the driver.
|
* with AP after this point and sends new keys to the driver.
|
||||||
* Driver starts wait_for_key timer for that purpose.
|
* Driver starts wait_for_key timer for that purpose.
|
||||||
*/
|
*/
|
||||||
if ((roam_synch_data->authStatus
|
if (roam_synch_data->authStatus
|
||||||
== CSR_ROAM_AUTH_STATUS_AUTHENTICATED)) {
|
== CSR_ROAM_AUTH_STATUS_AUTHENTICATED) {
|
||||||
QDF_TRACE(QDF_MODULE_ID_SME,
|
QDF_TRACE(QDF_MODULE_ID_SME,
|
||||||
QDF_TRACE_LEVEL_DEBUG,
|
QDF_TRACE_LEVEL_DEBUG,
|
||||||
FL("LFR3:Don't start waitforkey timer"));
|
FL("LFR3:Don't start waitforkey timer"));
|
||||||
|
@@ -3828,7 +3828,7 @@ static bool csr_scan_filter_given_chnl_band(tpAniSirGlobal mac_ctx,
|
|||||||
* In case if no-concurrent IBSS session exist then scan
|
* In case if no-concurrent IBSS session exist then scan
|
||||||
* full band
|
* full band
|
||||||
*/
|
*/
|
||||||
if ((dst_req->ChannelInfo.numOfChannels == 0)) {
|
if (dst_req->ChannelInfo.numOfChannels == 0) {
|
||||||
csr_get_cfg_valid_channels(mac_ctx, valid_chnl_list,
|
csr_get_cfg_valid_channels(mac_ctx, valid_chnl_list,
|
||||||
&valid_chnl_len);
|
&valid_chnl_len);
|
||||||
} else {
|
} else {
|
||||||
|
Reference in New Issue
Block a user