From 19f0819fa32aec7c330b4d7ca962700230c8acb1 Mon Sep 17 00:00:00 2001 From: Amarnath Hullur Subramanyam Date: Thu, 10 Sep 2015 11:16:45 +0530 Subject: WiFi-HAL: Cleanup unnecessary logs Cleanup unnecessary logs from llstats, gscan, rssi monitor common and tdls modules. Change-Id: If66a8f61b11a101d84546a06514aeb5e0d8fcbfa Bug: 23942132 --- qcwcn/wifi_hal/wificonfig.cpp | 32 ++++++++++++-------------------- 1 file changed, 12 insertions(+), 20 deletions(-) (limited to 'qcwcn/wifi_hal/wificonfig.cpp') diff --git a/qcwcn/wifi_hal/wificonfig.cpp b/qcwcn/wifi_hal/wificonfig.cpp index 5c41369..e332895 100644 --- a/qcwcn/wifi_hal/wificonfig.cpp +++ b/qcwcn/wifi_hal/wificonfig.cpp @@ -46,7 +46,7 @@ wifi_error wifi_extended_dtim_config_set(wifi_request_id id, wifi_handle wifiHandle = getWifiHandle(iface); hal_info *info = getHalInfo(wifiHandle); - ALOGD("wifi_extended_dtim_config_set(): Enter"); + ALOGD("%s: extended_dtim:%d", __FUNCTION__, extended_dtim); wifiConfigCommand = new WiFiConfigCommand( wifiHandle, @@ -55,7 +55,7 @@ wifi_error wifi_extended_dtim_config_set(wifi_request_id id, QCA_NL80211_VENDOR_SUBCMD_SET_WIFI_CONFIGURATION); if (wifiConfigCommand == NULL) { - ALOGE("%s: Error wifiConfigCommand NULL", __func__); + ALOGE("%s: Error wifiConfigCommand NULL", __FUNCTION__); return WIFI_ERROR_UNKNOWN; } @@ -100,7 +100,6 @@ wifi_error wifi_extended_dtim_config_set(wifi_request_id id, } cleanup: - ALOGI("%s: Delete object.", __func__); delete wifiConfigCommand; return (wifi_error)ret; } @@ -116,7 +115,7 @@ wifi_error wifi_set_country_code(wifi_interface_handle iface, wifi_handle wifiHandle = getWifiHandle(iface); hal_info *info = getHalInfo(wifiHandle); - ALOGD("wifi_set_country_code(): %s", country_code); + ALOGD("%s: %s", __FUNCTION__, country_code); /* No request id from caller, so generate one and pass it on to the driver. * Generate it randomly. @@ -129,7 +128,7 @@ wifi_error wifi_set_country_code(wifi_interface_handle iface, OUI_QCA, QCA_NL80211_VENDOR_SUBCMD_SET_WIFI_CONFIGURATION); if (wifiConfigCommand == NULL) { - ALOGE("%s: Error wifiConfigCommand NULL", __func__); + ALOGE("%s: Error wifiConfigCommand NULL", __FUNCTION__); return WIFI_ERROR_UNKNOWN; } @@ -155,7 +154,6 @@ wifi_error wifi_set_country_code(wifi_interface_handle iface, usleep(WAIT_TIME_FOR_SET_REG_DOMAIN); cleanup: - ALOGV("%s: Delete object.", __func__); delete wifiConfigCommand; return (wifi_error)ret; } @@ -172,14 +170,14 @@ wifi_error wifi_set_beacon_wifi_iface_stats_averaging_factor( wifi_handle wifiHandle = getWifiHandle(iface); hal_info *info = getHalInfo(wifiHandle); - ALOGD("wifi_set_beacon_wifi_iface_stats_averaging_factor(): Enter"); + ALOGD("%s factor:%u", __FUNCTION__, factor); wifiConfigCommand = new WiFiConfigCommand( wifiHandle, id, OUI_QCA, QCA_NL80211_VENDOR_SUBCMD_SET_WIFI_CONFIGURATION); if (wifiConfigCommand == NULL) { - ALOGE("%s: Error wifiConfigCommand NULL", __func__); + ALOGE("%s: Error wifiConfigCommand NULL", __FUNCTION__); return WIFI_ERROR_UNKNOWN; } @@ -225,7 +223,6 @@ wifi_error wifi_set_beacon_wifi_iface_stats_averaging_factor( } cleanup: - ALOGI("%s: Delete object.", __func__); delete wifiConfigCommand; return (wifi_error)ret; } @@ -241,7 +238,7 @@ wifi_error wifi_set_guard_time(wifi_request_id id, wifi_handle wifiHandle = getWifiHandle(iface); hal_info *info = getHalInfo(wifiHandle); - ALOGD("wifi_set_guard_time(): Enter"); + ALOGD("%s : guard_time:%u", __FUNCTION__, guard_time); wifiConfigCommand = new WiFiConfigCommand( wifiHandle, @@ -249,7 +246,7 @@ wifi_error wifi_set_guard_time(wifi_request_id id, OUI_QCA, QCA_NL80211_VENDOR_SUBCMD_SET_WIFI_CONFIGURATION); if (wifiConfigCommand == NULL) { - ALOGE("%s: Error wifiConfigCommand NULL", __func__); + ALOGE("%s: Error wifiConfigCommand NULL", __FUNCTION__); return WIFI_ERROR_UNKNOWN; } @@ -291,7 +288,6 @@ wifi_error wifi_set_guard_time(wifi_request_id id, } cleanup: - ALOGI("%s: Delete object.", __func__); delete wifiConfigCommand; return (wifi_error)ret; } @@ -326,16 +322,12 @@ int WiFiConfigCommand::create() { ret = mMsg.put_u32(NL80211_ATTR_VENDOR_SUBCMD, mSubcmd); if (ret < 0) goto out; - - ALOGI("%s: mVendor_id = %d, Subcmd = %d.", - __func__, mVendor_id, mSubcmd); out: return ret; } /* This function implements creation of generic NL command */ int WiFiConfigCommand::create_generic(u8 cmdId) { - ALOGI("%s: cmdId = %d", __func__, cmdId); int ret = mMsg.create(cmdId, 0, 0); return ret; } @@ -354,7 +346,7 @@ static int error_handler_wifi_config(struct sockaddr_nl *nla, int *ret = (int *)arg; tmp = nla; *ret = err->error; - ALOGE("%s: Error code:%d (%s)", __func__, *ret, strerror(-(*ret))); + ALOGE("%s: Error code:%d (%s)", __FUNCTION__, *ret, strerror(-(*ret))); return NL_STOP; } @@ -393,7 +385,7 @@ int WiFiConfigCommand::requestEvent() cb = nl_cb_alloc(NL_CB_DEFAULT); if (!cb) { - ALOGE("%s: Callback allocation failed",__func__); + ALOGE("%s: Callback allocation failed",__FUNCTION__); res = -1; goto out; } @@ -421,10 +413,10 @@ int WiFiConfigCommand::requestEvent() res = mCondition.wait(abstime); if (res == ETIMEDOUT) { - ALOGE("%s: Time out happened.", __func__); + ALOGE("%s: Time out happened.", __FUNCTION__); } ALOGD("%s: Command invoked return value:%d, mWaitForRsp=%d", - __func__, res, mWaitforRsp); + __FUNCTION__, res, mWaitforRsp); } out: /* Cleanup the mMsg */ -- cgit v1.2.3