qcacld-3.0: HTT: Reduce the log spam in kmsg
Move the logs to appropriate log levels to reduce
the log spam in kmsg.
Change-Id: I6572cb109ff752eba764f671770d9754024a4b31
CRs-Fixed: 2014745
diff --git a/core/dp/htt/htt.c b/core/dp/htt/htt.c
index 9c27922..b44c6dd 100644
--- a/core/dp/htt/htt.c
+++ b/core/dp/htt/htt.c
@@ -354,12 +354,14 @@
pdev->cfg.is_full_reorder_offload =
ol_cfg_is_full_reorder_offload(pdev->ctrl_pdev);
- qdf_print("is_full_reorder_offloaded? %d\n",
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "is_full_reorder_offloaded? %d\n",
(int)pdev->cfg.is_full_reorder_offload);
pdev->cfg.ce_classify_enabled =
ol_cfg_is_ce_classify_enabled(ctrl_pdev);
- qdf_print("ce_classify_enabled %d\n",
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "ce_classify_enabled? %d\n",
pdev->cfg.ce_classify_enabled);
if (pdev->cfg.is_high_latency) {
diff --git a/core/dp/htt/htt_h2t.c b/core/dp/htt/htt_h2t.c
index 2989a74..ab673fa 100644
--- a/core/dp/htt/htt_h2t.c
+++ b/core/dp/htt/htt_h2t.c
@@ -306,7 +306,8 @@
qdf_nbuf_t msg;
uint32_t *msg_word;
- qdf_print("Receive flow steering configuration, disable gEnableFlowSteering(=0) in ini if FW doesnot support it\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "Receive flow steering configuration, disable gEnableFlowSteering(=0) in ini if FW doesnot support it\n");
pkt = htt_htc_pkt_alloc(pdev);
if (!pkt)
return QDF_STATUS_E_NOMEM; /* failure */
@@ -341,9 +342,11 @@
HTT_H2T_MSG_TYPE_SET(*msg_word, HTT_H2T_MSG_TYPE_RFS_CONFIG);
if (ol_cfg_is_flow_steering_enabled(pdev->ctrl_pdev)) {
HTT_RX_RFS_CONFIG_SET(*msg_word, 1);
- qdf_print("Enable Rx flow steering\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "Enable Rx flow steering\n");
} else {
- qdf_print("Disable Rx flow steering\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "Disable Rx flow steering\n");
}
SET_HTC_PACKET_INFO_TX(&pkt->htc_pkt,
@@ -366,7 +369,8 @@
*/
QDF_STATUS htt_h2t_rx_ring_rfs_cfg_msg_ll(struct htt_pdev_t *pdev)
{
- qdf_print("Doesnot support receive flow steering configuration\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "Doesnot support receive flow steering configuration\n");
return QDF_STATUS_SUCCESS;
}
#endif /* HELIUMPLUS */
@@ -439,7 +443,8 @@
tmp = qdf_get_upper_32_bits(pdev->rx_ring.base_paddr);
if (tmp & 0xfffffe0) {
- qdf_print("%s:%d paddr > 37 bits!. Trimmed.",
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "%s:%d paddr > 37 bits!. Trimmed.",
__func__, __LINE__);
tmp &= 0x01f;
}
@@ -471,9 +476,11 @@
enable_hdr = 1;
enable_ppdu_start = 1;
enable_ppdu_end = 1;
- qdf_print("Pkt log is enabled\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "%s : %d Pkt log is enabled\n", __func__, __LINE__);
} else {
- qdf_print("Pkt log is disabled\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "%s : %d Pkt log is disabled\n", __func__, __LINE__);
enable_ctrl_data = 0;
enable_mgmt_data = 0;
enable_null_data = 0;
@@ -500,7 +507,8 @@
enable_ppdu_start = 1;
enable_ppdu_end = 1;
/* Disable ASPM for monitor mode */
- qdf_print("Monitor mode is enabled\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "%s : %d Monitor mode is enabled\n", __func__, __LINE__);
}
HTT_RX_RING_CFG_ENABLED_802_11_HDR_SET(*msg_word, enable_hdr);
@@ -724,7 +732,8 @@
*/
QDF_STATUS htt_h2t_rx_ring_rfs_cfg_msg_hl(struct htt_pdev_t *pdev)
{
- qdf_print("Doesnot support Receive flow steering configuration\n");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "Doesnot support Receive flow steering configuration\n");
return QDF_STATUS_SUCCESS;
}
@@ -746,7 +755,8 @@
if (stats_type_upload_mask >= 1 << HTT_DBG_NUM_STATS ||
stats_type_reset_mask >= 1 << HTT_DBG_NUM_STATS) {
/* FIX THIS - add more details? */
- qdf_print("%#x %#x stats not supported\n",
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG,
+ "%#x %#x stats not supported\n",
stats_type_upload_mask, stats_type_reset_mask);
htt_htc_pkt_free(pdev, pkt);
return -EINVAL; /* failure */
diff --git a/core/dp/htt/htt_rx.c b/core/dp/htt/htt_rx.c
index 8e3577b..34078b5 100644
--- a/core/dp/htt/htt_rx.c
+++ b/core/dp/htt/htt_rx.c
@@ -1482,7 +1482,7 @@
qdf_nbuf_trim_tail(frag_nbuf, HTT_RX_BUF_SIZE - len);
HTT_PKT_DUMP(qdf_trace_hex_dump(QDF_MODULE_ID_TXRX,
- QDF_TRACE_LEVEL_FATAL,
+ QDF_TRACE_LEVEL_DEBUG,
qdf_nbuf_data(frag_nbuf),
qdf_nbuf_len(frag_nbuf)));
prev_frag_nbuf = frag_nbuf;
@@ -1505,7 +1505,7 @@
amsdu_len -= len;
qdf_nbuf_trim_tail(frag_nbuf, HTT_RX_BUF_SIZE - len);
HTT_PKT_DUMP(qdf_trace_hex_dump(QDF_MODULE_ID_TXRX,
- QDF_TRACE_LEVEL_FATAL,
+ QDF_TRACE_LEVEL_DEBUG,
qdf_nbuf_data(frag_nbuf),
qdf_nbuf_len(frag_nbuf)));
@@ -1783,7 +1783,7 @@
msg_word = (uint32_t *)rx_ind_data;
HTT_PKT_DUMP(qdf_trace_hex_dump(QDF_MODULE_ID_TXRX,
- QDF_TRACE_LEVEL_FATAL,
+ QDF_TRACE_LEVEL_DEBUG,
(void *)rx_ind_data,
(int)qdf_nbuf_len(rx_ind_msg)));
@@ -1844,7 +1844,7 @@
HTT_PKT_DUMP(qdf_trace_hex_dump(QDF_MODULE_ID_TXRX,
- QDF_TRACE_LEVEL_FATAL,
+ QDF_TRACE_LEVEL_DEBUG,
qdf_nbuf_data(msdu),
qdf_nbuf_len(msdu)));
last_frag = ((struct htt_rx_in_ord_paddr_ind_msdu_t *)
diff --git a/core/dp/htt/htt_t2h.c b/core/dp/htt/htt_t2h.c
index d68c696..fb19075 100644
--- a/core/dp/htt/htt_t2h.c
+++ b/core/dp/htt/htt_t2h.c
@@ -168,20 +168,22 @@
htc_pm_runtime_put(pdev->htc_pdev);
pdev->tgt_ver.major = HTT_VER_CONF_MAJOR_GET(*msg_word);
pdev->tgt_ver.minor = HTT_VER_CONF_MINOR_GET(*msg_word);
- qdf_print
- ("target uses HTT version %d.%d; host uses %d.%d",
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO,
+ "target uses HTT version %d.%d; host uses %d.%d",
pdev->tgt_ver.major, pdev->tgt_ver.minor,
HTT_CURRENT_VERSION_MAJOR,
HTT_CURRENT_VERSION_MINOR);
if (pdev->tgt_ver.major != HTT_CURRENT_VERSION_MAJOR)
- qdf_print
- ("*** Incompatible host/target HTT versions!");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_WARN,
+ "*** Incompatible host/target HTT versions!");
/* abort if the target is incompatible with the host */
qdf_assert(pdev->tgt_ver.major ==
HTT_CURRENT_VERSION_MAJOR);
if (pdev->tgt_ver.minor != HTT_CURRENT_VERSION_MINOR) {
- qdf_print("*** Warning: host/target HTT versions are ");
- qdf_print(" different, though compatible!");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_WARN,
+ "*** Warning: host/target HTT versions are ");
+ QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_WARN,
+ " different, though compatible!");
}
break;
}