qcacld-3.0: Optimize logs in QOS module

Optimize logs in QOS module.

Change-Id: Iba70a021d76f7574ad1bc43971b356a444dbbdde
CRs-Fixed: 3654164

core/hdd/src/wlan_hdd_wmm.c

@@ -2599,7 +2599,6 @@ QDF_STATUS hdd_wmm_connect(struct hdd_adapter *adapter,
 	adapter->hdd_wmm_status.qos_connection = qos_connection;
 
 	for (ac = 0; ac < WLAN_MAX_AC; ac++) {
-		hdd_debug("ac %d off", ac);
 		/* admission is not required so access is allowed */
 		adapter->hdd_wmm_status.ac_status[ac].is_access_required = false;
 		adapter->hdd_wmm_status.ac_status[ac].is_access_allowed = true;

core/sme/src/common/sme_api.c

@@ -9075,16 +9075,12 @@ QDF_STATUS sme_update_dsc_pto_up_mapping(mac_handle_t mac_handle,
 
 	pSession = pe_find_session_by_vdev_id(mac, sessionId);
 
-	if (!pSession) {
-		sme_err("Session lookup fails for vdev %d", sessionId);
+	if (!pSession)
 		return QDF_STATUS_E_FAILURE;
-	}
 
 	pqosmapset = &pSession->QosMapSet;
-	if (!pqosmapset->present) {
-		sme_debug("QOS Mapping IE not present");
+	if (!pqosmapset->present)
 		return QDF_STATUS_E_FAILURE;
-	}
 
 	for (i = 0; i < SME_QOS_WMM_UP_MAX; i++) {
 		for (j = pqosmapset->dscp_range[i][0];

core/sme/src/common/sme_power_save.c

@@ -372,11 +372,8 @@ QDF_STATUS sme_enable_sta_ps_check(struct mac_context *mac_ctx,
 	 * also if command is power save disable  there is not need to check
 	 * for connected state as firmware can handle this
 	 */
-	if (!cm_is_vdevid_connected(mac_ctx->pdev, session_id)) {
-		sme_debug("vdev:%d STA not infra/connected state",
-			  session_id);
+	if (!cm_is_vdevid_connected(mac_ctx->pdev, session_id))
 		return QDF_STATUS_E_FAILURE;
-	}
 
 	return QDF_STATUS_SUCCESS;
 }

core/sme/src/qos/sme_qos.c

@@ -894,7 +894,7 @@ QDF_STATUS sme_qos_csr_event_ind(struct mac_context *mac,
 {
 	QDF_STATUS status = QDF_STATUS_E_FAILURE;
 
-	sme_debug("On Session %d Event %d received from CSR", sessionId, ind);
+	sme_debug("vdev %d event %d", sessionId, ind);
 	switch (ind) {
 	case SME_QOS_CSR_ASSOC_COMPLETE:
 		/* expecting assoc info in pEvent_info */
@@ -983,7 +983,7 @@ uint8_t sme_qos_get_acm_mask(struct mac_context *mac, struct bss_description
 		if (sme_qos_is_acm(mac, pSirBssDesc, ac, pIes))
 			acm_mask = acm_mask | (1 << (QCA_WLAN_AC_VO - ac));
 	}
-	sme_debug("mask is 0x%x", acm_mask);
+
 	return acm_mask;
 }
 
@@ -2667,7 +2667,6 @@ static enum sme_qos_statustype sme_qos_setup(struct mac_context *mac,
 static QDF_STATUS sme_qos_process_set_key_success_ind(struct mac_context *mac,
 					   uint8_t sessionId, void *pEvent_info)
 {
-	sme_debug("Set Key complete");
 	(void)sme_qos_process_buffered_cmd(sessionId);
 
 	return QDF_STATUS_SUCCESS;
@@ -4020,7 +4019,6 @@ static QDF_STATUS sme_qos_process_assoc_complete_ev(struct mac_context *mac, uin
 	QDF_STATUS status = QDF_STATUS_E_FAILURE;
 	enum qca_wlan_ac_type ac = QCA_WLAN_AC_BE;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	if (((SME_QOS_INIT == pSession->ac_info[QCA_WLAN_AC_BE].curr_state)
 	     && (SME_QOS_INIT ==
@@ -4106,7 +4104,6 @@ static QDF_STATUS sme_qos_process_reassoc_req_ev(struct mac_context *mac, uint8_
 	struct sme_qos_flowinfoentry *flow_info = NULL;
 	tListElem *entry = NULL;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 
 	if (pSession->ftHandoffInProgress) {
@@ -4324,8 +4321,6 @@ static QDF_STATUS sme_qos_process_reassoc_success_ev(struct mac_context *mac_ctx
 	enum qca_wlan_ac_type ac;
 	QDF_STATUS status = QDF_STATUS_E_FAILURE;
 
-	sme_debug("invoked on session %d", sessionid);
-
 	if (sessionid >= WLAN_MAX_VDEVS) {
 		sme_err("invoked on session %d", sessionid);
 		return status;
@@ -4429,7 +4424,6 @@ static QDF_STATUS sme_qos_process_reassoc_failure_ev(struct mac_context *mac,
 	struct sme_qos_acinfo *pACInfo;
 	enum qca_wlan_ac_type ac;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	for (ac = QCA_WLAN_AC_BE; ac < QCA_WLAN_AC_ALL; ac++) {
 		pACInfo = &pSession->ac_info[ac];
@@ -4538,7 +4532,6 @@ static QDF_STATUS sme_qos_process_handoff_assoc_req_ev(struct mac_context *mac,
 	struct sme_qos_acinfo *pACInfo;
 	uint8_t ac;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	for (ac = QCA_WLAN_AC_BE; ac < QCA_WLAN_AC_ALL; ac++) {
 		pACInfo = &pSession->ac_info[ac];
@@ -4596,7 +4589,6 @@ static QDF_STATUS sme_qos_process_handoff_success_ev(struct mac_context *mac,
 	uint8_t ac;
 	QDF_STATUS status = QDF_STATUS_E_FAILURE;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	/* go back to original state before handoff */
 	for (ac = QCA_WLAN_AC_BE; ac < QCA_WLAN_AC_ALL; ac++) {
@@ -4651,7 +4643,6 @@ static QDF_STATUS sme_qos_process_disconnect_ev(struct mac_context *mac, uint8_t
 {
 	struct sme_qos_sessioninfo *pSession;
 
-	sme_debug("invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	/*
 	 * In case of 11r - RIC, we request QoS and Hand-off at the
@@ -4764,8 +4755,6 @@ static QDF_STATUS sme_qos_process_preauth_success_ind(struct mac_context *mac_ct
 	struct wlan_objmgr_vdev *vdev;
 	struct mlme_legacy_priv *mlme_priv;
 
-	sme_debug("invoked on SME session %d", sessionid);
-
 	if (!sme_session) {
 		sme_err("sme_session is NULL");
 		return QDF_STATUS_E_INVAL;
@@ -5535,10 +5524,6 @@ static void sme_qos_state_transition(uint8_t sessionId,
 	pACInfo = &pSession->ac_info[ac];
 	pACInfo->prev_state = pACInfo->curr_state;
 	pACInfo->curr_state = new_state;
-	if (pACInfo->curr_state != pACInfo->prev_state)
-		sme_debug("On session %d new %d old %d, for AC %d",
-			  sessionId, pACInfo->curr_state,
-			  pACInfo->prev_state, ac);
 }
 
 /**
@@ -5848,10 +5833,9 @@ static QDF_STATUS sme_qos_delete_existing_flows(struct mac_context *mac,
 	struct sme_qos_flowinfoentry *flow_info = NULL;
 
 	pEntry = csr_ll_peek_head(&sme_qos_cb.flow_list, true);
-	if (!pEntry) {
-		sme_debug("Flow List empty, nothing to delete");
+	if (!pEntry)
 		return QDF_STATUS_E_FAILURE;
-	}
+
 	while (pEntry) {
 		pNextEntry = csr_ll_next(&sme_qos_cb.flow_list, pEntry, true);
 		flow_info = GET_BASE_ADDR(pEntry, struct sme_qos_flowinfoentry,
@@ -5933,7 +5917,6 @@ static QDF_STATUS sme_qos_process_buffered_cmd(uint8_t session_id)
 	QDF_STATUS qdf_ret_status = QDF_STATUS_SUCCESS;
 	struct sme_qos_cmdinfo *qos_cmd = NULL;
 
-	sme_debug("Invoked on session %d", session_id);
 	qos_session = &sme_qos_cb.sessionInfo[session_id];
 	if (!csr_ll_is_list_empty(&qos_session->bufferedCommandList, false)) {
 		list_elt = csr_ll_remove_head(&qos_session->bufferedCommandList,
@@ -6007,9 +5990,8 @@ static QDF_STATUS sme_qos_process_buffered_cmd(uint8_t session_id)
 		}
 		/* buffered command has been processed, reclaim the memory */
 		qdf_mem_free(pcmd);
-	} else {
-		sme_debug("cmd buffer empty");
 	}
+
 	return qdf_ret_status;
 }
 
@@ -6026,14 +6008,11 @@ static QDF_STATUS sme_qos_delete_buffered_requests(struct mac_context *mac,
 	struct sme_qos_cmdinfoentry *pcmd = NULL;
 	tListElem *pEntry = NULL, *pNextEntry = NULL;
 
-	sme_debug("Invoked on session %d", sessionId);
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
 	pEntry = csr_ll_peek_head(&pSession->bufferedCommandList, true);
-	if (!pEntry) {
-		sme_debug("Buffered List empty, nothing to delete on session %d",
-			  sessionId);
+	if (!pEntry)
 		return QDF_STATUS_E_FAILURE;
-	}
+
 	while (pEntry) {
 		pNextEntry = csr_ll_next(&pSession->bufferedCommandList, pEntry,
 					true);
@@ -6675,8 +6654,9 @@ void sme_qos_update_hand_off(uint8_t sessionId, bool updateHandOff)
 	struct sme_qos_sessioninfo *pSession;
 
 	pSession = &sme_qos_cb.sessionInfo[sessionId];
-	sme_debug("handoffRequested %d updateHandOff %d",
-		  pSession->handoffRequested, updateHandOff);
+	if (pSession->handoffRequested != updateHandOff)
+		sme_debug("handoffRequested %d updateHandOff %d",
+			  pSession->handoffRequested, updateHandOff);
 
 	pSession->handoffRequested = updateHandOff;