Wifi-HAL: remove some calls to ALOGI

Originally part of
https://partner-android-review.googlesource.com/#/c/558740

BUG=27698661
TEST=compile

Change-Id: I6c8d3e09eedd9675439d6a5a5ad17eabc16416db

11 files changed, 0 insertions, 68 deletions

diff --git a/qcwcn/wifi_hal/common.cpp b/qcwcn/wifi_hal/common.cpp
index 1dc8436..91fa7a4 100644
--- a/qcwcn/wifi_hal/common.cpp
+++ b/qcwcn/wifi_hal/common.cpp
@@ -362,10 +362,6 @@ wifi_error fetchLowiCbTableAndCapabilities(lowi_cb_table_t **lowi_wifihal_api,
             retVal = WIFI_ERROR_NOT_SUPPORTED;
             goto cleanup;
         }
-        else {
-            ALOGI("%s: Version Check passed:%d", __FUNCTION__,
-                  versionCheck);
-        }
     }
     else {
         ALOGI("%s: lowi_wifihal_get_lowi_version not present",
@@ -439,12 +435,9 @@ lowi_cb_table_t *getLowiCallbackTable(u32 requested_lowi_capabilities)
             lowiSupportedCapabilities);
         return NULL;
     }
-    ALOGI("%s: Returning valid LowiWifiHalApi instance:%p",
-          __FUNCTION__, LowiWifiHalApi);
     return LowiWifiHalApi;
 
 cleanup:
-    ALOGI("%s: Cleaning up Lowi due to failure. Return NULL", __FUNCTION__);
     if (LowiWifiHalApi && LowiWifiHalApi->destroy) {
         ret = LowiWifiHalApi->destroy();
     }
diff --git a/qcwcn/wifi_hal/cpp_bindings.h b/qcwcn/wifi_hal/cpp_bindings.h
index a7b9a5a..093a040 100644
--- a/qcwcn/wifi_hal/cpp_bindings.h
+++ b/qcwcn/wifi_hal/cpp_bindings.h
@@ -349,13 +349,11 @@ protected:
 
     /* Override this method to parse reply and dig out data; save it in the object */
     virtual int handleResponse(WifiEvent& reply) {
-        ALOGI("skipping a response");
         return NL_SKIP;
     }
 
     /* Override this method to parse event and dig out data; save it in the object */
     virtual int handleEvent(WifiEvent& event) {
-        ALOGI("skipping an event");
         return NL_SKIP;
     }
 
