diff options
| author | Yun Park <yunp@codeaurora.org> | 2017-01-25 12:27:29 -0800 |
|---|---|---|
| committer | qcabuildsw <qcabuildsw@localhost> | 2017-01-28 04:49:58 -0800 |
| commit | 5224e444a8afdd299d69fe5f591b2b74f3d3ecf3 (patch) | |
| tree | 7a7e9c2bc9498e0f3f145a74b00b5abdad447b6d | |
| parent | 7384c1639a61347c442dd2f51416749433a49b9a (diff) | |
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
| -rw-r--r-- | core/hdd/src/wlan_hdd_ipa.c | 12 |
1 files 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; |