From: Mohit Ghuley <mohit.ghuley@samsung.com>
Date: Mon, 7 Aug 2017 10:08:29 +0000 (+0530)
Subject: [7885][7572] wlbt : Removal of unneccesary logs from Wifi HAL Module.
X-Git-Tag: MMI-QSBS30.62-17-8~52
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=9a2ccc1cccf478202e078cebb28d5db97acdcfdc;p=GitHub%2FMotorolaMobilityLLC%2Fhardware-samsung_slsi-scsc_wifibt-wifi_hal.git

[7885][7572] wlbt : Removal of unneccesary logs from Wifi HAL Module.

Logs Updaton and removal of unnecessary logs from
Wifi HAL Module.

Wind Glory CQ Fix : GLORY00115554

Change-Id: I73f3a7ed101b033e83aa1834cc279925ce29a5ba
SCSC-Bug-Id:SSB-31296
Signed-off-by: Mohit Ghuley <mohit.ghuley@samsung.com>
---

diff --git a/cpp_bindings.cpp b/cpp_bindings.cpp
index fba1ca2..e815d05 100755
--- a/cpp_bindings.cpp
+++ b/cpp_bindings.cpp
@@ -492,8 +492,6 @@ void WifiEvent::log() {
             ALOGD("found attribute %s", attributeToString(i));
         }
     }
-
-    ALOGD("-- End of message --");
 }
 
 const char *WifiEvent::get_cmdString() {
@@ -647,7 +645,6 @@ out:
 
 /* Event handlers */
 int WifiCommand::response_handler(struct nl_msg *msg, void *arg) {
-    ALOGD("response_handler called");
     WifiCommand *cmd = (WifiCommand *)arg;
     WifiEvent reply(msg);
     int res = reply.parse();
@@ -677,21 +674,18 @@ int WifiCommand::event_handler(struct nl_msg *msg, void *arg) {
 
 /* Other event handlers */
 int WifiCommand::valid_handler(struct nl_msg *msg, void *arg) {
-    ALOGD("valid_handler called");
     int *err = (int *)arg;
     *err = 0;
     return NL_SKIP;
 }
 
 int WifiCommand::ack_handler(struct nl_msg *msg, void *arg) {
-    ALOGD("ack_handler called");
     int *err = (int *)arg;
     *err = 0;
     return NL_STOP;
 }
 
 int WifiCommand::finish_handler(struct nl_msg *msg, void *arg) {
-    ALOGD("finish_handler called");
     int *ret = (int *)arg;
     *ret = 0;
     return NL_SKIP;
diff --git a/gscan.cpp b/gscan.cpp
index b247725..b6c104c 100755
--- a/gscan.cpp
+++ b/gscan.cpp
@@ -134,7 +134,7 @@ public:
 
         int ret = mMsg.create(GOOGLE_OUI, SLSI_NL80211_VENDOR_SUBCMD_GET_CAPABILITIES);
         if (ret < 0) {
-            ALOGD("NL message creation failed");
+            ALOGE("NL message creation failed");
             return ret;
         }
 
@@ -144,8 +144,6 @@ public:
 protected:
     virtual int handleResponse(WifiEvent& reply) {
 
-        ALOGD("In GetCapabilities::handleResponse");
-
         if (reply.get_cmd() != NL80211_CMD_VENDOR) {
             ALOGD("Ignoring reply with cmd = %d", reply.get_cmd());
             return NL_SKIP;
@@ -157,7 +155,7 @@ protected:
         void *data = reply.get_vendor_data();
         int len = reply.get_vendor_data_len();
 
-        ALOGD("Id = %0x, subcmd = %d, len = %d, expected len = %d", id, subcmd, len,
+        ALOGD("GetCapabilities::handleResponse - Id = %0x, subcmd = %d, len = %d, expected len = %d", id, subcmd, len,
                     sizeof(*mCapabilities));
 
         memcpy(mCapabilities, data, min(len, (int) sizeof(*mCapabilities)));
@@ -210,8 +208,6 @@ public:
 protected:
     virtual int handleResponse(WifiEvent& reply) {
 
-        ALOGD("In GetChannelList::handleResponse");
-
         if (reply.get_cmd() != NL80211_CMD_VENDOR) {
             ALOGD("Ignoring reply with cmd = %d", reply.get_cmd());
             return NL_SKIP;
@@ -224,7 +220,7 @@ protected:
         nlattr *vendor_data = reply.get_attribute(NL80211_ATTR_VENDOR_DATA);
         int len = reply.get_vendor_data_len();
 
-        ALOGD("Id = %0x, subcmd = %d, len = %d", id, subcmd, len);
+        ALOGD("GetChannelList::handleResponse - Id = %0x, subcmd = %d, len = %d", id, subcmd, len);
         if (vendor_data == NULL || len == 0) {
             ALOGE("no vendor data in GetChannelList response; ignoring it");
             return NL_SKIP;
@@ -502,7 +498,7 @@ public:
 
         if(event_id == GSCAN_EVENT_COMPLETE_SCAN) {
             if (vendor_data == NULL || len != 4) {
-                ALOGD("Scan complete type not mentioned!");
+                ALOGE("Scan complete type not mentioned!");
                 return NL_SKIP;
             }
             wifi_scan_event evt_type;
@@ -652,7 +648,6 @@ public:
     }
 
     virtual int handleResponse(WifiEvent& reply) {
-        ALOGD("In GetScanResultsCommand::handleResponse");
 
         if (reply.get_cmd() != NL80211_CMD_VENDOR) {
             ALOGD("Ignoring reply with cmd = %d", reply.get_cmd());
@@ -662,7 +657,7 @@ public:
         int id = reply.get_vendor_id();
         int subcmd = reply.get_vendor_subcmd();
 
-        ALOGD("Id = %0x, subcmd = %d", id, subcmd);
+        ALOGD("GetScanResultsCommand::handleResponse - Id = %0x, subcmd = %d", id, subcmd);
 
         nlattr *vendor_data = reply.get_attribute(NL80211_ATTR_VENDOR_DATA);
         int len = reply.get_vendor_data_len();
@@ -859,7 +854,7 @@ public:
         int len = event.get_vendor_data_len();
 
         if (vendor_data == NULL || len == 0) {
-            ALOGD("No scan results found");
+            ALOGE("No scan results found");
             return NL_SKIP;
         }
 
@@ -1041,7 +1036,7 @@ public:
         int len = event.get_vendor_data_len();
 
         if (vendor_data == NULL || len == 0) {
-            ALOGD("No scan results found");
+            ALOGE("No scan results found");
             return NL_SKIP;
         }