From ed8ea05a7ef7cf9ab977e530329c0ce8b9cf378c Mon Sep 17 00:00:00 2001 From: Surya Prakash Raajen Date: Sun, 5 Jan 2020 16:07:24 +0530 Subject: [PATCH] qcacmn: Clear console log boot up error prints Clear console log boot up error prints by removing unnecessary prints, modifying prints to appropriate info or debug and by fixing the errors Change-Id: Ica3f126280399eef59b894236849e2079a62565f CRs-Fixed: 2597026 --- qdf/linux/src/qdf_trace.c | 2 -- .../src/target_if_direct_buf_rx_main.c | 6 ++--- wmi/src/wmi_unified.c | 24 +++++++++---------- 3 files changed, 15 insertions(+), 17 deletions(-) diff --git a/qdf/linux/src/qdf_trace.c b/qdf/linux/src/qdf_trace.c index c42d737b2d..822784dfb5 100644 --- a/qdf/linux/src/qdf_trace.c +++ b/qdf/linux/src/qdf_trace.c @@ -3058,8 +3058,6 @@ int qdf_print_ctrl_register(const struct category_info *cinfo, } } - pr_info("%s: Allocated print control object %d\n", - __func__, idx); return idx; } qdf_export_symbol(qdf_print_ctrl_register); diff --git a/target_if/direct_buf_rx/src/target_if_direct_buf_rx_main.c b/target_if/direct_buf_rx/src/target_if_direct_buf_rx_main.c index c266a74d47..ed634cf1f6 100644 --- a/target_if/direct_buf_rx/src/target_if_direct_buf_rx_main.c +++ b/target_if/direct_buf_rx/src/target_if_direct_buf_rx_main.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2017-2020 The Linux Foundation. All rights reserved. * * Permission to use, copy, modify, and/or distribute this software for * any purpose with or without fee is hereby granted, provided that the @@ -774,8 +774,8 @@ QDF_STATUS target_if_dbr_stop_ring_debug(struct wlan_objmgr_pdev *pdev, for (srng_id = 0; srng_id < DBR_SRNG_NUM; srng_id++) { ring_debug = &mod_debug->dbr_ring_debug[srng_id]; if (!ring_debug->entries) { - direct_buf_rx_err("DBR ring debug for module %d srng %d was already disabled", - mod_id, srng_id); + direct_buf_rx_debug("DBR ring debug for module %d srng %d was already disabled", + mod_id, srng_id); continue; } /* De-init debugsfs for this ring */ diff --git a/wmi/src/wmi_unified.c b/wmi/src/wmi_unified.c index 558e23478f..e4fbab3b9e 100644 --- a/wmi/src/wmi_unified.c +++ b/wmi/src/wmi_unified.c @@ -1911,14 +1911,14 @@ int wmi_unified_register_event(wmi_unified_t wmi_handle, if (event_id >= wmi_events_max || wmi_handle->wmi_events[event_id] == WMI_EVENT_ID_INVALID) { - QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_INFO, "%s: Event id %d is unavailable", __func__, event_id); return QDF_STATUS_E_FAILURE; } evt_id = wmi_handle->wmi_events[event_id]; if (wmi_unified_get_event_handler_ix(wmi_handle, evt_id) != -1) { - QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_INFO, "%s : event handler already registered 0x%x", __func__, evt_id); return QDF_STATUS_E_FAILURE; @@ -1968,7 +1968,7 @@ int wmi_unified_register_event_handler(wmi_unified_t wmi_handle, if (event_id >= wmi_events_max || wmi_handle->wmi_events[event_id] == WMI_EVENT_ID_INVALID) { - QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_INFO, "%s: Event id %d is unavailable", __func__, event_id); return QDF_STATUS_E_FAILURE; @@ -1976,8 +1976,8 @@ int wmi_unified_register_event_handler(wmi_unified_t wmi_handle, evt_id = wmi_handle->wmi_events[event_id]; if (wmi_unified_get_event_handler_ix(wmi_handle, evt_id) != -1) { - WMI_LOGE("event handler already registered 0x%x", - evt_id); + WMI_LOGI("event handler already registered 0x%x", + evt_id); return QDF_STATUS_E_FAILURE; } if (soc->max_event_idx == WMI_UNIFIED_MAX_EVENT) { @@ -2015,7 +2015,7 @@ int wmi_unified_unregister_event(wmi_unified_t wmi_handle, if (event_id >= wmi_events_max || wmi_handle->wmi_events[event_id] == WMI_EVENT_ID_INVALID) { - QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_WMI, QDF_TRACE_LEVEL_INFO, "%s: Event id %d is unavailable", __func__, event_id); return QDF_STATUS_E_FAILURE; @@ -2024,8 +2024,8 @@ int wmi_unified_unregister_event(wmi_unified_t wmi_handle, idx = wmi_unified_get_event_handler_ix(wmi_handle, evt_id); if (idx == -1) { - WMI_LOGE("event handler is not registered: evt id 0x%x", - evt_id); + WMI_LOGI("event handler is not registered: evt id 0x%x", + evt_id); return QDF_STATUS_E_FAILURE; } wmi_handle->event_handler[idx] = NULL; @@ -2062,16 +2062,16 @@ int wmi_unified_unregister_event_handler(wmi_unified_t wmi_handle, if (event_id >= wmi_events_max || wmi_handle->wmi_events[event_id] == WMI_EVENT_ID_INVALID) { - WMI_LOGE("Event id %d is unavailable", - event_id); + WMI_LOGI("Event id %d is unavailable", + event_id); return QDF_STATUS_E_FAILURE; } evt_id = wmi_handle->wmi_events[event_id]; idx = wmi_unified_get_event_handler_ix(wmi_handle, evt_id); if (idx == -1) { - WMI_LOGE("event handler is not registered: evt id 0x%x", - evt_id); + WMI_LOGI("event handler is not registered: evt id 0x%x", + evt_id); return QDF_STATUS_E_FAILURE; } wmi_handle->event_handler[idx] = NULL;