qcacmn: Optimize cp stats prints

Optimize cp stats prints

Change-Id: Idbaa19cc22e2ebcefdbae08169071203cd6ccc49
CRs-Fixed: 2633501

target_if/cp_stats/src/target_if_mc_cp_stats.c

@@ -249,10 +249,9 @@ static QDF_STATUS target_if_cp_stats_extract_mib_stats(
 {
 	QDF_STATUS status;
 
-	if (!stats_param->num_mib_stats) {
-		cp_stats_debug("no mib stats");
+	if (!stats_param->num_mib_stats)
 		return QDF_STATUS_SUCCESS;
-	}
+
 	if (stats_param->num_mib_stats != MAX_MIB_STATS ||
 	    (stats_param->num_mib_extd_stats &&
 	    stats_param->num_mib_extd_stats != MAX_MIB_STATS)) {
@@ -383,8 +382,8 @@ static QDF_STATUS target_if_cp_stats_extract_vdev_chain_rssi_stats(
 		for (j = 0; j < MAX_NUM_CHAINS; j++) {
 			dat_snr = rssi_stats.rssi_avg_data[j];
 			bcn_snr = rssi_stats.rssi_avg_beacon[j];
-			cp_stats_debug("Chain %d SNR bcn: %d data: %d", j,
-				       bcn_snr, dat_snr);
+			cp_stats_nofl_debug("Chain %d SNR bcn: %d data: %d", j,
+					    bcn_snr, dat_snr);
 			/*
 			 * Get the absolute rssi value from the current rssi
 			 * value the snr value is hardcoded into 0 in the
@@ -412,22 +411,22 @@ static QDF_STATUS target_if_cp_stats_extract_event(struct wmi_unified *wmi_hdl,
 		cp_stats_err("stats param extract failed: %d", status);
 		return status;
 	}
-	cp_stats_debug("num: pdev: %d, pdev_extd: %d, vdev: %d, peer: %d,"
-		       "peer_extd: %d rssi: %d, mib %d, mib_extd %d, "
-		       "bcnflt: %d, channel: %d, bcn: %d, peer_extd2: %d,"
-		       "last_event: %x",
-		       stats_param.num_pdev_stats,
-		       stats_param.num_pdev_ext_stats,
-		       stats_param.num_vdev_stats,
-		       stats_param.num_peer_stats,
-		       stats_param.num_peer_extd_stats,
-		       stats_param.num_rssi_stats,
-		       stats_param.num_mib_stats,
-		       stats_param.num_mib_extd_stats,
-		       stats_param.num_bcnflt_stats,
-		       stats_param.num_chan_stats,
-		       stats_param.num_bcn_stats,
-		       stats_param.num_peer_adv_stats, stats_param.last_event);
+	cp_stats_nofl_debug("num: pdev: %d, pdev_extd: %d, vdev: %d, peer: %d,"
+			    "peer_extd: %d rssi: %d, mib %d, mib_extd %d, "
+			    "bcnflt: %d, channel: %d, bcn: %d, peer_extd2: %d,"
+			    "last_event: %x",
+			    stats_param.num_pdev_stats,
+			    stats_param.num_pdev_ext_stats,
+			    stats_param.num_vdev_stats,
+			    stats_param.num_peer_stats,
+			    stats_param.num_peer_extd_stats,
+			    stats_param.num_rssi_stats,
+			    stats_param.num_mib_stats,
+			    stats_param.num_mib_extd_stats,
+			    stats_param.num_bcnflt_stats,
+			    stats_param.num_chan_stats,
+			    stats_param.num_bcn_stats,
+			    stats_param.num_peer_adv_stats, stats_param.last_event);
 
 	ev->last_event = stats_param.last_event;
 	status = target_if_cp_stats_extract_pdev_stats(wmi_hdl, &stats_param,

umac/cp_stats/dispatcher/src/wlan_cp_stats_mc_tgt_api.c

@@ -39,13 +39,10 @@ static bool tgt_mc_cp_stats_is_last_event(struct stats_event *ev,
 	if (IS_MSB_SET(ev->last_event)) {
 		is_last_event = IS_LSB_SET(ev->last_event);
 	} else {
-		if (stats_type == TYPE_CONNECTION_TX_POWER) {
-			cp_stats_debug("FW does not support last event bit");
+		if (stats_type == TYPE_CONNECTION_TX_POWER)
 			is_last_event = true;
-		} else {
-			cp_stats_debug("FW does not support last event bit");
+		else
 			is_last_event = !!ev->peer_stats;
-		}
 	}
 
 	if (is_last_event)