qcacld-3.0: Remove dead code tUpdateCFParams and WMA_UPDATE_CF_IND

WMA_UPDATE_CF_IND message (aka SIR_HAL_UPDATE_CF_IND) is dead code, because
it is not processed anywhere.

Remove dead code WMA_UPDATE_CF_IND. The only use of tUpdateCFParams is in
lim_send_cf_params() which only creates a WMA_UPDATE_CF_IND message, so
remove tUpdateCFParams and lim_send_cf_params().

Change-Id: I1b613300cdd7eeeb33825e319977ff941cae548c
CRs-Fixed: 2367931
This commit is contained in:
bings 2018-12-17 13:48:05 +08:00 • committed by nshrivas
commit 705b412f40
7 changed files with 0 additions and 76 deletions

View file

@ -268,7 +268,6 @@ struct sir_cfg_action_frm_tb_ppdu {
#define SIR_HAL_UPDATE_EDCA_PROFILE_IND (SIR_HAL_ITC_MSG_TYPES_BEGIN + 22)
#define SIR_HAL_UPDATE_BEACON_IND (SIR_HAL_ITC_MSG_TYPES_BEGIN + 23)
#define SIR_HAL_UPDATE_CF_IND (SIR_HAL_ITC_MSG_TYPES_BEGIN + 24)
#define SIR_HAL_CHNL_SWITCH_REQ (SIR_HAL_ITC_MSG_TYPES_BEGIN + 25)
#define SIR_HAL_ADD_TS_REQ (SIR_HAL_ITC_MSG_TYPES_BEGIN + 26)
#define SIR_HAL_DEL_TS_REQ (SIR_HAL_ITC_MSG_TYPES_BEGIN + 27)

View file

@ -36,56 +36,6 @@
#endif /* FEATURE_WLAN_DIAG_SUPPORT */
#include "lim_utils.h"
/**
* lim_send_cf_params()
*
***FUNCTION:
* This function is called to send CFP Parameters to WMA, when they are changed.
*
***LOGIC:
*
***ASSUMPTIONS:
* NA
*
***NOTE:
* NA
*
* @param mac pointer to Global Mac structure.
* @param bssIdx Bss Index of the BSS to which STA is associated.
* @param cfpCount CFP Count, if that is changed.
* @param cfpPeriod CFP Period if that is changed.
*
* @return success if message send is ok, else false.
*/
QDF_STATUS lim_send_cf_params(struct mac_context *mac, uint8_t bssIdx,
uint8_t cfpCount, uint8_t cfpPeriod)
{
tpUpdateCFParams pCFParams = NULL;
QDF_STATUS retCode = QDF_STATUS_SUCCESS;
struct scheduler_msg msgQ = {0};
pCFParams = qdf_mem_malloc(sizeof(tUpdateCFParams));
if (!pCFParams)
return QDF_STATUS_E_NOMEM;
pCFParams->cfpCount = cfpCount;
pCFParams->cfpPeriod = cfpPeriod;
pCFParams->bssIdx = bssIdx;
msgQ.type = WMA_UPDATE_CF_IND;
msgQ.reserved = 0;
msgQ.bodyptr = pCFParams;
msgQ.bodyval = 0;
pe_debug("Sending WMA_UPDATE_CF_IND");
MTRACE(mac_trace_msg_tx(mac, NO_SESSION, msgQ.type));
retCode = wma_post_ctrl_msg(mac, &msgQ);
if (QDF_STATUS_SUCCESS != retCode) {
qdf_mem_free(pCFParams);
pe_err("Posting WMA_UPDATE_CF_IND failed, reason=%X",
retCode);
}
return retCode;
}
/**
* lim_send_beacon_params() - updates bcn params to WMA
*

View file

@ -34,8 +34,6 @@
#include "lim_types.h"
#include "wma_if.h"
#include "sir_params.h"
QDF_STATUS lim_send_cf_params(struct mac_context *mac, uint8_t bssIdx,
uint8_t cfpCount, uint8_t cfpPeriod);
QDF_STATUS lim_send_beacon_params(struct mac_context *mac,
tpUpdateBeaconParams pUpdatedBcnParams,
struct pe_session *pe_session);

View file

@ -415,10 +415,6 @@ sch_bcn_process_sta(struct mac_context *mac_ctx,
return false;
}
mac_ctx->mlme_cfg->rates.cfp_period = bcn->cfParamSet.cfpPeriod;
lim_send_cf_params(mac_ctx, *bssIdx,
bcn->cfParamSet.cfpCount,
bcn->cfParamSet.cfpPeriod);
}
/* No need to send DTIM Period and Count to HAL/SMAC */

View file

@ -418,7 +418,6 @@ uint8_t *mac_trace_get_wma_msg_string(uint16_t wma_msg)
CASE_RETURN_STRING(WMA_UPDATE_EDCA_PROFILE_IND);
CASE_RETURN_STRING(WMA_UPDATE_BEACON_IND);
CASE_RETURN_STRING(WMA_UPDATE_CF_IND);
CASE_RETURN_STRING(WMA_CHNL_SWITCH_REQ);
CASE_RETURN_STRING(WMA_ADD_TS_REQ);
CASE_RETURN_STRING(WMA_DEL_TS_REQ);

View file

@ -861,23 +861,6 @@ typedef struct {
tSirMacAddr peer_mac;
} tUpdateUserPos, *tpUpdateUserPos;
/**
* struct tUpdateCFParams -CF parameters
* @bssIdx: BSSID index
* @cfpCount: CFP count
* @cfpPeriod: the number of DTIM intervals between the start of CFPs
*/
typedef struct {
uint8_t bssIdx;
/*
* cfpCount indicates how many DTIMs (including the current frame)
* appear before the next CFP start. A CFPCount of 0 indicates that
* the current DTIM marks the start of the CFP.
*/
uint8_t cfpCount;
uint8_t cfpPeriod;
} tUpdateCFParams, *tpUpdateCFParams;
/**
* struct tSwitchChannelParams - switch channel request parameter
* @channelNumber: channel number

View file

@ -152,7 +152,6 @@
#define WMA_UPDATE_EDCA_PROFILE_IND SIR_HAL_UPDATE_EDCA_PROFILE_IND
#define WMA_UPDATE_BEACON_IND SIR_HAL_UPDATE_BEACON_IND
#define WMA_UPDATE_CF_IND SIR_HAL_UPDATE_CF_IND
#define WMA_CHNL_SWITCH_REQ SIR_HAL_CHNL_SWITCH_REQ
#define WMA_ADD_TS_REQ SIR_HAL_ADD_TS_REQ
#define WMA_DEL_TS_REQ SIR_HAL_DEL_TS_REQ