qcacld-3.0: Reduce the log spam in wlan_hdd_debugfs.c

Move the logs to appropriate log levels to reduce
the log spam in wlan_hdd_debugfs.c

Change-Id: I03b0bbaaff808ac6597ae3d05b4da090feeb7220
CRs-Fixed: 2014745

1 files changed, 11 insertions, 20 deletions

diff --git a/core/hdd/src/wlan_hdd_debugfs.c b/core/hdd/src/wlan_hdd_debugfs.c
index 1ad8cc01c8cb..53ee4d8a72cf 100644
--- a/core/hdd/src/wlan_hdd_debugfs.c
+++ b/core/hdd/src/wlan_hdd_debugfs.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2013-2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2013-2017 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -74,8 +74,7 @@ static ssize_t __wcnss_wowenable_write(struct file *file,
 
 	pAdapter = (hdd_adapter_t *)file->private_data;
 	if ((NULL == pAdapter) || (WLAN_HDD_ADAPTER_MAGIC != pAdapter->magic)) {
-		hdd_alert("Invalid adapter or adapter has invalid magic.");
-
+		hdd_err("Invalid adapter or adapter has invalid magic");
 		return -EINVAL;
 	}
 
@@ -87,14 +86,12 @@ static ssize_t __wcnss_wowenable_write(struct file *file,
 
 	if (!sme_is_feature_supported_by_fw(WOW)) {
 		hdd_err("Wake-on-Wireless feature is not supported in firmware!");
-
 		return -EINVAL;
 	}
 
 	if (count > MAX_USER_COMMAND_SIZE_WOWL_ENABLE) {
-		hdd_err("Command length is larger than %d bytes.",
+		hdd_err("Command length is larger than %d bytes",
 			MAX_USER_COMMAND_SIZE_WOWL_ENABLE);
-
 		return -EINVAL;
 	}
 
@@ -115,7 +112,6 @@ static ssize_t __wcnss_wowenable_write(struct file *file,
 	if (!wow_enable) {
 		if (!hdd_exit_wowl(pAdapter)) {
 			hdd_err("hdd_exit_wowl failed!");
-
 			return -EFAULT;
 		}
 
@@ -142,7 +138,6 @@ static ssize_t __wcnss_wowenable_write(struct file *file,
 
 	if (!hdd_enter_wowl(pAdapter, wow_mp, wow_pbm)) {
 		hdd_err("hdd_enter_wowl failed!");
-
 		return -EFAULT;
 	}
 	EXIT();
@@ -197,8 +192,7 @@ static ssize_t __wcnss_wowpattern_write(struct file *file,
 	ENTER();
 
 	if ((NULL == pAdapter) || (WLAN_HDD_ADAPTER_MAGIC != pAdapter->magic)) {
-		hdd_alert("Invalid adapter or adapter has invalid magic.");
-
+		hdd_err("Invalid adapter or adapter has invalid magic");
 		return -EINVAL;
 	}
 
@@ -209,14 +203,12 @@ static ssize_t __wcnss_wowpattern_write(struct file *file,
 
 	if (!sme_is_feature_supported_by_fw(WOW)) {
 		hdd_err("Wake-on-Wireless feature is not supported in firmware!");
-
 		return -EINVAL;
 	}
 
 	if (count > MAX_USER_COMMAND_SIZE_WOWL_PATTERN) {
-		hdd_err("Command length is larger than %d bytes.",
+		hdd_err("Command length is larger than %d bytes",
 			MAX_USER_COMMAND_SIZE_WOWL_PATTERN);
-
 		return -EINVAL;
 	}
 
@@ -321,8 +313,7 @@ static ssize_t __wcnss_patterngen_write(struct file *file,
 
 	pAdapter = (hdd_adapter_t *)file->private_data;
 	if ((NULL == pAdapter) || (WLAN_HDD_ADAPTER_MAGIC != pAdapter->magic)) {
-		hdd_alert("Invalid adapter or adapter has invalid magic.");
-
+		hdd_err("Invalid adapter or adapter has invalid magic");
 		return -EINVAL;
 	}
 
@@ -340,7 +331,7 @@ static ssize_t __wcnss_patterngen_write(struct file *file,
 	if (count <= MAX_USER_COMMAND_SIZE_FRAME)
 		cmd = qdf_mem_malloc(count + 1);
 	else {
-		hdd_err("Command length is larger than %d bytes.",
+		hdd_err("Command length is larger than %d bytes",
 			MAX_USER_COMMAND_SIZE_FRAME);
 
 		return -EINVAL;
@@ -366,7 +357,7 @@ static ssize_t __wcnss_patterngen_write(struct file *file,
 		goto failure;
 
 	if (pattern_idx > (MAXNUM_PERIODIC_TX_PTRNS - 1)) {
-		hdd_err("Pattern index %d is not in the range (0 ~ %d).",
+		hdd_err("Pattern index: %d is not in the range (0 ~ %d)",
 			pattern_idx, MAXNUM_PERIODIC_TX_PTRNS - 1);
 
 		goto failure;
@@ -412,7 +403,7 @@ static ssize_t __wcnss_patterngen_write(struct file *file,
 	 * In STA mode check if it's in connected state before adding
 	 * patterns
 	 */
-	hdd_info("device mode %d", pAdapter->device_mode);
+	hdd_debug("device mode %d", pAdapter->device_mode);
 	if ((QDF_STA_MODE == pAdapter->device_mode) &&
 	    (!hdd_conn_is_connected(WLAN_HDD_GET_STATION_CTX_PTR(pAdapter)))) {
 			hdd_err("Not in Connected state!");
@@ -762,7 +753,7 @@ static int __wcnss_debugfs_open(struct inode *inode, struct file *file)
 
 	adapter = (hdd_adapter_t *)file->private_data;
 	if ((NULL == adapter) || (WLAN_HDD_ADAPTER_MAGIC != adapter->magic)) {
-		hdd_alert("Invalid adapter or adapter has invalid magic.");
+		hdd_err("Invalid adapter or adapter has invalid magic");
 		return -EINVAL;
 	}
 
@@ -900,6 +891,6 @@ void hdd_debugfs_exit(hdd_adapter_t *adapter)
 	if (adapter->debugfs_phy)
 		debugfs_remove_recursive(adapter->debugfs_phy);
 	else
-		hdd_info("Interface %s has no debugfs entry", dev->name);
+		hdd_debug("Interface %s has no debugfs entry", dev->name);
 }
 #endif /* #ifdef WLAN_OPEN_SOURCE */