From 51264a63e1f876f7c490c826fe640a3d7a0b8518 Mon Sep 17 00:00:00 2001 From: Rakesh Pillai Date: Wed, 8 May 2019 19:15:56 +0530 Subject: [PATCH] qcacmn: Disable excessive logging during dp init Excessive logging during dp init causes watchdog bite when multiple init-deinit cycle is followed. Change the loglevel to debug, so that it does not flood the console with logs. CRs-Fixed: 2444944 Change-Id: I43daab413800424651e3f5edf558360886d7d7fe --- dp/wifi3.0/dp_htt.c | 16 +++++----------- hif/src/ce/ce_service_srng.c | 2 +- hif/src/hif_exec.c | 12 ++++++------ hif/src/pcie/if_pci.c | 2 +- 4 files changed, 13 insertions(+), 19 deletions(-) diff --git a/dp/wifi3.0/dp_htt.c b/dp/wifi3.0/dp_htt.c index 400d2ce9c4f3..3fdc4d03f734 100644 --- a/dp/wifi3.0/dp_htt.c +++ b/dp/wifi3.0/dp_htt.c @@ -678,8 +678,7 @@ int htt_srng_setup(void *htt_soc, int mac_id, void *hal_srng, else HTT_SRING_SETUP_PDEV_ID_SET(*msg_word, mac_id); - QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO, - "%s: mac_id %d", __func__, mac_id); + dp_info("%s: mac_id %d", __func__, mac_id); HTT_SRING_SETUP_RING_TYPE_SET(*msg_word, htt_ring_type); /* TODO: Discuss with FW on changing this to unique ID and using * htt_ring_type to send the type of ring @@ -704,15 +703,10 @@ int htt_srng_setup(void *htt_soc, int mac_id, void *hal_srng, HTT_SRING_SETUP_ENTRY_SIZE_SET(*msg_word, ring_entry_size); HTT_SRING_SETUP_RING_SIZE_SET(*msg_word, (ring_entry_size * srng_params.num_entries)); - QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO, - "%s: entry_size %d", __func__, - ring_entry_size); - QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO, - "%s: num_entries %d", __func__, - srng_params.num_entries); - QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO, - "%s: ring_size %d", __func__, - (ring_entry_size * srng_params.num_entries)); + dp_info("%s: entry_size %d", __func__, ring_entry_size); + dp_info("%s: num_entries %d", __func__, srng_params.num_entries); + dp_info("%s: ring_size %d", __func__, + (ring_entry_size * srng_params.num_entries)); if (htt_ring_type == HTT_SW_TO_HW_RING) HTT_SRING_SETUP_RING_MISC_CFG_FLAG_LOOPCOUNT_DISABLE_SET( *msg_word, 1); diff --git a/hif/src/ce/ce_service_srng.c b/hif/src/ce/ce_service_srng.c index d3f5790c5736..a6d0d63a91c1 100644 --- a/hif/src/ce/ce_service_srng.c +++ b/hif/src/ce/ce_service_srng.c @@ -651,7 +651,7 @@ static void ce_srng_src_ring_setup(struct hif_softc *scn, uint32_t ce_id, { struct hal_srng_params ring_params = {0}; - HIF_INFO("%s: ce_id %d", __func__, ce_id); + hif_debug("%s: ce_id %d", __func__, ce_id); ring_params.ring_base_paddr = src_ring->base_addr_CE_space; ring_params.ring_base_vaddr = src_ring->base_addr_owner_space; diff --git a/hif/src/hif_exec.c b/hif/src/hif_exec.c index e5cc84fdbd25..4135a6c5bfa8 100644 --- a/hif/src/hif_exec.c +++ b/hif/src/hif_exec.c @@ -811,8 +811,8 @@ qdf_export_symbol(hif_register_ext_group); struct hif_exec_context *hif_exec_create(enum hif_exec_type type, uint32_t scale) { - HIF_INFO("%s: create exec_type %d budget %d\n", - __func__, type, QCA_NAPI_BUDGET * scale); + hif_debug("%s: create exec_type %d budget %d\n", + __func__, type, QCA_NAPI_BUDGET * scale); switch (type) { case HIF_EXEC_NAPI_TYPE: @@ -861,10 +861,10 @@ void hif_deregister_exec_group(struct hif_opaque_softc *hif_ctx, if (!hif_ext_group) continue; - HIF_INFO("%s: Deregistering grp id %d name %s\n", - __func__, - hif_ext_group->grp_id, - hif_ext_group->context_name); + hif_debug("%s: Deregistering grp id %d name %s\n", + __func__, + hif_ext_group->grp_id, + hif_ext_group->context_name); if (strcmp(hif_ext_group->context_name, context_name) == 0) { hif_ext_group->sched_ops->kill(hif_ext_group); diff --git a/hif/src/pcie/if_pci.c b/hif/src/pcie/if_pci.c index cafb15ad8177..74be81cffd31 100644 --- a/hif/src/pcie/if_pci.c +++ b/hif/src/pcie/if_pci.c @@ -2341,7 +2341,7 @@ static int hif_ce_srng_msi_free_irq(struct hif_softc *scn) msi_data = (ce_id % msi_data_count) + msi_irq_start; irq = pld_get_msi_irq(scn->qdf_dev->dev, msi_data); - HIF_INFO("%s: (ce_id %d, msi_data %d, irq %d)", __func__, + hif_debug("%s: (ce_id %d, msi_data %d, irq %d)", __func__, ce_id, msi_data, irq); free_irq(irq, &ce_sc->tasklets[ce_id]);