diff --git a/qcwcn/wifi_hal/gscan.cpp b/qcwcn/wifi_hal/gscan.cpp
index 3cc5141..2aeba64 100644
--- a/qcwcn/wifi_hal/gscan.cpp
+++ b/qcwcn/wifi_hal/gscan.cpp
@@ -208,7 +208,6 @@ wifi_error wifi_get_gscan_capabilities(wifi_interface_handle handle,
      * Generate it randomly.
      */
     requestId = get_requestid();
-    ALOGI("%s: RequestId:%d", __FUNCTION__, requestId);
 
     if (capabilities == NULL) {
         ALOGE("%s: NULL capabilities pointer provided. Exit.",
@@ -532,7 +531,6 @@ wifi_error wifi_stop_gscan(wifi_request_id id,
         return (wifi_error)ret;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
     if (gScanStartCmdEventHandler == NULL ||
         gScanStartCmdEventHandler->isEventHandlingEnabled() == false) {
         ALOGE("%s: GSCAN isn't running or already stopped. "
@@ -627,8 +625,6 @@ wifi_error wifi_set_bssid_hotlist(wifi_request_id id,
         return (wifi_error)ret;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     /* Wi-Fi HAL doesn't need to check if a similar request to set bssid
      * hotlist was made earlier. If set_bssid_hotlist() is called while
      * another one is running, the request will be sent down to driver and
@@ -802,7 +798,6 @@ wifi_error wifi_reset_bssid_hotlist(wifi_request_id id,
         return (wifi_error)ret;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
 
     if (gScanSetBssidHotlistCmdEventHandler == NULL ||
         (gScanSetBssidHotlistCmdEventHandler->isEventHandlingEnabled() ==
@@ -902,8 +897,6 @@ wifi_error wifi_set_significant_change_handler(wifi_request_id id,
         return (wifi_error)ret;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     /* Wi-Fi HAL doesn't need to check if a similar request to set significant
      * change list was made earlier. If set_significant_change() is called while
      * another one is running, the request will be sent down to driver and
@@ -1089,8 +1082,6 @@ wifi_error wifi_reset_significant_change_handler(wifi_request_id id,
         return (wifi_error)ret;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     if (gScanSetSignificantChangeCmdEventHandler == NULL ||
         (gScanSetSignificantChangeCmdEventHandler->isEventHandlingEnabled() ==
         false)) {
@@ -1195,7 +1186,6 @@ wifi_error wifi_get_cached_gscan_results(wifi_interface_handle iface,
     /* No request id from caller, so generate one and pass it on to the driver. */
     /* Generate it randomly */
     requestId = get_requestid();
-    ALOGI("%s: RequestId:%d", __FUNCTION__, requestId);
 
     if (results == NULL || num == NULL) {
         ALOGE("%s: NULL pointer provided. Exit.",
@@ -2126,8 +2116,6 @@ wifi_error wifi_set_epno_list(wifi_request_id id,
         return WIFI_ERROR_NOT_SUPPORTED;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     /* Wi-Fi HAL doesn't need to check if a similar request to set ePNO
      * list was made earlier. If wifi_set_epno_list() is called while
      * another one is running, the request will be sent down to driver and
@@ -2387,8 +2375,6 @@ wifi_error wifi_set_passpoint_list(wifi_request_id id,
         return WIFI_ERROR_NOT_SUPPORTED;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     /* Wi-Fi HAL doesn't need to check if a similar request to set ePNO
      * passpoint list was made earlier. If wifi_set_passpoint_list() is called
      * while another one is running, the request will be sent down to driver and
@@ -2555,8 +2541,6 @@ wifi_error wifi_reset_passpoint_list(wifi_request_id id,
         return WIFI_ERROR_NOT_SUPPORTED;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     if (gScanPnoSetPasspointListCmdEventHandler == NULL ||
         (gScanPnoSetPasspointListCmdEventHandler->isEventHandlingEnabled() ==
          false)) {
@@ -2797,8 +2781,6 @@ wifi_error wifi_set_bssid_blacklist(wifi_request_id id,
         return WIFI_ERROR_NOT_SUPPORTED;
     }
 
-    ALOGI("%s: RequestId:%d", __FUNCTION__, id);
-
     for (i = 0; i < params.num_bssid; i++) {
         ALOGI("BSSID: %d : %02x:%02x:%02x:%02x:%02x:%02x", i,
                 params.bssids[i][0], params.bssids[i][1],
diff --git a/qcwcn/wifi_hal/gscan_event_handler.cpp b/qcwcn/wifi_hal/gscan_event_handler.cpp
index 019694e..1905d97 100644
--- a/qcwcn/wifi_hal/gscan_event_handler.cpp
+++ b/qcwcn/wifi_hal/gscan_event_handler.cpp
@@ -367,8 +367,6 @@ static wifi_error gscan_get_significant_change_results(u32 num_results,
     u32 len = 0;
     struct nlattr *scanResultsInfo;
 
-    ALOGI("gscan_get_significant_change_results: starting counter: %d", i);
-
     for (scanResultsInfo = (struct nlattr *) nla_data(tb_vendor[
             QCA_WLAN_VENDOR_ATTR_GSCAN_RESULTS_LIST]),
             rem = nla_len(tb_vendor[
@@ -400,10 +398,6 @@ static wifi_error gscan_get_significant_change_results(u32 num_results,
             tb2[
         QCA_WLAN_VENDOR_ATTR_GSCAN_RESULTS_SIGNIFICANT_CHANGE_RESULT_BSSID]),
             len);
-        ALOGI("\nsignificant_change_result:%d, BSSID:"
-            "%02x:%02x:%02x:%02x:%02x:%02x \n", i, results[i]->bssid[0],
-            results[i]->bssid[1], results[i]->bssid[2], results[i]->bssid[3],
-            results[i]->bssid[4], results[i]->bssid[5]);
 
         if (!
             tb2[
@@ -418,8 +412,6 @@ static wifi_error gscan_get_significant_change_results(u32 num_results,
             nla_get_u32(
             tb2[
         QCA_WLAN_VENDOR_ATTR_GSCAN_RESULTS_SIGNIFICANT_CHANGE_RESULT_CHANNEL]);
-        ALOGI("significant_change_result:%d, channel:%d.\n",
-            i, results[i]->channel);
 
         if (!
             tb2[
@@ -434,9 +426,6 @@ static wifi_error gscan_get_significant_change_results(u32 num_results,
             nla_get_u32(
             tb2[
         QCA_WLAN_VENDOR_ATTR_GSCAN_RESULTS_SIGNIFICANT_CHANGE_RESULT_NUM_RSSI]);
-        ALOGI("gscan_get_significant_change_results: "
-            "significant_change_result:%d, num_rssi:%d.\n",
-            i, results[i]->num_rssi);
 
         if (!
             tb2[
@@ -447,10 +436,6 @@ static wifi_error gscan_get_significant_change_results(u32 num_results,
                 "SIGNIFICANT_CHANGE_RESULT_RSSI_LIST not found");
             return WIFI_ERROR_INVALID_ARGS;
         }
-        ALOGI("gscan_get_significant_change_results: before reading the RSSI "
-            "list: num_rssi:%d, size_of_rssi:%zu, total size:%zu, ",
-            results[i]->num_rssi,
-            sizeof(wifi_rssi), results[i]->num_rssi * sizeof(wifi_rssi));
 
         memcpy(&(results[i]->rssi[0]),
             nla_data(
@@ -478,7 +463,6 @@ wifi_error GScanCommandEventHandler::gscan_parse_hotlist_ssid_results(
     struct nlattr *scanResultsInfo;
     int rem = 0;
     u32 len = 0;
-    ALOGI("gscan_parse_hotlist_ssid_results: starting counter: %d", i);
 
     for (scanResultsInfo = (struct nlattr *) nla_data(tb_vendor[
             QCA_WLAN_VENDOR_ATTR_GSCAN_RESULTS_LIST]),
@@ -614,7 +598,6 @@ wifi_error GScanCommandEventHandler::gscan_parse_passpoint_network_result(
     u32 resultsBufSize = 0;
     u32 len = 0;
     int rem = 0;
-    ALOGI("%s: Entering", __FUNCTION__);
 
     for (scanResultsInfo = (struct nlattr *) nla_data(tb_vendor[
             QCA_WLAN_VENDOR_ATTR_GSCAN_PNO_RESULTS_PASSPOINT_MATCH_RESULT_LIST]),
@@ -858,8 +841,6 @@ wifi_error GScanCommandEventHandler::gscan_parse_passpoint_network_result(
             nla_get_u32(
                 tb2[
                 QCA_WLAN_VENDOR_ATTR_GSCAN_PNO_RESULTS_PASSPOINT_MATCH_ANQP_LEN]);
-        ALOGI("%s: PNO_RESULTS_PASSPOINT_MATCH_ANQP_LEN = %d",
-              __FUNCTION__, mPasspointAnqpLen);
 
         if (!mPasspointAnqpLen)
         {
diff --git a/qcwcn/wifi_hal/ifaceeventhandler.cpp b/qcwcn/wifi_hal/ifaceeventhandler.cpp
index 3135717..c949b80 100755
--- a/qcwcn/wifi_hal/ifaceeventhandler.cpp
+++ b/qcwcn/wifi_hal/ifaceeventhandler.cpp
@@ -188,7 +188,6 @@ wifiEventHandler::~wifiEventHandler()
 
 int wifiEventHandler::handleEvent(WifiEvent &event)
 {
-    ALOGI("wifiEventHandler::handleEvent");
     struct genlmsghdr *gnlh = event.header();
     mSubcmd = gnlh->cmd;
     nla_parse(tb, NL80211_ATTR_MAX, genlmsg_attrdata(gnlh, 0),
diff --git a/qcwcn/wifi_hal/nan.cpp b/qcwcn/wifi_hal/nan.cpp
index 2014f45..63da00d 100644
--- a/qcwcn/wifi_hal/nan.cpp
+++ b/qcwcn/wifi_hal/nan.cpp
@@ -653,7 +653,6 @@ NanCommand::~NanCommand()
 }
 
 int NanCommand::handleResponse(WifiEvent &reply){
-    ALOGI("skipping a response");
     return NL_SKIP;
 }
 
@@ -697,7 +696,6 @@ out:
 //Call the appropriate callback handler after parsing the vendor data.
 int NanCommand::handleEvent(WifiEvent &event)
 {
-    ALOGI("Got a NAN message from Driver");
     WifiVendorCommand::handleEvent(event);
     ALOGD("%s: Subcmd=%u Vendor data len received:%d",
           __FUNCTION__, mSubcmd, mDataLen);
diff --git a/qcwcn/wifi_hal/nan_ind.cpp b/qcwcn/wifi_hal/nan_ind.cpp
index c420c8b..84a535c 100644
--- a/qcwcn/wifi_hal/nan_ind.cpp
+++ b/qcwcn/wifi_hal/nan_ind.cpp
@@ -32,7 +32,6 @@ int NanCommand::handleNanIndication()
     u16 msg_id;
     int res = 0;
 
-    ALOGI("handleNanIndication called %p", this);
     msg_id = getIndicationType();
 
     ALOGI("handleNanIndication msg_id:%u", msg_id);
diff --git a/qcwcn/wifi_hal/nan_req.cpp b/qcwcn/wifi_hal/nan_req.cpp
index c4e43dc..4893a92 100644
--- a/qcwcn/wifi_hal/nan_req.cpp
+++ b/qcwcn/wifi_hal/nan_req.cpp
@@ -1173,7 +1173,6 @@ int NanCommand::calcNanFurtherAvailabilityMapSize(
 
 int NanCommand::putNanCapabilities(transaction_id id)
 {
-    ALOGI("NAN_DISABLE");
     size_t message_len = sizeof(NanCapabilitiesReqMsg);
 
     pNanCapabilitiesReqMsg pFwReq = (pNanCapabilitiesReqMsg)malloc(message_len);
@@ -1182,7 +1181,6 @@ int NanCommand::putNanCapabilities(transaction_id id)
         return WIFI_ERROR_OUT_OF_MEMORY;
     }
 
-    ALOGI("Message Len %zu", message_len);
     memset (pFwReq, 0, message_len);
     pFwReq->fwHeader.msgVersion = (u16)NAN_MSG_VERSION1;
     pFwReq->fwHeader.msgId = NAN_MSG_ID_CAPABILITIES_REQ;
diff --git a/qcwcn/wifi_hal/rssi_monitor.cpp b/qcwcn/wifi_hal/rssi_monitor.cpp
index 06bb835..b65b78a 100644
--- a/qcwcn/wifi_hal/rssi_monitor.cpp
+++ b/qcwcn/wifi_hal/rssi_monitor.cpp
@@ -111,8 +111,6 @@ int RSSIMonitorCommand::handleEvent(WifiEvent &event)
 
             memset(addr, 0, sizeof(mac_addr));
 
-            ALOGI("QCA_NL80211_VENDOR_SUBCMD_MONITOR_RSSI Received");
-
             if (!tb_vendor[
                 QCA_WLAN_VENDOR_ATTR_RSSI_MONITORING_REQUEST_ID])
             {
diff --git a/qcwcn/wifi_hal/tdls.cpp b/qcwcn/wifi_hal/tdls.cpp
index f7eafad..89dea7a 100755
--- a/qcwcn/wifi_hal/tdls.cpp
+++ b/qcwcn/wifi_hal/tdls.cpp
@@ -367,7 +367,6 @@ wifi_error wifi_enable_tdls(wifi_interface_handle iface,
     wifi_handle handle = getWifiHandle(iface);
     pTdlsCommand = TdlsCommand::instance(handle);
 
-    ALOGI("%s: Enter", __FUNCTION__);
     if (pTdlsCommand == NULL) {
         ALOGE("%s: Error TdlsCommand NULL", __FUNCTION__);
         return WIFI_ERROR_UNKNOWN;
@@ -427,7 +426,6 @@ wifi_error wifi_enable_tdls(wifi_interface_handle iface,
     }
 
 cleanup:
-    ALOGI("%s: Exit", __FUNCTION__);
     return (wifi_error)ret;
 }
 
@@ -448,7 +446,6 @@ wifi_error wifi_disable_tdls(wifi_interface_handle iface, mac_addr addr)
     wifi_handle handle = getWifiHandle(iface);
     pTdlsCommand = TdlsCommand::instance(handle);
 
-    ALOGI("%s: Enter", __FUNCTION__);
     if (pTdlsCommand == NULL) {
         ALOGE("%s: Error TdlsCommand NULL", __FUNCTION__);
         return WIFI_ERROR_UNKNOWN;
@@ -482,7 +479,6 @@ wifi_error wifi_disable_tdls(wifi_interface_handle iface, mac_addr addr)
     }
 
 cleanup:
-    ALOGI("%s: Exit", __FUNCTION__);
     delete pTdlsCommand;
     return (wifi_error)ret;
 }
@@ -500,7 +496,6 @@ wifi_error wifi_get_tdls_status(wifi_interface_handle iface, mac_addr addr,
     wifi_handle handle = getWifiHandle(iface);
     pTdlsCommand = TdlsCommand::instance(handle);
 
-    ALOGI("%s: Enter", __FUNCTION__);
     if (pTdlsCommand == NULL) {
         ALOGE("%s: Error TdlsCommand NULL", __FUNCTION__);
         return WIFI_ERROR_UNKNOWN;
@@ -534,7 +529,6 @@ wifi_error wifi_get_tdls_status(wifi_interface_handle iface, mac_addr addr,
     pTdlsCommand->getStatusRspParams(status);
 
 cleanup:
-    ALOGI("%s: Exit", __FUNCTION__);
     return (wifi_error)ret;
 }
 
@@ -544,7 +538,6 @@ wifi_error wifi_get_tdls_capabilities(wifi_interface_handle iface,
 {
     int ret = 0;
     TdlsCommand *pTdlsCommand;
-    ALOGI("%s: Enter", __FUNCTION__);
 
     if (capabilities == NULL) {
         ALOGE("%s: capabilities is NULL", __FUNCTION__);
@@ -578,7 +571,6 @@ wifi_error wifi_get_tdls_capabilities(wifi_interface_handle iface,
     pTdlsCommand->getCapsRspParams(capabilities);
 
 cleanup:
-    ALOGI("%s: Exit", __FUNCTION__);
     if (ret < 0)
         memset(capabilities, 0, sizeof(wifi_tdls_capabilities));
     delete pTdlsCommand;
diff --git a/qcwcn/wifi_hal/wifi_hal.cpp b/qcwcn/wifi_hal/wifi_hal.cpp
index 3541d87..4e90646 100644
--- a/qcwcn/wifi_hal/wifi_hal.cpp
+++ b/qcwcn/wifi_hal/wifi_hal.cpp
@@ -635,7 +635,6 @@ static void internal_cleaned_up_handler(wifi_handle handle)
     if (info->rx_aggr_pkts)
         free(info->rx_aggr_pkts);
     wifi_logger_ring_buffers_deinit(info);
-    ALOGI("Cleanup Gscan Event Handlers");
     cleanupGscanHandlers(info);
 
     if (info->exit_sockets[0] >= 0) {
@@ -657,8 +656,6 @@ static void internal_cleaned_up_handler(wifi_handle handle)
     pthread_mutex_destroy(&info->cb_lock);
     pthread_mutex_destroy(&info->pkt_fate_stats_lock);
     free(info);
-
-    ALOGI("Internal cleanup completed");
 }
 
 void wifi_cleanup(wifi_handle handle, wifi_cleaned_up_handler handler)
@@ -746,9 +743,6 @@ void wifi_event_loop(wifi_handle handle)
         }
         rb_timerhandler(info);
     } while (!info->clean_up);
-
-
-    ALOGI("Cleaning up");
     internal_cleaned_up_handler(handle);
 }