diff options
| author | Sachin Ahuja <sahuja@codeaurora.org> | 2016-12-28 16:02:34 +0530 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2017-01-06 20:57:27 -0800 |
| commit | cabc146b4e115088466305f159587139eb0471d9 (patch) | |
| tree | b26937e0275ff7b0761a13e453fe061982009fe5 | |
| parent | 6d29e4fb88e0411cda12018ce0dfb376b9b5c371 (diff) | |
qcacld-3.0: Change the loglevel of prints appropiately
Currently there are many instances in driver where log levels
are defined as error but actually those are not error level.
This can lead to WD bark if these logs are printed frequently.
Change these log prints to appropiate level.
Change-Id: If21a688d6a1d84c070427cb2a7980c8bcb4da495
CRs-Fixed: 1105924
| -rw-r--r-- | core/hdd/src/wlan_hdd_softap_tx_rx.c | 26 | ||||
| -rw-r--r-- | core/hdd/src/wlan_hdd_stats.c | 16 | ||||
| -rw-r--r-- | core/hdd/src/wlan_hdd_tdls.c | 34 | ||||
| -rw-r--r-- | core/hdd/src/wlan_hdd_wext.c | 34 |
4 files changed, 55 insertions, 55 deletions
diff --git a/core/hdd/src/wlan_hdd_softap_tx_rx.c b/core/hdd/src/wlan_hdd_softap_tx_rx.c index e807d54ef3c4..8af1e9ae1d3f 100644 --- a/core/hdd/src/wlan_hdd_softap_tx_rx.c +++ b/core/hdd/src/wlan_hdd_softap_tx_rx.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2016 The Linux Foundation. All rights reserved. + * Copyright (c) 2012-2017 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -61,25 +61,25 @@ */ static void hdd_softap_dump_sk_buff(struct sk_buff *skb) { - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: head = %p ", __func__, skb->head); /* QDF_TRACE( QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR,"%s: data = %p ", __func__, skb->data); */ - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: tail = %p ", __func__, skb->tail); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: end = %p ", __func__, skb->end); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: len = %d ", __func__, skb->len); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: data_len = %d ", __func__, skb->data_len); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "%s: mac_len = %d", __func__, skb->mac_len); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "0x%x 0x%x 0x%x 0x%x 0x%x 0x%x 0x%x 0x%x ", skb->data[0], skb->data[1], skb->data[2], skb->data[3], skb->data[4], skb->data[5], skb->data[6], skb->data[7]); - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_INFO, "0x%x 0x%x 0x%x 0x%x 0x%x 0x%x 0x%x 0x%x", skb->data[8], skb->data[9], skb->data[10], skb->data[11], skb->data[12], skb->data[13], skb->data[14], skb->data[15]); @@ -456,7 +456,7 @@ static void __hdd_softap_tx_timeout(struct net_device *dev) */ hdd_ctx = WLAN_HDD_GET_CTX(adapter); if (cds_is_driver_recovering()) { - QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_SAP_DATA, QDF_TRACE_LEVEL_WARN, "%s: Recovery in Progress. Ignore!!!", __func__); return; } @@ -466,14 +466,14 @@ static void __hdd_softap_tx_timeout(struct net_device *dev) for (i = 0; i < NUM_TX_QUEUES; i++) { txq = netdev_get_tx_queue(dev, i); QDF_TRACE(QDF_MODULE_ID_HDD_DATA, - QDF_TRACE_LEVEL_ERROR, + QDF_TRACE_LEVEL_INFO, "Queue%d status: %d txq->trans_start %lu", i, netif_tx_queue_stopped(txq), txq->trans_start); } wlan_hdd_display_netif_queue_history(hdd_ctx); ol_tx_dump_flow_pool_info(); - QDF_TRACE(QDF_MODULE_ID_HDD_DATA, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD_DATA, QDF_TRACE_LEVEL_INFO, "carrier state: %d", netif_carrier_ok(dev)); } @@ -897,7 +897,7 @@ QDF_STATUS hdd_softap_stop_bss(hdd_adapter_t *pAdapter) * unloading */ if (cds_is_load_or_unload_in_progress()) { - hdd_err("Loading_unloading in Progress, state: 0x%x. Ignore!!!", + hdd_warn("Loading_unloading in Progress, state: 0x%x. Ignore!!!", cds_get_driver_state()); return QDF_STATUS_E_PERM; } diff --git a/core/hdd/src/wlan_hdd_stats.c b/core/hdd/src/wlan_hdd_stats.c index 50638d3eafcb..fd8092334f0a 100644 --- a/core/hdd/src/wlan_hdd_stats.c +++ b/core/hdd/src/wlan_hdd_stats.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2016 The Linux Foundation. All rights reserved. + * Copyright (c) 2012-2017 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -584,7 +584,7 @@ static bool hdd_get_interface_info(hdd_adapter_t *pAdapter, staMac = (uint8_t *) &(pAdapter->macAddressCurrent. bytes[0]); - hdd_err("client " MAC_ADDRESS_STR + hdd_warn("client " MAC_ADDRESS_STR " is in the middle of WPS/EAPOL exchange.", MAC_ADDR_ARRAY(staMac)); pInfo->state = WIFI_AUTHENTICATING; @@ -1207,7 +1207,7 @@ __wlan_hdd_cfg80211_ll_stats_set(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -1330,7 +1330,7 @@ __wlan_hdd_cfg80211_ll_stats_get(struct wiphy *wiphy, /* ENTER() intentionally not used in a frequently invoked API */ if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -1459,7 +1459,7 @@ __wlan_hdd_cfg80211_ll_stats_clear(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -1602,7 +1602,7 @@ static int __wlan_hdd_cfg80211_stats_ext_request(struct wiphy *wiphy, return ret_val; if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -1772,7 +1772,7 @@ static int __wlan_hdd_cfg80211_get_station(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EINVAL; } @@ -2422,7 +2422,7 @@ static int __wlan_hdd_cfg80211_dump_survey(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EINVAL; } diff --git a/core/hdd/src/wlan_hdd_tdls.c b/core/hdd/src/wlan_hdd_tdls.c index a044675a3da6..a2881f72ed8c 100644 --- a/core/hdd/src/wlan_hdd_tdls.c +++ b/core/hdd/src/wlan_hdd_tdls.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2016 The Linux Foundation. All rights reserved. + * Copyright (c) 2012-2017 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -707,7 +707,7 @@ int wlan_hdd_tdls_init(hdd_adapter_t *pAdapter) pHddCtx->tdls_mode = eTDLS_SUPPORT_NOT_ENABLED; pAdapter->sessionCtx.station.pHddTdlsCtx = NULL; mutex_unlock(&pHddCtx->tdls_lock); - hdd_err("TDLS not enabled (%d) or FW doesn't support", + hdd_warn("TDLS not enabled (%d) or FW doesn't support", pHddCtx->config->fEnableTDLSSupport); return 0; } @@ -1722,7 +1722,7 @@ int wlan_hdd_tdls_set_params(struct net_device *dev, /* config->tdls is mapped to 0->1, 1->2, 2->3 */ req_tdls_mode = config->tdls + 1; if (pHddCtx->tdls_mode == req_tdls_mode) { - hdd_err("Already in mode %d", config->tdls); + hdd_warn("Already in mode %d", config->tdls); return -EINVAL; } @@ -1733,7 +1733,7 @@ int wlan_hdd_tdls_set_params(struct net_device *dev, sizeof(tdls_config_params_t)); } - hdd_err("iw set tdls params: %d %d %d %d %d %d %d", + hdd_notice("iw set tdls params: %d %d %d %d %d %d %d", config->tdls, config->tx_period_t, config->tx_packet_n, @@ -2962,7 +2962,7 @@ void wlan_hdd_tdls_timer_restart(hdd_adapter_t *pAdapter, /* Check whether driver load unload is in progress */ if (cds_is_load_or_unload_in_progress()) { - hdd_err("Driver load/unload is in progress."); + hdd_warn("Driver load/unload is in progress."); return; } @@ -3109,7 +3109,7 @@ int wlan_hdd_tdls_get_status(hdd_adapter_t *pAdapter, } if (pHddCtx->config->fTDLSExternalControl && (false == curr_peer->isForcedPeer)) { - hdd_err("curr_peer is not Forced"); + hdd_notice("curr_peer is not Forced"); *state = QCA_WIFI_HAL_TDLS_DISABLED; *reason = eTDLS_LINK_UNSPECIFIED; } else { @@ -3211,7 +3211,7 @@ __wlan_hdd_cfg80211_exttdls_get_status(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -3300,7 +3300,7 @@ __wlan_hdd_cfg80211_configure_tdls_mode(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -3542,7 +3542,7 @@ __wlan_hdd_cfg80211_exttdls_enable(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -3663,7 +3663,7 @@ static int __wlan_hdd_cfg80211_exttdls_disable(struct wiphy *wiphy, ENTER_DEV(dev); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EPERM; } @@ -3973,7 +3973,7 @@ static int __wlan_hdd_cfg80211_tdls_mgmt(struct wiphy *wiphy, #endif if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EINVAL; } @@ -4189,13 +4189,13 @@ static int __wlan_hdd_cfg80211_tdls_mgmt(struct wiphy *wiphy, rc, pAdapter->mgmtTxCompletionStatus); if (cds_is_driver_recovering()) { - hdd_err("Recovery in Progress. State: 0x%x Ignore!!!", + hdd_warn("Recovery in Progress. State: 0x%x Ignore!!!", cds_get_driver_state()); return -EAGAIN; } if (cds_is_driver_unloading()) { - hdd_err("Unload in progress. State: 0x%x Ignore!!!", + hdd_warn("Unload in progress. State: 0x%x Ignore!!!", cds_get_driver_state()); return -EAGAIN; } @@ -4484,7 +4484,7 @@ static int __wlan_hdd_cfg80211_tdls_oper(struct wiphy *wiphy, ENTER(); if (QDF_GLOBAL_FTM_MODE == hdd_get_conparam()) { - hdd_err("Command not allowed in FTM mode"); + hdd_warn("Command not allowed in FTM mode"); return -EINVAL; } @@ -4793,7 +4793,7 @@ static int __wlan_hdd_cfg80211_tdls_oper(struct wiphy *wiphy, return -EINVAL; } - QDF_TRACE(QDF_MODULE_ID_HDD, QDF_TRACE_LEVEL_ERROR, + QDF_TRACE(QDF_MODULE_ID_HDD, QDF_TRACE_LEVEL_INFO, "%s: NL80211_TDLS_DISABLE_LINK for peer " MAC_ADDRESS_STR " link_status: %d", __func__, MAC_ADDR_ARRAY(peer), @@ -5013,7 +5013,7 @@ int hdd_set_tdls_offchannel(hdd_context_t *hdd_ctx, int offchannel) return -EINVAL; } } else { - hdd_err("Either TDLS or TDLS Off-channel is not enabled"); + hdd_warn("Either TDLS or TDLS Off-channel is not enabled"); return -ENOTSUPP; } hdd_notice("change tdls off channel from %d to %d", @@ -5059,7 +5059,7 @@ int hdd_set_tdls_secoffchanneloffset(hdd_context_t *hdd_ctx, int offchanoffset) return -EINVAL; } /* end switch */ } else { - hdd_err("Either TDLS or TDLS Off-channel is not enabled"); + hdd_warn("Either TDLS or TDLS Off-channel is not enabled"); return -ENOTSUPP; } hdd_notice("change tdls secondary off channel offset to 0x%x", diff --git a/core/hdd/src/wlan_hdd_wext.c b/core/hdd/src/wlan_hdd_wext.c index 1184fa10d9a7..b04e4595a835 100644 --- a/core/hdd/src/wlan_hdd_wext.c +++ b/core/hdd/src/wlan_hdd_wext.c @@ -1803,7 +1803,7 @@ QDF_STATUS wlan_hdd_get_rssi(hdd_adapter_t *pAdapter, int8_t *rssi_value) return QDF_STATUS_E_FAULT; } if (cds_is_driver_recovering()) { - hdd_err("Recovery in Progress. State: 0x%x Ignore!!!", + hdd_warn("Recovery in Progress. State: 0x%x Ignore!!!", cds_get_driver_state()); /* return a cached value */ *rssi_value = pAdapter->rssi; @@ -2463,7 +2463,7 @@ int hdd_set_rx_stbc(hdd_adapter_t *adapter, int value) tHalHandle hal = WLAN_HDD_GET_HAL_CTX(adapter); int ret; - hdd_alert("%d", value); + hdd_alert("set rx_stbc : %d", value); if (value) { /* make sure HT capabilities allow this */ QDF_STATUS status; @@ -4227,7 +4227,7 @@ QDF_STATUS wlan_hdd_get_class_astats(hdd_adapter_t *pAdapter) return QDF_STATUS_E_FAULT; } if (cds_is_driver_recovering()) { - hdd_err("Recovery in Progress. State: 0x%x Ignore!!!", + hdd_warn("Recovery in Progress. State: 0x%x Ignore!!!", cds_get_driver_state()); return QDF_STATUS_SUCCESS; } @@ -5326,7 +5326,7 @@ static int __iw_set_mlme(struct net_device *dev, WLAN_CONTROL_PATH); } else { - hdd_err("%d Command Disassociate/Deauthenticate called but station is not in associated state", + hdd_warn("%d Command Disassociate/Deauthenticate called but station is not in associated state", (int)mlme->cmd); } break; @@ -5808,7 +5808,7 @@ static int __iw_setint_getnone(struct net_device *dev, case 0x03: enable_mp = (set_value & 0x01) ? 1 : 0; enable_pbm = (set_value & 0x02) ? 1 : 0; - hdd_err("magic packet ? = %s pattern byte matching ? = %s", + hdd_notice("magic packet ? = %s pattern byte matching ? = %s", (enable_mp ? "YES" : "NO"), (enable_pbm ? "YES" : "NO")); hdd_enter_wowl(pAdapter, enable_mp, enable_pbm); @@ -7031,7 +7031,7 @@ static int __iw_setchar_getnone(struct net_device *dev, } break; case WE_SET_AP_WPS_IE: - hdd_err("Received WE_SET_AP_WPS_IE"); + hdd_notice("Received WE_SET_AP_WPS_IE"); sme_update_p2p_ie(WLAN_HDD_GET_HAL_CTX(pAdapter), pBuffer, s_priv_data.length); break; @@ -8055,7 +8055,7 @@ static int __iw_get_char_setnone(struct net_device *dev, #ifdef WLAN_FEATURE_11W case WE_GET_11W_INFO: { - hdd_err("WE_GET_11W_ENABLED = %d", + hdd_notice("WE_GET_11W_ENABLED = %d", pWextState->roamProfile.MFPEnabled); snprintf(extra, WE_MAX_STR_LEN, @@ -8486,7 +8486,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_CLIST_CMD: { - hdd_err("<iwpriv wlan0 pm_clist> is called"); + hdd_notice("<iwpriv wlan0 pm_clist> is called"); cds_incr_connection_count_utfw(apps_args[0], apps_args[1], apps_args[2], apps_args[3], apps_args[4], apps_args[5], apps_args[6], @@ -8496,7 +8496,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_DLIST_CMD: { - hdd_err("<iwpriv wlan0 pm_dlist> is called"); + hdd_notice("<iwpriv wlan0 pm_dlist> is called"); cds_decr_connection_count_utfw(apps_args[0], apps_args[1]); } @@ -8504,7 +8504,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_ULIST_CMD: { - hdd_err("<iwpriv wlan0 pm_ulist> is called"); + hdd_notice("<iwpriv wlan0 pm_ulist> is called"); cds_update_connection_info_utfw(apps_args[0], apps_args[1], apps_args[2], apps_args[3], apps_args[4], apps_args[5], apps_args[6], @@ -8514,7 +8514,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_DBS_CMD: { - hdd_err("<iwpriv wlan0 pm_dbs> is called"); + hdd_notice("<iwpriv wlan0 pm_dbs> is called"); if (apps_args[0] == 0) wma_set_dbs_capability_ut(0); else @@ -8534,7 +8534,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, uint8_t weight_list[QDF_MAX_NUM_CHAN] = {0}; uint32_t pcl_len = 0, i = 0; - hdd_err("<iwpriv wlan0 pm_pcl> is called"); + hdd_notice("<iwpriv wlan0 pm_pcl> is called"); cds_get_pcl(apps_args[0], pcl, &pcl_len, @@ -8551,7 +8551,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, struct cds_conc_connection_info *conn_info; uint32_t i = 0, len = 0; - hdd_err("<iwpriv wlan0 pm_cinfo> is called"); + hdd_notice("<iwpriv wlan0 pm_cinfo> is called"); conn_info = cds_get_conn_info(&len); pr_info("+-----------------------------+\n"); for (i = 0; i < len; i++) { @@ -8599,7 +8599,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_QUERY_ACTION_CMD: { enum cds_conc_next_action action; - hdd_err("<iwpriv wlan0 pm_query_action> is called"); + hdd_notice("<iwpriv wlan0 pm_query_action> is called"); action = cds_current_connections_update(pAdapter->sessionId, apps_args[0], SIR_UPDATE_REASON_UT); @@ -8609,7 +8609,7 @@ static int __iw_set_var_ints_getnone(struct net_device *dev, case WE_POLICY_MANAGER_QUERY_ALLOW_CMD: { bool allow; - hdd_err("<iwpriv wlan0 pm_query_allow> is called"); + hdd_notice("<iwpriv wlan0 pm_query_allow> is called"); allow = cds_allow_concurrency( apps_args[0], apps_args[1], apps_args[2]); pr_info("allow %d {0 = don't allow, 1 = allow}", allow); @@ -9753,7 +9753,7 @@ static int wlan_hdd_set_filter(hdd_context_t *hdd_ctx, int i = 0; if (hdd_ctx->config->disablePacketFilter) { - hdd_err("packet filtering disabled in ini returning"); + hdd_warn("packet filtering disabled in ini returning"); return 0; } @@ -10658,7 +10658,7 @@ static int __iw_set_two_ints_getnone(struct net_device *dev, break; #ifdef WLAN_DEBUG case WE_SET_FW_CRASH_INJECT: - hdd_err("WE_SET_FW_CRASH_INJECT: %d %d", + hdd_notice("WE_SET_FW_CRASH_INJECT: %d %d", value[1], value[2]); pr_err("SSR is triggered by iwpriv CRASH_INJECT: %d %d\n", value[1], value[2]); |
