commit 1deb497aec7e41336aa76a5a8213de5dd68e6459
author himta ram <hram@codeaurora.org> Fri Aug 30 12:10:36 2019 +0530
committer himta ram <hram@codeaurora.org> Fri Aug 30 12:10:39 2019 +0530
tree ad0ae4777268af704dd7097212e56aad66357d59
parent 957e5f7cde893885a914a8d51f465d9712ad2988

FM: fm_hci: logs clean up

Convert unnecessary ALOGI logs to ALOGV
to reduce excessive logging in FM logs.

CRs-Fixed: 2520424
Change-Id: Ia76dacc3cf63363d7425a9617e338f8dc3f80023

fm_hci/fm_hci.cpp

diff --git a/fm_hci/fm_hci.cpp b/fm_hci/fm_hci.cpp
index 1686c5a..c4b421d 100644
--- a/fm_hci/fm_hci.cpp
+++ b/fm_hci/fm_hci.cpp
@@ -97,11 +97,11 @@
 static int enqueue_fm_rx_event(struct fm_event_header_t *hdr)
 {
 
-    ALOGI("%s: putting lock before enqueue ", __func__);
+    ALOGV("%s: putting lock before enqueue ", __func__);
     hci.rx_cond_mtx.lock();
     hci.rx_event_queue.push(hdr);
     hci.rx_cond_mtx.unlock();
-    ALOGI("%s:notify to waiting thred", __func__);
+    ALOGV("%s:notify to waiting thred", __func__);
     hci.rx_cond.notify_all();
     ALOGI("%s: FM-Event ENQUEUED SUCCESSFULLY", __func__);
 
@@ -140,11 +140,11 @@
             ALOGI("%s: FM_CMD_COMPLETE: current_credits %d, %d Credits got from the SOC", __func__, hci.command_credits, evt_buf->params[0]);
             if (hci.command_credits == 0) {
                 hci.command_credits += evt_buf->params[0];
-                ALOGI(" dequeue_fm_rx_event: wait for tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: wait for tx_cond_lock ");
                 hci.tx_cond_mtx.lock();
-                ALOGI(" dequeue_fm_rx_event: Notifying tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: Notifying tx_cond_lock ");
                 hci.tx_cond.notify_all();
-                ALOGI(" dequeue_fm_rx_event: UNLOCKING tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: UNLOCKING tx_cond_lock ");
                 hci.tx_cond_mtx.unlock();
             } else {
                 hci.command_credits += evt_buf->params[0];
@@ -154,11 +154,11 @@
             ALOGI("%s: FM_CMD_STATUS: current_credits %d, %d Credits got from the SOC", __func__, hci.command_credits, evt_buf->params[1]);
             if (hci.command_credits == 0) {
                 hci.command_credits += evt_buf->params[1];
-                ALOGI(" dequeue_fm_rx_event: wait for tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: wait for tx_cond_lock ");
                 hci.tx_cond_mtx.lock();
-                ALOGI(" dequeue_fm_rx_event: Notifying tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: Notifying tx_cond_lock ");
                 hci.tx_cond.notify_all();
-                ALOGI(" dequeue_fm_rx_event: UNLOCKING tx_cond_lock ");
+                ALOGV(" dequeue_fm_rx_event: UNLOCKING tx_cond_lock ");
                 hci.tx_cond_mtx.unlock();
             } else {
                 hci.command_credits += evt_buf->params[1];
@@ -203,11 +203,11 @@
 
     ALOGI("%s:  notifying credits %d", __func__, hci.command_credits);
     if (hci.command_credits > 0) {
-        ALOGI(" enqueue_fm_tx_cmd: wait for tx_cond_lock ");
+        ALOGV(" enqueue_fm_tx_cmd: wait for tx_cond_lock ");
         hci.tx_cond_mtx.lock();
-        ALOGI(" enqueue_fm_tx_cmd: Notifying tx_cond_lock ");
+        ALOGV(" enqueue_fm_tx_cmd: Notifying tx_cond_lock ");
         hci.tx_cond.notify_all();
-        ALOGI(" enqueue_fm_tx_cmd: UNLOCK tx_cond_lock ");
+        ALOGV(" enqueue_fm_tx_cmd: UNLOCK tx_cond_lock ");
         hci.tx_cond_mtx.unlock();
     }
 
@@ -233,16 +233,15 @@
 {
     fm_command_header_t *hdr;
 
-    ALOGI("%s", __func__);
+    ALOGI("%s command credits %d ", __func__, hci.command_credits);
 
     while (1) 
     {
-       ALOGI(" dequeue_fm_tx_cmd:  command credits %d ", hci.command_credits);
        if (hci.command_credits == 0) {
           return;
        }
         hci.tx_queue_mtx.lock();
-        ALOGI("%s is_que_empty %d", __func__,hci.tx_cmd_queue.empty());
+        ALOGV("%s is_que_empty %d", __func__,hci.tx_cmd_queue.empty());
         if(hci.tx_cmd_queue.empty()){
             ALOGI(" %s No more FM CMDs are available in the Queue",__func__);
             hci.tx_queue_mtx.unlock();
@@ -252,7 +251,7 @@
         hdr = hci.tx_cmd_queue.front();
         hci.tx_cmd_queue.pop();
         hci.tx_queue_mtx.unlock();
-        ALOGI("%s: packet popped %d credits", __func__,hci.command_credits);
+        ALOGV("%s: packet popped %d credits", __func__,hci.command_credits);
 
 
         hci.command_credits--;