Просмотр исходного кода

qcacld-3.0: Reduce the log spam in wma_main.c

Move the logs to appropriate log levels to reduce
the log spam in wma_main.c

Change-Id: Iec3959a1cb4b63601b2a586c0cc264b589dd5b8e
CRs-Fixed: 2014745
Srinivas Girigowda 8 лет назад
Родитель
Сommit
3ad735a4d6
1 измененных файлов с 79 добавлено и 79 удалено
  1. 79 79
      core/wma/src/wma_main.c

+ 79 - 79
core/wma/src/wma_main.c

@@ -578,7 +578,7 @@ int wma_cli_set2_command(int vdev_id, int param_id, int sval1,
 
 	if (QDF_STATUS_SUCCESS !=
 	    scheduler_post_msg(QDF_MODULE_ID_WMA, &msg)) {
-		WMA_LOGP("%s: Failed to post WMA_CLI_SET_CMD msg",
+		WMA_LOGE("%s: Failed to post WMA_CLI_SET_CMD msg",
 			  __func__);
 		qdf_mem_free(iwcmd);
 		return -EIO;
@@ -1499,7 +1499,7 @@ static void wma_process_cli_set_cmd(tp_wma_handle wma,
 			}
 			break;
 		default:
-			WMA_LOGE("Invalid wma_cli_set pdev command/Not yet implemented 0x%x",
+			WMA_LOGD("Invalid wma_cli_set pdev command/Not yet implemented 0x%x",
 				 privcmd->param_id);
 			break;
 		}
@@ -1616,7 +1616,7 @@ static int wma_process_fw_event_mc_thread_ctx(void *ctx, void *ev)
 
 	if (QDF_STATUS_SUCCESS !=
 		scheduler_post_msg(QDF_MODULE_ID_WMA, &cds_msg)) {
-		WMA_LOGP("%s: Failed to post WMA_PROCESS_FW_EVENT msg",
+		WMA_LOGE("%s: Failed to post WMA_PROCESS_FW_EVENT msg",
 			 __func__);
 		qdf_nbuf_free(ev);
 		qdf_mem_free(params_buf);
@@ -1774,7 +1774,7 @@ static void wma_state_info_dump(char **buf_ptr, uint16_t *size)
 		return;
 	}
 
-	WMA_LOGI("%s: size of buffer: %d", __func__, *size);
+	WMA_LOGE("%s: size of buffer: %d", __func__, *size);
 
 	for (vdev_id = 0; vdev_id < wma->max_bssid; vdev_id++) {
 		iface = &wma->interfaces[vdev_id];
@@ -2085,7 +2085,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	htc_handle = cds_get_context(QDF_MODULE_ID_HTC);
 
 	if (!htc_handle) {
-		WMA_LOGP("%s: Invalid HTC handle", __func__);
+		WMA_LOGE("%s: Invalid HTC handle", __func__);
 		return QDF_STATUS_E_INVAL;
 	}
 
@@ -2095,7 +2095,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 				       sizeof(t_wma_handle));
 
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: Memory allocation failed for wma_handle",
+		WMA_LOGE("%s: Memory allocation failed for wma_handle",
 			 __func__);
 		return qdf_status;
 	}
@@ -2127,7 +2127,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	wmi_handle = wmi_unified_attach(wma_handle, NULL,
 					WMI_TLV_TARGET, use_cookie, &ops, psoc);
 	if (!wmi_handle) {
-		WMA_LOGP("%s: failed to attach WMI", __func__);
+		WMA_LOGE("%s: failed to attach WMI", __func__);
 		qdf_status = QDF_STATUS_E_NOMEM;
 		goto err_wma_handle;
 	}
@@ -2222,7 +2222,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	wma_handle->interfaces = qdf_mem_malloc(sizeof(struct wma_txrx_node) *
 						wma_handle->max_bssid);
 	if (!wma_handle->interfaces) {
-		WMA_LOGP("%s: failed to allocate interface table", __func__);
+		WMA_LOGE("%s: failed to allocate interface table", __func__);
 		qdf_status = QDF_STATUS_E_NOMEM;
 		goto err_scn_context;
 	}
@@ -2247,7 +2247,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 
 	qdf_status = qdf_event_create(&wma_handle->wma_ready_event);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: wma_ready_event initialization failed", __func__);
+		WMA_LOGE("%s: wma_ready_event initialization failed", __func__);
 		goto err_event_init;
 	}
 
@@ -2262,7 +2262,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 
 	qdf_status = qdf_event_create(&wma_handle->target_suspend);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: target suspend event initialization failed",
+		WMA_LOGE("%s: target suspend event initialization failed",
 			 __func__);
 		goto err_event_init;
 	}
@@ -2270,7 +2270,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	/* Init Tx Frame Complete event */
 	qdf_status = qdf_event_create(&wma_handle->tx_frm_download_comp_event);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		WMA_LOGP("%s: failed to init tx_frm_download_comp_event",
+		WMA_LOGE("%s: failed to init tx_frm_download_comp_event",
 			 __func__);
 		goto err_event_init;
 	}
@@ -2278,13 +2278,13 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	/* Init tx queue empty check event */
 	qdf_status = qdf_event_create(&wma_handle->tx_queue_empty_event);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		WMA_LOGP("%s: failed to init tx_queue_empty_event", __func__);
+		WMA_LOGE("%s: failed to init tx_queue_empty_event", __func__);
 		goto err_event_init;
 	}
 
 	qdf_status = qdf_event_create(&wma_handle->wma_resume_event);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: wma_resume_event initialization failed",
