diff options
| author | Srinivas Girigowda <sgirigow@qca.qualcomm.com> | 2016-04-09 23:03:53 -0700 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2016-04-13 10:13:58 +0530 |
| commit | b131b526a7d6f1c7f9f25ffa3658b5b47404615c (patch) | |
| tree | 2f361d192a4267f700dbc6f1408aefca1ffe92a3 | |
| parent | fdda9c0af64d6e5cdf006e2d8dd57e655821a962 (diff) | |
qcacld-2.0: Move certain logs to appropriate log levels
Some of the logs are printing quite often and spamming the kernel logs.
Hence move them to appropriate log levels and remove the useless debug
messages.
Change-Id: If38d7a6a3cf13a5879f6a628d3d7eb4f5156a95b
CRs-Fixed: 1001441
| -rw-r--r-- | CORE/CLD_TXRX/HTT/htt_rx.c | 1 | ||||
| -rw-r--r-- | CORE/SERVICES/HIF/PCIe/if_pci.c | 8 | ||||
| -rw-r--r-- | CORE/SERVICES/WMA/wma.c | 11 | ||||
| -rw-r--r-- | CORE/SERVICES/WMI/wmi_tlv_helper.c | 2 |
4 files changed, 12 insertions, 10 deletions
diff --git a/CORE/CLD_TXRX/HTT/htt_rx.c b/CORE/CLD_TXRX/HTT/htt_rx.c index 9cc203cd23a2..3fe3ad69ad5a 100644 --- a/CORE/CLD_TXRX/HTT/htt_rx.c +++ b/CORE/CLD_TXRX/HTT/htt_rx.c @@ -1909,7 +1909,6 @@ htt_rx_amsdu_rx_in_order_pop_ll( #ifdef FEATURE_MARK_FIRST_WOW_WAKEUP if (HTT_RX_IN_ORD_PADDR_IND_MSDU_INFO_GET(*(msg_word + 1)) & FW_MSDU_INFO_FIRST_WAKEUP_M) { - adf_os_print("%s: first packet after WOW wakeup\n", __func__); adf_nbuf_update_skb_mark(msdu, HTT_MARK_FIRST_WAKEUP_PACKET); } #endif diff --git a/CORE/SERVICES/HIF/PCIe/if_pci.c b/CORE/SERVICES/HIF/PCIe/if_pci.c index dfe18fa17d04..b7b0f6f52544 100644 --- a/CORE/SERVICES/HIF/PCIe/if_pci.c +++ b/CORE/SERVICES/HIF/PCIe/if_pci.c @@ -2651,7 +2651,7 @@ __hif_pci_suspend(struct pci_dev *pdev, pm_message_t state, bool runtime_pm) goto out; } - pr_info("%s: Suspend completes (D0WOW)\n", __func__); + pr_debug("%s: Suspend completes (D0WOW)\n", __func__); ret = 0; goto out; } @@ -2702,11 +2702,11 @@ __hif_pci_suspend(struct pci_dev *pdev, pm_message_t state, bool runtime_pm) } skip: - pr_info("%s: Suspend completes%s in%s mode event:%d device_state:%d\n", + pr_debug("%s: Suspend completes%s in%s mode event:%d device_state:%d\n", __func__, runtime_pm ? " for runtime pm" : "", wma_is_wow_mode_selected(temp_module) ? " wow" : " pdev", state.event, val); - printk("%s: Suspend completes%s\n", __func__, + pr_debug("%s: Suspend completes%s\n", __func__, runtime_pm ? " for runtime pm" : ""); ret = 0; @@ -2845,7 +2845,7 @@ skip: } #endif - pr_info("%s: Resume completes%s in%s mode\n", __func__, + pr_debug("%s: Resume completes%s in%s mode\n", __func__, runtime_pm ? " for runtime pm" : "", wma_is_wow_mode_selected(temp_module) ? " wow" : " pdev"); out: diff --git a/CORE/SERVICES/WMA/wma.c b/CORE/SERVICES/WMA/wma.c index a3517dbc8dc9..65ba52653220 100644 --- a/CORE/SERVICES/WMA/wma.c +++ b/CORE/SERVICES/WMA/wma.c @@ -20508,10 +20508,13 @@ static int wma_wow_wakeup_host_event(void *handle, u_int8_t *event, wake_info = param_buf->fixed_param; - WMA_LOGA("WOW wakeup host event received (reason: %s(%d)) for vdev %d", - wma_wow_wake_reason_str(wake_info->wake_reason, wma), - wake_info->wake_reason, - wake_info->vdev_id); + if ((wake_info->wake_reason != WOW_REASON_UNSPECIFIED) || + (wake_info->wake_reason == WOW_REASON_UNSPECIFIED && + !wmi_get_runtime_pm_inprogress(wma->wmi_handle))) + WMA_LOGA("WOW wakeup host event received (reason: %s(%d)) for vdev %d", + wma_wow_wake_reason_str(wake_info->wake_reason, wma), + wake_info->wake_reason, + wake_info->vdev_id); vos_event_set(&wma->wma_resume_event); diff --git a/CORE/SERVICES/WMI/wmi_tlv_helper.c b/CORE/SERVICES/WMI/wmi_tlv_helper.c index 1edd553fbbad..2dc7e53e5250 100644 --- a/CORE/SERVICES/WMI/wmi_tlv_helper.c +++ b/CORE/SERVICES/WMI/wmi_tlv_helper.c @@ -521,7 +521,7 @@ wmitlv_check_and_pad_tlvs( } /* Warning: Needs to allocate a larger structure and pad with zeros */ - wmi_tlv_print_error("%s: WARN: TLV array of structures needs padding. tlv_size_diff=%d\n", + wmi_tlv_print_verbose("%s: WARN: TLV array of structures needs padding. tlv_size_diff=%d\n", __func__, tlv_size_diff); /* incoming structure length */ |