From 5224e444a8afdd299d69fe5f591b2b74f3d3ecf3 Mon Sep 17 00:00:00 2001 From: Yun Park Date: Wed, 25 Jan 2017 12:27:29 -0800 Subject: qcacld-3.0: Remove unnecessary log from hdd_ipa_nbuf_cb Remove flooding log from hdd_ipa_nbuf_cb. Also fix INVALID REASON log from hdd_ipa_uc_op_cb even when hdd_ipa_uc_op_metering() is succeeded. Change-Id: Ided529cca3d84cbc3dd6b33dd31fa9086489a176 CRs-Fixed: 1116111 --- core/hdd/src/wlan_hdd_ipa.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/core/hdd/src/wlan_hdd_ipa.c b/core/hdd/src/wlan_hdd_ipa.c index 2c7b25eb4e2f..8f87a4d9343a 100644 --- a/core/hdd/src/wlan_hdd_ipa.c +++ b/core/hdd/src/wlan_hdd_ipa.c @@ -2313,17 +2313,15 @@ static void hdd_ipa_uc_op_cb(struct op_msg_type *op_msg, void *usr_ctxt) uc_fw_stat->rx_num_ind_drop_no_buf + uc_fw_stat->rx_num_pkts_indicated); qdf_mutex_release(&hdd_ipa->ipa_lock); - } else if (hdd_ipa_uc_op_metering(hdd_ctx, op_msg)) { - HDD_IPA_LOG(LOGE, "Invalid message: op_code=%d, reason=%d", - msg->op_code, hdd_ipa->stat_req_reason); } else if (msg->op_code == HDD_IPA_UC_OPCODE_UC_READY) { qdf_mutex_acquire(&hdd_ipa->ipa_lock); hdd_ipa_uc_loaded_handler(hdd_ipa); qdf_mutex_release(&hdd_ipa->ipa_lock); - } else { - HDD_IPA_LOG(LOGE, "INVALID REASON %d", - hdd_ipa->stat_req_reason); + } else if (hdd_ipa_uc_op_metering(hdd_ctx, op_msg)) { + HDD_IPA_LOG(LOGE, "Invalid message: op_code=%d, reason=%d", + msg->op_code, hdd_ipa->stat_req_reason); } + qdf_mem_free(op_msg); } @@ -4006,8 +4004,6 @@ void hdd_ipa_nbuf_cb(qdf_nbuf_t skb) struct hdd_ipa_tx_desc *tx_desc; uint16_t id; - HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "%x", QDF_NBUF_CB_TX_IPA_PRIV(skb)); - if (!qdf_nbuf_ipa_owned_get(skb)) { dev_kfree_skb_any(skb); return; -- cgit v1.2.3