+		WMA_LOGE("%s: wma_resume_event initialization failed",
 			 __func__);
 		goto err_event_init;
 	}
@@ -2292,21 +2292,21 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	qdf_status = cds_shutdown_notifier_register(wma_shutdown_notifier_cb,
 						    wma_handle);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: Shutdown notifier register failed: %d",
+		WMA_LOGE("%s: Shutdown notifier register failed: %d",
 			 __func__, qdf_status);
 		goto err_event_init;
 	}
 
 	qdf_status = qdf_event_create(&wma_handle->runtime_suspend);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: runtime_suspend event initialization failed",
+		WMA_LOGE("%s: runtime_suspend event initialization failed",
 			 __func__);
 		goto err_event_init;
 	}
 
 	qdf_status = qdf_event_create(&wma_handle->recovery_event);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: recovery event initialization failed", __func__);
+		WMA_LOGE("%s: recovery event initialization failed", __func__);
 		goto err_event_init;
 	}
 
@@ -2403,7 +2403,7 @@ QDF_STATUS wma_open(struct wlan_objmgr_psoc *psoc, void *cds_context,
 	/* Firmware debug log */
 	qdf_status = dbglog_init(wma_handle->wmi_handle);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: Firmware Dbglog initialization failed", __func__);
+		WMA_LOGE("%s: Firmware Dbglog initialization failed", __func__);
 		goto err_dbglog_init;
 	}
 
@@ -2580,7 +2580,7 @@ QDF_STATUS wma_pre_start(void *cds_ctx)
 
 	/* Validate the wma_handle */
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: invalid argument", __func__);
+		WMA_LOGE("%s: invalid argument", __func__);
 		qdf_status = QDF_STATUS_E_INVAL;
 		goto end;
 	}
@@ -2588,7 +2588,7 @@ QDF_STATUS wma_pre_start(void *cds_ctx)
 	status = wmi_unified_connect_htc_service(wma_handle->wmi_handle,
 						 wma_handle->htc_handle);
 	if (A_OK != status) {
-		WMA_LOGP("%s: wmi_unified_connect_htc_service", __func__);
+		WMA_LOGE("%s: wmi_unified_connect_htc_service", __func__);
 		qdf_status = QDF_STATUS_E_FAULT;
 		goto end;
 	}
@@ -2602,7 +2602,7 @@ QDF_STATUS wma_pre_start(void *cds_ctx)
 
 	qdf_status = scheduler_post_msg(QDF_MODULE_ID_WMA, &wma_msg);
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		WMA_LOGP("%s: Failed to post WNI_CFG_DNLD_REQ msg", __func__);
+		WMA_LOGE("%s: Failed to post WNI_CFG_DNLD_REQ msg", __func__);
 		QDF_ASSERT(0);
 		qdf_status = QDF_STATUS_E_FAILURE;
 	}
