From 0cb7296478b306313db46cb755a0d76e5448a34f Mon Sep 17 00:00:00 2001 From: Abhinav Kumar Date: Mon, 24 Feb 2020 15:19:30 +0530 Subject: [PATCH] qcacmn: Optimize logs for roaming and tdls Optimize logs for roaming and tdls Change-Id: I55d3f1242c240064fe34dbfe6552ee5bae6522f6 CRs-Fixed: 2627699 --- wmi/src/wmi_unified_roam_tlv.c | 15 ++++++--------- wmi/src/wmi_unified_sta_tlv.c | 5 ++--- 2 files changed, 8 insertions(+), 12 deletions(-) diff --git a/wmi/src/wmi_unified_roam_tlv.c b/wmi/src/wmi_unified_roam_tlv.c index 7c1291b612ea..1470a2f1471d 100644 --- a/wmi/src/wmi_unified_roam_tlv.c +++ b/wmi/src/wmi_unified_roam_tlv.c @@ -1079,7 +1079,7 @@ send_roam_scan_offload_mode_cmd_tlv(wmi_unified_t wmi_handle, #endif /* WLAN_FEATURE_ROAM_OFFLOAD */ sizeof(wmi_start_scan_cmd_fixed_param); #ifdef WLAN_FEATURE_ROAM_OFFLOAD - WMI_LOGD("auth_mode = %d", auth_mode); + wmi_debug("auth_mode = %d", auth_mode); if (roam_req->is_roam_req_valid && roam_req->roam_offload_enabled) { len += sizeof(wmi_roam_offload_tlv_param); @@ -1123,8 +1123,6 @@ send_roam_scan_offload_mode_cmd_tlv(wmi_unified_t wmi_handle, if (roam_req->is_roam_req_valid) WMI_LOGD("%s : roam offload = %d", __func__, roam_req->roam_offload_enabled); - else - WMI_LOGD("%s : roam_req is NULL", __func__); len += (4 * WMI_TLV_HDR_SIZE); } @@ -1539,7 +1537,7 @@ send_roam_scan_offload_ap_profile_cmd_tlv(wmi_unified_t wmi_handle, ap_profile->profile.rsn_mcastmgmtcipherset; profile->rssi_abs_thresh = ap_profile->profile.rssi_abs_thresh; - WMI_LOGD("AP profile: flags %x rssi_threshold %d ssid:%.*s authmode %d uc cipher %d mc cipher %d mc mgmt cipher %d rssi abs thresh %d", + WMI_LOGD("AP PROFILE: flags %x rssi_threshold %d ssid:%.*s authmode %d uc cipher %d mc cipher %d mc mgmt cipher %d rssi abs thresh %d", profile->flags, profile->rssi_threshold, profile->ssid.ssid_len, ap_profile->profile.ssid.mac_ssid, profile->rsn_authmode, profile->rsn_ucastcipherset, @@ -1734,8 +1732,6 @@ send_roam_scan_offload_ap_profile_cmd_tlv(wmi_unified_t wmi_handle, wmi_buf_free(buf); } - WMI_LOGD("WMI --> WMI_ROAM_AP_PROFILE and other parameters"); - return status; } @@ -1987,9 +1983,10 @@ static QDF_STATUS send_roam_scan_offload_rssi_change_cmd_tlv(wmi_unified_t wmi_h goto error; } - WMI_LOGD(FL("roam_scan_rssi_change_thresh=%d, bcn_rssi_weight=%d"), - rssi_change_thresh, bcn_rssi_weight); - WMI_LOGD(FL("hirssi_delay_btw_scans=%d"), hirssi_delay_btw_scans); + wmi_nofl_debug("roam_scan_rssi_change_thresh %d bcn_rssi_weight %d hirssi_delay_btw_scans %d", + rssi_change_thresh, bcn_rssi_weight, + hirssi_delay_btw_scans); + return QDF_STATUS_SUCCESS; error: wmi_buf_free(buf); diff --git a/wmi/src/wmi_unified_sta_tlv.c b/wmi/src/wmi_unified_sta_tlv.c index a5b570d4d44f..251ff0d3010d 100644 --- a/wmi/src/wmi_unified_sta_tlv.c +++ b/wmi/src/wmi_unified_sta_tlv.c @@ -676,7 +676,7 @@ send_update_fw_tdls_state_cmd_tlv(wmi_unified_t wmi_handle, cmd->tdls_discovery_wake_timeout = tdls_param->tdls_discovery_wake_timeout; - WMI_LOGD("%s: tdls_state: %d, state: %d, " + WMI_LOGD("%s: vdev %d tdls_state: %d, state: %d, " "notification_interval_ms: %d, " "tx_discovery_threshold: %d, " "tx_teardown_threshold: %d, " @@ -691,7 +691,7 @@ send_update_fw_tdls_state_cmd_tlv(wmi_unified_t wmi_handle, "teardown_notification_ms: %d, " "tdls_peer_kickout_threshold: %d, " "tdls_discovery_wake_timeout: %d", - __func__, tdls_state, cmd->state, + __func__,tdls_param->vdev_id, tdls_state, cmd->state, cmd->notification_interval_ms, cmd->tx_discovery_threshold, cmd->tx_teardown_threshold, @@ -714,7 +714,6 @@ send_update_fw_tdls_state_cmd_tlv(wmi_unified_t wmi_handle, wmi_buf_free(wmi_buf); return QDF_STATUS_E_FAILURE; } - WMI_LOGD("%s: vdev_id %d", __func__, tdls_param->vdev_id); return QDF_STATUS_SUCCESS; }