diff options
author | Srinivas Dasari <dasaris@codeaurora.org> | 2020-03-24 15:38:42 +0530 |
---|---|---|
committer | Paul Chen <chenpaul@google.com> | 2020-04-09 05:17:50 +0000 |
commit | 60428eafe6d6525337128f2a7278627cae09f7a7 (patch) | |
tree | 2fb1933ec54a3bd63b78317cce28091d25b135ea | |
parent | 3de8877b6f6729fe3dca9217cf8c50bf029a2db6 (diff) | |
download | qcacld-60428eafe6d6525337128f2a7278627cae09f7a7.tar.gz |
qcacld-3.0: cleanup NAN/NDP logging
Few NAN/NDP logs are redundant in the current logging infra.
Optimize the same and add few necessary logs.
Bug: 152385205
Change-Id: Ie261db317af48955a16269539948ff1596c4bbcb
CRs-Fixed: 2644418
-rw-r--r-- | core/hdd/src/wlan_hdd_nan_datapath.c | 12 | ||||
-rw-r--r-- | core/mac/src/pe/nan/nan_datapath.c | 10 |
2 files changed, 9 insertions, 13 deletions
diff --git a/core/hdd/src/wlan_hdd_nan_datapath.c b/core/hdd/src/wlan_hdd_nan_datapath.c index cf96523ff8..d3214371d3 100644 --- a/core/hdd/src/wlan_hdd_nan_datapath.c +++ b/core/hdd/src/wlan_hdd_nan_datapath.c @@ -320,16 +320,16 @@ void hdd_ndp_event_handler(struct hdd_adapter *adapter, case eCSR_ROAM_RESULT_NDI_CREATE_RSP: success = (roam_info->ndp.ndi_create_params.status == NAN_DATAPATH_RSP_STATUS_SUCCESS); - hdd_debug("posting ndi create status: %d to umac", - success); + hdd_debug("posting ndi create status: %d (%s) to umac", + success, success ? "Success" : "Failure"); os_if_nan_post_ndi_create_rsp(psoc, adapter->session_id, success); return; case eCSR_ROAM_RESULT_NDI_DELETE_RSP: success = (roam_info->ndp.ndi_create_params.status == NAN_DATAPATH_RSP_STATUS_SUCCESS); - hdd_debug("posting ndi delete status: %d to umac", - success); + hdd_debug("posting ndi delete status: %d (%s) to umac", + success, success ? "Success" : "Failure"); os_if_nan_post_ndi_delete_rsp(psoc, adapter->session_id, success); return; @@ -360,8 +360,6 @@ static int __wlan_hdd_cfg80211_process_ndp_cmd(struct wiphy *wiphy, int ret_val; struct hdd_context *hdd_ctx = wiphy_priv(wiphy); - hdd_enter(); - ret_val = wlan_hdd_validate_context(hdd_ctx); if (ret_val) return ret_val; @@ -829,8 +827,6 @@ void hdd_ndp_peer_departed_handler(uint8_t vdev_id, uint16_t sta_id, struct hdd_adapter *adapter; struct hdd_station_ctx *sta_ctx; - hdd_enter(); - hdd_ctx = cds_get_context(QDF_MODULE_ID_HDD); if (!hdd_ctx) { hdd_err("hdd_ctx is null"); diff --git a/core/mac/src/pe/nan/nan_datapath.c b/core/mac/src/pe/nan/nan_datapath.c index 02437d2dca..d53359bf2a 100644 --- a/core/mac/src/pe/nan/nan_datapath.c +++ b/core/mac/src/pe/nan/nan_datapath.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016-2018 The Linux Foundation. All rights reserved. + * Copyright (c) 2016-2018, 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 @@ -202,10 +202,10 @@ static void lim_ndp_delete_peers(tpAniSirGlobal mac_ctx, } for (i = 0; i < num_peers; i++) { - pe_info("ndp_map[%d]: MAC: " MAC_ADDRESS_STR " num_active %d", - i, - MAC_ADDR_ARRAY(ndp_map[i].peer_ndi_mac_addr.bytes), - ndp_map[i].num_active_ndp_sessions); + pe_debug("ndp_map[%d]: MAC: " MAC_ADDRESS_STR " num_active %d", + i, + MAC_ADDR_ARRAY(ndp_map[i].peer_ndi_mac_addr.bytes), + ndp_map[i].num_active_ndp_sessions); /* Do not delete a peer with active NDPs */ if (ndp_map[i].num_active_ndp_sessions > 0) |