@@ -2718,7 +2718,7 @@ static int wma_pdev_set_hw_mode_resp_evt_handler(void *handle,
 
 	hw_mode_resp = qdf_mem_malloc(sizeof(*hw_mode_resp));
 	if (!hw_mode_resp) {
-		WMA_LOGI("%s: Memory allocation failed", __func__);
+		WMA_LOGE("%s: Memory allocation failed", __func__);
 		/* Since this memory allocation itself failed, we cannot
 		 * send fail response back to LIM here
 		 */
@@ -2739,7 +2739,7 @@ static int wma_pdev_set_hw_mode_resp_evt_handler(void *handle,
 	hw_mode_resp->cfgd_hw_mode_index = wmi_event->cfgd_hw_mode_index;
 	hw_mode_resp->num_vdev_mac_entries = wmi_event->num_vdev_mac_entries;
 
-	WMA_LOGI("%s: status:%d cfgd_hw_mode_index:%d num_vdev_mac_entries:%d",
+	WMA_LOGE("%s: status:%d cfgd_hw_mode_index:%d num_vdev_mac_entries:%d",
 			__func__, wmi_event->status,
 			wmi_event->cfgd_hw_mode_index,
 			wmi_event->num_vdev_mac_entries);
@@ -2759,7 +2759,7 @@ static int wma_pdev_set_hw_mode_resp_evt_handler(void *handle,
 		}
 		mac_id = WMA_PDEV_TO_MAC_MAP(vdev_mac_entry[i].pdev_id);
 
-		WMA_LOGI("%s: vdev_id:%d mac_id:%d",
+		WMA_LOGE("%s: vdev_id:%d mac_id:%d",
 			__func__, vdev_id, mac_id);
 
 		hw_mode_resp->vdev_mac_map[i].vdev_id = vdev_id;
@@ -2779,7 +2779,7 @@ static int wma_pdev_set_hw_mode_resp_evt_handler(void *handle,
 		wmi_event->cfgd_hw_mode_index);
 	}
 
-	WMA_LOGI("%s: Updated: old_hw_mode_index:%d new_hw_mode_index:%d",
+	WMA_LOGE("%s: Updated: old_hw_mode_index:%d new_hw_mode_index:%d",
 		__func__, wma->old_hw_mode_index, wma->new_hw_mode_index);
 
 	wma_send_msg(wma, SIR_HAL_PDEV_SET_HW_MODE_RESP,
@@ -2823,7 +2823,7 @@ void wma_process_pdev_hw_mode_trans_ind(void *handle,
 	hw_mode_trans_ind->new_hw_mode_index = fixed_param->new_hw_mode_index;
 	hw_mode_trans_ind->num_vdev_mac_entries =
 					fixed_param->num_vdev_mac_entries;
-	WMA_LOGI("%s: old_hw_mode_index:%d new_hw_mode_index:%d entries=%d",
+	WMA_LOGE("%s: old_hw_mode_index:%d new_hw_mode_index:%d entries=%d",
 		__func__, fixed_param->old_hw_mode_index,
 		fixed_param->new_hw_mode_index,
 		fixed_param->num_vdev_mac_entries);
@@ -2843,7 +2843,7 @@ void wma_process_pdev_hw_mode_trans_ind(void *handle,
 
 		mac_id = WMA_PDEV_TO_MAC_MAP(vdev_mac_entry[i].pdev_id);
 
-		WMA_LOGI("%s: vdev_id:%d mac_id:%d",
+		WMA_LOGE("%s: vdev_id:%d mac_id:%d",
 				__func__, vdev_id, mac_id);
 
 		hw_mode_trans_ind->vdev_mac_map[i].vdev_id = vdev_id;
@@ -2858,7 +2858,7 @@ void wma_process_pdev_hw_mode_trans_ind(void *handle,
 	policy_mgr_update_old_hw_mode_index(wma->psoc,
 		fixed_param->old_hw_mode_index);
 
-	WMA_LOGI("%s: Updated: old_hw_mode_index:%d new_hw_mode_index:%d",
+	WMA_LOGE("%s: Updated: old_hw_mode_index:%d new_hw_mode_index:%d",
 		__func__, wma->old_hw_mode_index, wma->new_hw_mode_index);
 }
 
@@ -2900,8 +2900,8 @@ static int wma_pdev_hw_mode_transition_evt_handler(void *handle,
 
 	hw_mode_trans_ind = qdf_mem_malloc(sizeof(*hw_mode_trans_ind));
 	if (!hw_mode_trans_ind) {
-		WMA_LOGI("%s: Memory allocation failed", __func__);
-		return QDF_STATUS_E_NULL_VALUE;
+		WMA_LOGE("%s: Memory allocation failed", __func__);
+		return QDF_STATUS_E_NOMEM;
 	}
 
 	wmi_event = param_buf->fixed_param;
@@ -2964,7 +2964,7 @@ static int wma_pdev_set_dual_mode_config_resp_evt_handler(void *handle,
 	}
 
 	wmi_event = param_buf->fixed_param;
-	WMA_LOGI("%s: status:%d", __func__, wmi_event->status);
+	WMA_LOGD("%s: status:%d", __func__, wmi_event->status);
 	dual_mac_cfg_resp->status = wmi_event->status;
 
 	if (SET_HW_MODE_STATUS_OK == dual_mac_cfg_resp->status) {
@@ -3007,7 +3007,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 
 	/* validate the wma_handle */
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: Invalid handle", __func__);
+		WMA_LOGE("%s: Invalid handle", __func__);
 		qdf_status = QDF_STATUS_E_INVAL;
 		goto end;
 	}
@@ -3017,7 +3017,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 						    wma_scan_event_callback,
 						    WMA_RX_SERIALIZER_CTX);
 	if (0 != status) {
-		WMA_LOGP("%s: Failed to register scan callback", __func__);
+		WMA_LOGE("%s: Failed to register scan callback", __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 		goto end;
 	}
@@ -3027,7 +3027,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 						    wma_roam_event_callback,
 						    WMA_RX_SERIALIZER_CTX);
 	if (0 != status) {
-		WMA_LOGP("%s: Failed to register Roam callback", __func__);
+		WMA_LOGE("%s: Failed to register Roam callback", __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 		goto end;
 	}
@@ -3037,7 +3037,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 						    wma_wow_wakeup_host_event,
 						    WMA_RX_TASKLET_CTX);
 	if (status) {
-		WMA_LOGP("%s: Failed to register wow wakeup host event handler",
+		WMA_LOGE("%s: Failed to register wow wakeup host event handler",
 			 __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 		goto end;
@@ -3048,7 +3048,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 				wma_pdev_resume_event_handler,
 				WMA_RX_TASKLET_CTX);
 	if (status) {
-		WMA_LOGP("%s: Failed to register PDEV resume event handler",
+		WMA_LOGE("%s: Failed to register PDEV resume event handler",
 			 __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 		goto end;
@@ -3145,7 +3145,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 
 	qdf_status = wma_tx_attach(wma_handle);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: Failed to register tx management", __func__);
+		WMA_LOGE("%s: Failed to register tx management", __func__);
 		goto end;
 	}
 
@@ -3175,7 +3175,7 @@ QDF_STATUS wma_start(void *cds_ctx)
 						wma_vdev_tsf_handler,
 						WMA_RX_SERIALIZER_CTX);
 	if (0 != status) {
-		WMA_LOGP("%s: Failed to register tsf callback", __func__);
+		WMA_LOGE("%s: Failed to register tsf callback", __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 		goto end;
 	}
@@ -3250,7 +3250,7 @@ QDF_STATUS wma_stop(void *cds_ctx, uint8_t reason)
 
 	/* validate the wma_handle */
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: Invalid handle", __func__);
+		WMA_LOGE("%s: Invalid handle", __func__);
 		qdf_status = QDF_STATUS_E_INVAL;
 		goto end;
 	}
@@ -3289,7 +3289,7 @@ QDF_STATUS wma_stop(void *cds_ctx, uint8_t reason)
 	}
 	qdf_status = wma_tx_detach(wma_handle);
 	if (qdf_status != QDF_STATUS_SUCCESS) {
-		WMA_LOGP("%s: Failed to deregister tx management", __func__);
+		WMA_LOGE("%s: Failed to deregister tx management", __func__);
 		goto end;
 	}
 
@@ -3313,7 +3313,7 @@ static void wma_cleanup_hold_req(tp_wma_handle wma)
 	qdf_spin_lock_bh(&wma->wma_hold_req_q_lock);
 	if (!qdf_list_size(&wma->wma_hold_req_queue)) {
 		qdf_spin_unlock_bh(&wma->wma_hold_req_q_lock);
-		WMA_LOGI(FL("request queue is empty"));
+		WMA_LOGD(FL("request queue is empty"));
 		return;
 	}
 
@@ -3472,25 +3472,25 @@ static void wma_cleanup_dbs_phy_caps(t_wma_handle *wma_handle)
 	if (wma_handle->phy_caps.hw_mode_to_mac_cap_map) {
 		qdf_mem_free(wma_handle->phy_caps.hw_mode_to_mac_cap_map);
 		wma_handle->phy_caps.hw_mode_to_mac_cap_map = NULL;
-		WMA_LOGI("%s: hw_mode_to_mac_cap_map freed", __func__);
+		WMA_LOGD("%s: hw_mode_to_mac_cap_map freed", __func__);
 	}
 
 	if (wma_handle->phy_caps.each_hw_mode_cap) {
 		qdf_mem_free(wma_handle->phy_caps.each_hw_mode_cap);
 		wma_handle->phy_caps.each_hw_mode_cap = NULL;
-		WMA_LOGI("%s: each_hw_mode_cap freed", __func__);
+		WMA_LOGD("%s: each_hw_mode_cap freed", __func__);
 	}
 
 	if (wma_handle->phy_caps.each_phy_cap_per_hwmode) {
 		qdf_mem_free(wma_handle->phy_caps.each_phy_cap_per_hwmode);
 		wma_handle->phy_caps.each_phy_cap_per_hwmode = NULL;
-		WMA_LOGI("%s: each_phy_cap_per_hwmode freed", __func__);
+		WMA_LOGD("%s: each_phy_cap_per_hwmode freed", __func__);
 	}
 
 	if (wma_handle->phy_caps.each_phy_hal_reg_cap) {
 		qdf_mem_free(wma_handle->phy_caps.each_phy_hal_reg_cap);
 		wma_handle->phy_caps.each_phy_hal_reg_cap = NULL;
-		WMA_LOGI("%s: each_phy_hal_reg_cap freed", __func__);
+		WMA_LOGD("%s: each_phy_hal_reg_cap freed", __func__);
 	}
 }
 
@@ -3519,7 +3519,7 @@ QDF_STATUS wma_close(void *cds_ctx)
 
 	/* validate the wmi handle */
 	if (NULL == wma_handle->wmi_handle) {
-		WMA_LOGP("%s: Invalid wmi handle", __func__);
+		WMA_LOGE("%s: Invalid wmi handle", __func__);
 		return QDF_STATUS_E_INVAL;
 	}
 
@@ -3527,7 +3527,7 @@ QDF_STATUS wma_close(void *cds_ctx)
 	if (wma_handle->hw_mode.hw_mode_list) {
 		qdf_mem_free(wma_handle->hw_mode.hw_mode_list);
 		wma_handle->hw_mode.hw_mode_list = NULL;
-		WMA_LOGI("%s: DBS list is freed", __func__);
+		WMA_LOGD("%s: DBS list is freed", __func__);
 	}
 	wma_cleanup_dbs_phy_caps(wma_handle);
 	if (cds_get_conparam() != QDF_GLOBAL_FTM_MODE) {
@@ -3540,13 +3540,13 @@ QDF_STATUS wma_close(void *cds_ctx)
 	/* unregister Firmware debug log */
 	qdf_status = dbglog_deinit(wma_handle->wmi_handle);
 	if (qdf_status != QDF_STATUS_SUCCESS)
-		WMA_LOGP("%s: dbglog_deinit failed", __func__);
+		WMA_LOGE("%s: dbglog_deinit failed", __func__);
 
 	/* close the qdf events */
 	qdf_event_destroy(&wma_handle->wma_ready_event);
 	qdf_status = qdf_mc_timer_destroy(&wma_handle->service_ready_ext_timer);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status))
-		WMA_LOGP("%s: Failed to destroy service ready ext event timer",
+		WMA_LOGE("%s: Failed to destroy service ready ext event timer",
 			__func__);
 
 	qdf_event_destroy(&wma_handle->target_suspend);
@@ -3822,14 +3822,14 @@ static inline void wma_update_target_services(tp_wma_handle wh,
 			cds_get_context(QDF_MODULE_ID_SOC),
 			(struct cdp_cfg *)cds_get_context(QDF_MODULE_ID_CFG),
 			HTT_TX_IPA_NEW_MSDU_ID_SPACE_BEGIN);
-		WMA_LOGI("%s: TX_MSDU_ID_NEW_PARTITION=%d", __func__,
+		WMA_LOGD("%s: TX_MSDU_ID_NEW_PARTITION=%d", __func__,
 				HTT_TX_IPA_NEW_MSDU_ID_SPACE_BEGIN);
 	} else {
 		cdp_ipa_set_uc_tx_partition_base(
 			cds_get_context(QDF_MODULE_ID_SOC),
 			(struct cdp_cfg *)cds_get_context(QDF_MODULE_ID_CFG),
 			HTT_TX_IPA_MSDU_ID_SPACE_BEGIN);
-		WMA_LOGI("%s: TX_MSDU_ID_OLD_PARTITION=%d", __func__,
+		WMA_LOGD("%s: TX_MSDU_ID_OLD_PARTITION=%d", __func__,
 				HTT_TX_IPA_MSDU_ID_SPACE_BEGIN);
 	}
 
@@ -4019,7 +4019,7 @@ static void wma_update_target_ext_ht_cap(tp_wma_handle wma_handle,
 	 */
 	if (!wma_handle ||
 		(0 == wma_handle->phy_caps.num_hw_modes.num_hw_modes)) {
-		WMA_LOGI("%s: No extended HT cap for current SOC", __func__);
+		WMA_LOGD("%s: No extended HT cap for current SOC", __func__);
 		return;
 	}
 
@@ -4050,7 +4050,7 @@ static void wma_update_target_ext_ht_cap(tp_wma_handle wma_handle,
 					sizeof(struct wma_tgt_ht_cap));
 	}
 
-	WMA_LOGI("%s: [ext ht cap] ht_rx_stbc - %d, ht_tx_stbc - %d\n"
+	WMA_LOGD("%s: [ext ht cap] ht_rx_stbc - %d, ht_tx_stbc - %d\n"
 			"mpdu_density - %d ht_rx_ldpc - %d ht_sgi_20 - %d\n"
 			"ht_sgi_40 - %d num_rf_chains - %d", __func__,
 			ht_cap->ht_rx_stbc, ht_cap->ht_tx_stbc,
@@ -4192,7 +4192,7 @@ static void wma_update_target_ext_vht_cap(t_wma_handle *wma_handle,
 	 */
 	if (!wma_handle ||
 		(0 == wma_handle->phy_caps.num_hw_modes.num_hw_modes)) {
-		WMA_LOGI("%s: No extended VHT cap for current SOC", __func__);
+		WMA_LOGD("%s: No extended VHT cap for current SOC", __func__);
 		return;
 	}
 
@@ -4221,7 +4221,7 @@ static void wma_update_target_ext_vht_cap(t_wma_handle *wma_handle,
 					sizeof(struct wma_tgt_vht_cap));
 	}
 
-	WMA_LOGI("%s: [ext vhtcap] max_mpdu %d supp_chan_width %x rx_ldpc %x\n"
+	WMA_LOGD("%s: [ext vhtcap] max_mpdu %d supp_chan_width %x rx_ldpc %x\n"
 		"short_gi_80 %x tx_stbc %x rx_stbc %x txop_ps %x\n"
 		"su_bformee %x mu_bformee %x max_ampdu_len_exp %d", __func__,
 		vht_cap->vht_max_mpdu, vht_cap->supp_chan_width,
@@ -4497,13 +4497,13 @@ int wma_rx_service_ready_event(void *handle, uint8_t *cmd_param_info,
 
 	param_buf = (WMI_SERVICE_READY_EVENTID_param_tlvs *) cmd_param_info;
 	if (!(handle && param_buf)) {
-		WMA_LOGP("%s: Invalid arguments", __func__);
+		WMA_LOGE("%s: Invalid arguments", __func__);
 		return -EINVAL;
 	}
 
 	ev = param_buf->fixed_param;
 	if (!ev) {
-		WMA_LOGP("%s: Invalid buffer", __func__);
+		WMA_LOGE("%s: Invalid buffer", __func__);
 		return -EINVAL;
 	}
 
@@ -4782,7 +4782,7 @@ int wma_rx_service_ready_event(void *handle, uint8_t *cmd_param_info,
 		ret = qdf_mc_timer_start(&wma_handle->service_ready_ext_timer,
 				WMA_SERVICE_READY_EXT_TIMEOUT);
 		if (!QDF_IS_STATUS_SUCCESS(ret))
-			WMA_LOGP("Failed to start the service ready ext timer");
+			WMA_LOGE("Failed to start the service ready ext timer");
 
 		WMA_LOGD("%s: WMA waiting for WMI_SERVICE_READY_EXT_EVENTID",
 				__func__);
@@ -4831,16 +4831,16 @@ static QDF_STATUS wma_get_phyid_for_given_band(
 		if ((band == CDS_BAND_2GHZ) &&
 				(WLAN_2G_CAPABILITY == cap->supported_bands)) {
 			*phyid = idx + i;
-			WMA_LOGI("Select 2G capable phyid[%d]", *phyid);
+			WMA_LOGD("Select 2G capable phyid[%d]", *phyid);
 			return QDF_STATUS_SUCCESS;
 		} else if ((band == CDS_BAND_5GHZ) &&
 				(WLAN_5G_CAPABILITY == cap->supported_bands)) {
 			*phyid = idx + i;
-			WMA_LOGI("Select 5G capable phyid[%d]", *phyid);
+			WMA_LOGD("Select 5G capable phyid[%d]", *phyid);
 			return QDF_STATUS_SUCCESS;
 		}
 	}
-	WMA_LOGI("Using default single hw mode phyid[%d]", *phyid);
+	WMA_LOGD("Using default single hw mode phyid[%d]", *phyid);
 	return QDF_STATUS_SUCCESS;
 }
 
@@ -5014,12 +5014,12 @@ static void wma_print_populate_soc_caps(t_wma_handle *wma_handle)
 	WMI_MAC_PHY_CAPABILITIES *tmp;
 
 	/* print number of hw modes */
-	WMA_LOGI("%s: num of hw modes [%d]", __func__,
+	WMA_LOGD("%s: num of hw modes [%d]", __func__,
 		wma_handle->phy_caps.num_hw_modes.num_hw_modes);
-	WMA_LOGI("%s: <====== HW mode cap printing starts ======>", __func__);
+	WMA_LOGD("%s: <====== HW mode cap printing starts ======>", __func__);
 	/* print cap of each hw mode */
 	for (i = 0; i < wma_handle->phy_caps.num_hw_modes.num_hw_modes; i++) {
-		WMA_LOGI("====>: hw mode id[%d], phy_id map[%d]",
+		WMA_LOGD("====>: hw mode id[%d], phy_id map[%d]",
 			wma_handle->phy_caps.each_hw_mode_cap[i].hw_mode_id,
 			wma_handle->phy_caps.each_hw_mode_cap[i].phy_id_map);
 		if (wma_handle->phy_caps.each_hw_mode_cap[i].phy_id_map ==
@@ -5192,7 +5192,7 @@ static QDF_STATUS wma_update_hw_mode_list(t_wma_handle *wma_handle)
 	if (wma_handle->hw_mode.hw_mode_list) {
 		qdf_mem_free(wma_handle->hw_mode.hw_mode_list);
 		wma_handle->hw_mode.hw_mode_list = NULL;
-		WMA_LOGI("%s: DBS list is freed", __func__);
+		WMA_LOGD("%s: DBS list is freed", __func__);
 	}
 
 	wma_handle->num_dbs_hw_modes = phy_caps->num_hw_modes.num_hw_modes;
@@ -5268,12 +5268,12 @@ static void wma_populate_soc_caps(t_wma_handle *wma_handle,
 	WMA_LOGD("%s: Enter", __func__);
 
 	if (!wma_handle) {
-		WMA_LOGP("%s: Invalid WMA handle", __func__);
+		WMA_LOGE("%s: Invalid WMA handle", __func__);
 		return;
 	}
 
 	if (!param_buf) {
-		WMA_LOGP("%s: Invalid event", __func__);
+		WMA_LOGE("%s: Invalid event", __func__);
 		return;
 	}
 	phy_caps = &wma_handle->phy_caps;
@@ -5294,7 +5294,7 @@ static void wma_populate_soc_caps(t_wma_handle *wma_handle,
 		WMA_LOGE("%s: Number of hw modes is zero", __func__);
 		return;
 	}
-	WMA_LOGI("%s: Given number of hw modes[%d]",
+	WMA_LOGD("%s: Given number of hw modes[%d]",
 		 __func__, phy_caps->num_hw_modes.num_hw_modes);
 
 	/*
@@ -5405,19 +5405,19 @@ int wma_rx_service_ready_ext_event(void *handle, uint8_t *event,
 	WMA_LOGD("%s: Enter", __func__);
 
 	if (!wma_handle) {
-		WMA_LOGP("%s: Invalid WMA handle", __func__);
+		WMA_LOGE("%s: Invalid WMA handle", __func__);
 		return -EINVAL;
 	}
 
 	param_buf = (WMI_SERVICE_READY_EXT_EVENTID_param_tlvs *) event;
 	if (!param_buf) {
-		WMA_LOGP("%s: Invalid event", __func__);
+		WMA_LOGE("%s: Invalid event", __func__);
 		return -EINVAL;
 	}
 
 	ev = param_buf->fixed_param;
 	if (!ev) {
-		WMA_LOGP("%s: Invalid buffer", __func__);
+		WMA_LOGE("%s: Invalid buffer", __func__);
 		return -EINVAL;
 	}
 
@@ -5429,7 +5429,7 @@ int wma_rx_service_ready_ext_event(void *handle, uint8_t *event,
 
 	ret = qdf_mc_timer_stop(&wma_handle->service_ready_ext_timer);
 	if (!QDF_IS_STATUS_SUCCESS(ret)) {
-		WMA_LOGP("Failed to stop the service ready ext timer");
+		WMA_LOGE("Failed to stop the service ready ext timer");
 		return -EINVAL;
 	}
 	wma_populate_soc_caps(wma_handle, param_buf);
@@ -5475,7 +5475,7 @@ int wma_rx_ready_event(void *handle, uint8_t *cmd_param_info,
 
 	param_buf = (WMI_READY_EVENTID_param_tlvs *) cmd_param_info;
 	if (!(wma_handle && param_buf)) {
-		WMA_LOGP("%s: Invalid arguments", __func__);
+		WMA_LOGE("%s: Invalid arguments", __func__);
 		QDF_ASSERT(0);
 		return -EINVAL;
 	}
@@ -5557,7 +5557,7 @@ void wma_setneedshutdown(void *cds_ctx)
 	wma_handle = cds_get_context(QDF_MODULE_ID_WMA);
 
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: Invalid arguments", __func__);
+		WMA_LOGE("%s: Invalid arguments", __func__);
 		QDF_ASSERT(0);
 		return;
 	}
@@ -5581,7 +5581,7 @@ bool wma_needshutdown(void *cds_ctx)
 	wma_handle = cds_get_context(QDF_MODULE_ID_WMA);
 
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: Invalid arguments", __func__);
+		WMA_LOGE("%s: Invalid arguments", __func__);
 		QDF_ASSERT(0);
 		return false;
 	}
@@ -5606,7 +5606,7 @@ QDF_STATUS wma_wait_for_ready_event(WMA_HANDLE handle)
 					   WMA_READY_EVENTID_TIMEOUT);
 
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		WMA_LOGP("%s: Timeout waiting for ready event from FW",
+		WMA_LOGE("%s: Timeout waiting for ready event from FW",
 			 __func__);
 		qdf_status = QDF_STATUS_E_FAILURE;
 	}
@@ -5888,7 +5888,7 @@ static void wma_set_wifi_start_packet_stats(void *wma_handle,
 
 	/* No need to register for ring IDs other than packet stats */
 	if (start_log->ring_id != RING_ID_PER_PACKET_STATS) {
-		WMA_LOGI("%s: Ring id is not for per packet stats: %d",
+		WMA_LOGD("%s: Ring id is not for per packet stats: %d",
 			__func__, start_log->ring_id);
 		return;
 	}
@@ -6126,7 +6126,7 @@ void wma_update_sta_inactivity_timeout(tp_wma_handle wma,
 		max_unresponsive_time))
 		WMA_LOGE("Failed to Set MAX UNRESPONSIVE TIME");
 
-	WMA_LOGI("%s:vdev_id:%d min_inactive_time: %u max_inactive_time: %u max_unresponsive_time: %u",
+	WMA_LOGD("%s:vdev_id:%d min_inactive_time: %u max_inactive_time: %u max_unresponsive_time: %u",
 			__func__, vdev_id,
 			min_inactive_time, max_inactive_time,
 			max_unresponsive_time);
@@ -6287,7 +6287,7 @@ QDF_STATUS wma_mc_process_msg(void *cds_context, struct scheduler_msg *msg)
 	wma_handle = cds_get_context(QDF_MODULE_ID_WMA);
 
 	if (NULL == wma_handle) {
-		WMA_LOGP("%s: wma_handle is NULL", __func__);
+		WMA_LOGE("%s: wma_handle is NULL", __func__);
 		QDF_ASSERT(0);
 		qdf_mem_free(msg->bodyptr);
 		qdf_status = QDF_STATUS_E_INVAL;
@@ -7328,7 +7328,7 @@ QDF_STATUS wma_send_pdev_set_antenna_mode(tp_wma_handle wma_handle,
 	cmd->num_txrx_chains = msg->num_rx_chains;
 	cmd->num_txrx_chains |= (msg->num_tx_chains << 16);
 
-	WMA_LOGI("%s: Num of chains TX: %d RX: %d txrx_chains: 0x%x",
+	WMA_LOGD("%s: Num of chains TX: %d RX: %d txrx_chains: 0x%x",
 		 __func__, msg->num_tx_chains,
 		 msg->num_rx_chains, cmd->num_txrx_chains);