From 1bc742d65e447211184b13597d7362ef7d5c1027 Mon Sep 17 00:00:00 2001 From: Manjunathappa Prakash Date: Tue, 14 Aug 2018 18:13:43 -0700 Subject: [PATCH] qcacld-3.0: Do not log DP info logs on the console Reduce the log level so that logs appear on cnss-diag log instead on console. Change-Id: I66ba3a941b9c6be9ec4fb04af39f9362822787d2 CRs-Fixed: 2296841 --- core/dp/htt/htt.c | 4 ++-- core/dp/htt/htt_h2t.c | 12 ++++++------ core/dp/htt/htt_rx_ll.c | 2 +- core/dp/htt/htt_t2h.c | 20 ++++++++++---------- 4 files changed, 19 insertions(+), 19 deletions(-) diff --git a/core/dp/htt/htt.c b/core/dp/htt/htt.c index 7ade27d8ee014..ae15b37cf4335 100644 --- a/core/dp/htt/htt.c +++ b/core/dp/htt/htt.c @@ -437,13 +437,13 @@ htt_pdev_alloc(ol_txrx_pdev_handle txrx_pdev, pdev->cfg.is_full_reorder_offload = ol_cfg_is_full_reorder_offload(pdev->ctrl_pdev); - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "full_reorder_offloaded %d", (int)pdev->cfg.is_full_reorder_offload); pdev->cfg.ce_classify_enabled = ol_cfg_is_ce_classify_enabled(ctrl_pdev); - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "ce_classify %d", pdev->cfg.ce_classify_enabled); diff --git a/core/dp/htt/htt_h2t.c b/core/dp/htt/htt_h2t.c index 0d9734a44c966..94f6f4d1f4f94 100644 --- a/core/dp/htt/htt_h2t.c +++ b/core/dp/htt/htt_h2t.c @@ -304,7 +304,7 @@ QDF_STATUS htt_h2t_rx_ring_rfs_cfg_msg_ll(struct htt_pdev_t *pdev) uint32_t msg_local; struct cds_config_info *cds_cfg; - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "Receive flow steering configuration, disable gEnableFlowSteering(=0) in ini if FW doesnot support it\n"); pkt = htt_htc_pkt_alloc(pdev); if (!pkt) @@ -340,22 +340,22 @@ QDF_STATUS htt_h2t_rx_ring_rfs_cfg_msg_ll(struct htt_pdev_t *pdev) HTT_H2T_MSG_TYPE_SET(msg_local, HTT_H2T_MSG_TYPE_RFS_CONFIG); if (ol_cfg_is_flow_steering_enabled(pdev->ctrl_pdev)) { HTT_RX_RFS_CONFIG_SET(msg_local, 1); - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "Enable Rx flow steering"); } else { - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "Disable Rx flow steering"); } cds_cfg = cds_get_ini_config(); if (cds_cfg != NULL) { msg_local |= ((cds_cfg->max_msdus_per_rxinorderind & 0xff) << 16); - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "Updated maxMSDUsPerRxInd"); } *msg_word = msg_local; - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "%s: Sending msg_word: 0x%08x", __func__, *msg_word); @@ -490,7 +490,7 @@ QDF_STATUS htt_h2t_rx_ring_cfg_msg_ll(struct htt_pdev_t *pdev) enable_hdr = 1; enable_ppdu_start = 1; enable_ppdu_end = 1; - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "%s : %d Pkt log is enabled\n", __func__, __LINE__); } else { QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, diff --git a/core/dp/htt/htt_rx_ll.c b/core/dp/htt/htt_rx_ll.c index 95d171f04116e..018e2b5914446 100644 --- a/core/dp/htt/htt_rx_ll.c +++ b/core/dp/htt/htt_rx_ll.c @@ -1844,7 +1844,7 @@ int htt_rx_attach(struct htt_pdev_t *pdev) htt_rx_ring_fill_n(pdev, pdev->rx_ring.fill_level); if (pdev->cfg.is_full_reorder_offload) { - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, "HTT: full reorder offload enabled"); htt_rx_amsdu_pop = htt_rx_amsdu_rx_in_order_pop_ll; htt_rx_frag_pop = htt_rx_amsdu_rx_in_order_pop_ll; diff --git a/core/dp/htt/htt_t2h.c b/core/dp/htt/htt_t2h.c index 2e5ced2943d4b..8a0566fb796df 100644 --- a/core/dp/htt/htt_t2h.c +++ b/core/dp/htt/htt_t2h.c @@ -204,22 +204,22 @@ static void htt_t2h_lp_msg_handler(void *context, qdf_nbuf_t htt_t2h_msg, 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_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); + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, + "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_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_WARN, - "*** Incompatible host/target HTT versions!"); + "*** 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_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!"); + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, + "*** Warning: host/target HTT versions are "); + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO_LOW, + "different, though compatible!"); } break; }