qcacld-3.0: wlan: Cleanup North-Bound load/unload logs
qcacld-2.0 to qcacld-3.0 propagation
If IOCTLs or CFG callbacks are issued continuously during driver
load/unload or during SSR, observing stability issues because of
excessive logging.
Hence move log level to INFO from ERROR in wlan_hdd_validate_context
function and don't print return status in its caller functions.
Change-Id: I8fd9b25e0c774e1e12b01128eb86783c9dec2ab7
CRs-Fixed: 800631
diff --git a/core/hdd/src/wlan_hdd_ioctl.c b/core/hdd/src/wlan_hdd_ioctl.c
index 1c8f70a..8c8ecf4 100644
--- a/core/hdd/src/wlan_hdd_ioctl.c
+++ b/core/hdd/src/wlan_hdd_ioctl.c
@@ -6134,6 +6134,8 @@
uint8_t *command = NULL;
int ret = 0;
+ ENTER();
+
if (CDF_FTM_MODE == hdd_get_conparam()) {
hddLog(LOGE, FL("Command not allowed in FTM mode"));
return -EINVAL;
@@ -6176,7 +6178,7 @@
exit:
if (command)
kfree(command);
-
+ EXIT();
return ret;
}
@@ -6256,6 +6258,8 @@
hdd_context_t *hdd_ctx;
int ret;
+ ENTER();
+
if (dev != adapter->dev) {
CDF_TRACE(CDF_MODULE_ID_HDD, CDF_TRACE_LEVEL_FATAL,
"%s: HDD adapter/dev inconsistency", __func__);
@@ -6280,12 +6284,8 @@
hdd_ctx = WLAN_HDD_GET_CTX(adapter);
ret = wlan_hdd_validate_context(hdd_ctx);
- if (ret) {
- CDF_TRACE(CDF_MODULE_ID_HDD, CDF_TRACE_LEVEL_ERROR,
- "%s: invalid context", __func__);
- ret = -EBUSY;
+ if (ret)
goto exit;
- }
switch (cmd) {
case (SIOCDEVPRIVATE + 1):
@@ -6301,6 +6301,7 @@
break;
}
exit:
+ EXIT();
return ret;
}