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

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

Change-Id: I0210c57968d26819de5e15bbe00193cd3e65703a
CRs-Fixed: 2014745

1 files changed, 53 insertions, 53 deletions

diff --git a/core/hdd/src/wlan_hdd_scan.c b/core/hdd/src/wlan_hdd_scan.c
index 5510b27fed23..2fd759c6d3a4 100644
--- a/core/hdd/src/wlan_hdd_scan.c
+++ b/core/hdd/src/wlan_hdd_scan.c
@@ -215,7 +215,7 @@ static int hdd_indicate_scan_result(hdd_scan_info_t *scanInfo,
 	char custom[MAX_CUSTOM_LEN];
 	char *p;
 
-	hdd_notice(MAC_ADDRESS_STR, MAC_ADDR_ARRAY(descriptor->bssId));
+	hdd_debug(MAC_ADDRESS_STR, MAC_ADDR_ARRAY(descriptor->bssId));
 
 	error = 0;
 	last_event = current_event;
@@ -256,7 +256,7 @@ static int hdd_indicate_scan_result(hdd_scan_info_t *scanInfo,
 		modestr = "n";
 		break;
 	default:
-		hdd_warn("Unknown network type [%d]", descriptor->nwType);
+		hdd_warn("Unknown network type: %d", descriptor->nwType);
 		modestr = "?";
 		break;
 	}
@@ -406,7 +406,7 @@ static int hdd_indicate_scan_result(hdd_scan_info_t *scanInfo,
 
 			if ((maxNumRates - numBasicRates) > MAX_RATES) {
 				no_of_rates = MAX_RATES;
-				hdd_warn("Accessing array out of bound that array is pDot11ExtSuppRates->rates ");
+				hdd_debug("Limit rates to MAX_RATES");
 			} else {
 				no_of_rates = maxNumRates - numBasicRates;
 			}
@@ -529,7 +529,7 @@ static bool wlan_hdd_is_scan_pending(hdd_adapter_t *adapter)
 		/* Any scan pending on the adapter */
 		if (adapter == hdd_scan_req->adapter) {
 			qdf_spin_unlock(&hdd_ctx->hdd_scan_req_q_lock);
-			hdd_info("pending scan id %d", hdd_scan_req->scan_id);
+			hdd_debug("pending scan id %d", hdd_scan_req->scan_id);
 			return true;
 		}
 	} while (QDF_STATUS_SUCCESS ==
@@ -561,7 +561,7 @@ static int wlan_hdd_scan_request_enqueue(hdd_adapter_t *adapter,
 	ENTER();
 	hdd_scan_req = qdf_mem_malloc(sizeof(*hdd_scan_req));
 	if (NULL == hdd_scan_req) {
-		hdd_alert("malloc failed for Scan req");
+		hdd_err("malloc failed for Scan req");
 		return -ENOMEM;
 	}
 
@@ -603,7 +603,7 @@ static QDF_STATUS wlan_hdd_scan_request_dequeue(hdd_context_t *hdd_ctx,
 	struct hdd_scan_req *hdd_scan_req;
 	qdf_list_node_t *pNode = NULL, *ppNode = NULL;
 
-	hdd_info("Dequeue Scan id: %d", scan_id);
+	hdd_debug("Dequeue Scan id: %d", scan_id);
 
 	if ((source == NULL) || (timestamp == NULL) || (req == NULL))
 		return QDF_STATUS_E_NULL_VALUE;
@@ -634,7 +634,7 @@ static QDF_STATUS wlan_hdd_scan_request_dequeue(hdd_context_t *hdd_ctx,
 				*timestamp = hdd_scan_req->timestamp;
 				qdf_mem_free(hdd_scan_req);
 				qdf_spin_unlock(&hdd_ctx->hdd_scan_req_q_lock);
-				hdd_info("removed Scan id: %d, req = %p, pending scans %d",
+				hdd_debug("removed Scan id: %d, req = %p, pending scans %d",
 				      scan_id, req,
 				      qdf_list_size(&hdd_ctx->hdd_scan_req_q));
 				return QDF_STATUS_SUCCESS;
@@ -684,7 +684,7 @@ hdd_scan_request_callback(tHalHandle halHandle, void *pContext,
 	uint32_t timestamp;
 
 	ENTER();
-	hdd_warn("called with halHandle = %p, pContext = %p, scanID = %d, returned status = %d",
+	hdd_debug("called with halHandle = %p, pContext = %p, scanID = %d, returned status = %d",
 		 halHandle, pContext, (int)scanId, (int)status);
 
 	/* if there is a scan request pending when the wlan driver is unloaded
@@ -693,7 +693,7 @@ hdd_scan_request_callback(tHalHandle halHandle, void *pContext,
 	 * do some quick sanity before proceeding
 	 */
 	if (pAdapter->dev != dev) {
-		hdd_warn("device mismatch %p vs %p", pAdapter->dev, dev);
+		hdd_debug("device mismatch %p vs %p", pAdapter->dev, dev);
 		return QDF_STATUS_SUCCESS;
 	}
 
@@ -799,7 +799,7 @@ static int __iw_set_scan(struct net_device *dev, struct iw_request_info *info,
 					QDF_ASSERT(0);
 				}
 			} else {
-				hdd_err("Invalid essid length : %d",
+				hdd_err("Invalid essid length: %d",
 					scanReq->essid_len);
 			}
 		}
@@ -863,7 +863,7 @@ static int __iw_set_scan(struct net_device *dev, struct iw_request_info *info,
 				  pAdapter->sessionId, &scanRequest,
 				  &hdd_scan_request_callback, dev);
 	if (!QDF_IS_STATUS_SUCCESS(status)) {
-		hdd_alert("sme_scan_request  fail %d!!!", status);
+		hdd_err("sme_scan_request  fail %d!!!", status);
 		goto error;
 	}
 
@@ -933,11 +933,11 @@ static int __iw_get_scan(struct net_device *dev,
 	if (0 != ret)
 		return ret;
 
-	hdd_notice("enter buffer length %d!!!",
+	hdd_debug("enter buffer length %d!!!",
 		  (wrqu->data.length) ? wrqu->data.length : IW_SCAN_MAX_DATA);
 
 	if (true == pAdapter->scan_info.mScanPending) {
-		hdd_alert("mScanPending is true !!!");
+		hdd_err("mScanPending is true !!!");
 		return -EAGAIN;
 	}
 
@@ -955,7 +955,7 @@ static int __iw_get_scan(struct net_device *dev,
 
 	if (NULL == pResult) {
 		/* no scan results */
-		hdd_notice("iw_get_scan: NULL Scan Result ");
+		hdd_debug("iw_get_scan: NULL Scan Result ");
 		return 0;
 	}
 
@@ -972,7 +972,7 @@ static int __iw_get_scan(struct net_device *dev,
 
 	sme_scan_result_purge(hHal, pResult);
 
-	hdd_notice("exit total %d BSS reported !!!", i);
+	hdd_debug("exit total %d BSS reported !!!", i);
 	EXIT();
 	return status;
 }
@@ -1194,7 +1194,7 @@ static QDF_STATUS hdd_cfg80211_scan_done_callback(tHalHandle halHandle,
 	if (0 != ret)
 		return QDF_STATUS_E_INVAL;
 
-	hdd_notice("called with hal = %p, pContext = %p, ID = %d, status = %d",
+	hdd_debug("called with hal = %p, pContext = %p, ID = %d, status = %d",
 		   halHandle, pContext, (int)scanId, (int)status);
 
 	pScanInfo->mScanPendingCounter = 0;
@@ -1220,7 +1220,7 @@ static QDF_STATUS hdd_cfg80211_scan_done_callback(tHalHandle halHandle,
 	}
 
 	if (!aborted && !hddctx->beacon_probe_rsp_cnt_per_scan) {
-		hdd_notice("NO SCAN result");
+		hdd_debug("NO SCAN result");
 		if (hddctx->config->bug_report_for_no_scan_results) {
 			current_timestamp = jiffies_to_msecs(jiffies);
 			time_elapsed = current_timestamp -
@@ -1312,7 +1312,7 @@ static bool wlan_hdd_sap_skip_scan_check(hdd_context_t *hdd_ctx,
 	int i, j;
 	bool skip;
 
-	hdd_info("HDD_ACS_SKIP_STATUS = %d",
+	hdd_debug("HDD_ACS_SKIP_STATUS = %d",
 		hdd_ctx->skip_acs_scan_status);
 	if (hdd_ctx->skip_acs_scan_status != eSAP_SKIP_ACS_SCAN)
 		return false;
@@ -1335,7 +1335,7 @@ static bool wlan_hdd_sap_skip_scan_check(hdd_context_t *hdd_ctx,
 		}
 		if (!find) {
 			skip = false;
-			hdd_info("Chan %d isn't in ACS chan list",
+			hdd_debug("Chan %d isn't in ACS chan list",
 				request->channels[i]->hw_value);
 			break;
 		}
@@ -1525,7 +1525,7 @@ static void wlan_hdd_update_scan_rand_attrs(void *scan_req,
 	memcpy(scan_mac_addr, cfg_mac_addr, QDF_MAC_ADDR_SIZE);
 	memcpy(scan_mac_addr_mask, cfg_mac_addr_mask, QDF_MAC_ADDR_SIZE);
 
-	hdd_info("Mac Addr: "MAC_ADDRESS_STR " and Mac Mask: " MAC_ADDRESS_STR,
+	hdd_debug("Mac Addr: "MAC_ADDRESS_STR " and Mac Mask: " MAC_ADDRESS_STR,
 		 MAC_ADDR_ARRAY(scan_mac_addr),
 		 MAC_ADDR_ARRAY(scan_mac_addr_mask));
 }
@@ -1594,7 +1594,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 	if (!sme_is_session_id_valid(pHddCtx->hHal, pAdapter->sessionId))
 		return -EINVAL;
 
-	hdd_notice("Device_mode %s(%d)",
+	hdd_debug("Device_mode %s(%d)",
 		hdd_device_mode_to_string(pAdapter->device_mode),
 		pAdapter->device_mode);
 
@@ -1636,7 +1636,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 		if (true == pScanInfo->mScanPending) {
 			if (MAX_PENDING_LOG >
 				pScanInfo->mScanPendingCounter++) {
-				hdd_err("mScanPending is true");
+				hdd_warn("mScanPending is true");
 			}
 			return -EBUSY;
 		}
@@ -1647,7 +1647,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 		 * If no action frame pending
 		 */
 		if (0 != wlan_hdd_check_remain_on_channel(pAdapter)) {
-			hdd_err("Remain On Channel Pending");
+			hdd_warn("Remain On Channel Pending");
 			return -EBUSY;
 		}
 	}
@@ -1665,7 +1665,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 			hdd_err("TDLS in progress.scan rejected %d",
 			status);
 		else
-			hdd_err("TDLS teardown is ongoing %d",
+			hdd_warn("TDLS teardown is ongoing %d",
 			       status);
 		hdd_wlan_block_scan_by_tdls_event();
 		return status;
@@ -1688,7 +1688,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 			pHddCtx->last_scan_reject_timestamp =
 				jiffies_to_msecs(jiffies);
 		} else {
-			hdd_err("curr_session id %d curr_reason %d time delta %lu",
+			hdd_debug("curr_session id %d curr_reason %d time delta %lu",
 				curr_session_id, curr_reason,
 				(jiffies_to_msecs(jiffies) -
 				 pHddCtx->last_scan_reject_timestamp));
@@ -1721,7 +1721,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 	/* Check whether SAP scan can be skipped or not */
 	if (pAdapter->device_mode == QDF_SAP_MODE &&
 	   wlan_hdd_sap_skip_scan_check(pHddCtx, request)) {
-		hdd_err("sap scan skipped");
+		hdd_debug("sap scan skipped");
 		pAdapter->request = request;
 		INIT_WORK(&pAdapter->scan_block_work,
 			wlan_hdd_cfg80211_scan_block_cb);
@@ -1763,7 +1763,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 				     &request->ssids[j].ssid[0],
 				     SsidInfo->SSID.length);
 			SsidInfo->SSID.ssId[SsidInfo->SSID.length] = '\0';
-			hdd_notice("SSID number %d: %s", j,
+			hdd_debug("SSID number %d: %s", j,
 				SsidInfo->SSID.ssId);
 		}
 		/* set the scan type to active */
@@ -1818,8 +1818,8 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 			len += snprintf(chList + len, 5, "%d ", channelList[i]);
 			num_chan++;
 		}
-		hdd_notice("Channel-List: %s", chList);
-		hdd_notice("No. of Scan Channels: %d", num_chan);
+		hdd_debug("Channel-List: %s", chList);
+		hdd_debug("No. of Scan Channels: %d", num_chan);
 	}
 	if (!num_chan) {
 		hdd_err("Received zero non-dsrc channels");
@@ -1891,8 +1891,8 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 			    && (QDF_P2P_CLIENT_MODE == pAdapter->device_mode)) {
 				global_p2p_connection_status =
 					P2P_CLIENT_CONNECTING_STATE_1;
-				hdd_err("[P2P State] Changing state from Go nego completed to Connection is started");
-				hdd_err("[P2P]P2P Scanning is started for 8way Handshake");
+				hdd_debug("[P2P State] Changing state from Go nego completed to Connection is started");
+				hdd_debug("[P2P]P2P Scanning is started for 8way Handshake");
 			} else
 			if ((global_p2p_connection_status ==
 			     P2P_CLIENT_DISCONNECTED_STATE)
@@ -1900,14 +1900,14 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 				pAdapter->device_mode)) {
 				global_p2p_connection_status =
 					P2P_CLIENT_CONNECTING_STATE_2;
-				hdd_err("[P2P State] Changing state from Disconnected state to Connection is started");
-				hdd_err("[P2P]P2P Scanning is started for 4way Handshake");
+				hdd_debug("[P2P State] Changing state from Disconnected state to Connection is started");
+				hdd_debug("[P2P]P2P Scanning is started for 4way Handshake");
 			}
 #endif
 
 			/* no_cck will be set during p2p find to disable 11b rates */
 			if (request->no_cck) {
-				hdd_notice("This is a P2P Search");
+				hdd_debug("This is a P2P Search");
 				scan_req.p2pSearch = 1;
 
 				if (request->n_channels ==
@@ -1951,7 +1951,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 	hdd_prevent_suspend_timeout(HDD_WAKE_LOCK_SCAN_DURATION,
 				    WIFI_POWER_EVENT_WAKELOCK_SCAN);
 
-	hdd_info("requestType %d, scanType %d, minChnTime %d, maxChnTime %d,p2pSearch %d, skipDfsChnlIn P2pSearch %d",
+	hdd_debug("requestType %d, scanType %d, minChnTime %d, maxChnTime %d,p2pSearch %d, skipDfsChnlIn P2pSearch %d",
 	       scan_req.requestType, scan_req.scanType,
 	       scan_req.minChnTime, scan_req.maxChnTime,
 	       scan_req.p2pSearch, scan_req.skipDfsChnlInP2pSearch);
@@ -1970,7 +1970,7 @@ static int __wlan_hdd_cfg80211_scan(struct wiphy *wiphy,
 	if (QDF_STATUS_SUCCESS != status) {
 		hdd_err("sme_scan_request returned error %d", status);
 		if (QDF_STATUS_E_RESOURCES == status) {
-			hdd_err("HO is in progress.So defer the scan by informing busy");
+			hdd_warn("HO is in progress. Defer the scan by informing busy");
 			status = -EBUSY;
 		} else {
 			status = -EIO;
@@ -2637,7 +2637,7 @@ hdd_sched_scan_callback(void *callbackContext,
 	if (true == pHddCtx->isWiphySuspended) {
 		pHddCtx->isSchedScanUpdatePending = true;
 		qdf_spin_unlock(&pHddCtx->sched_scan_lock);
-		hdd_notice("Update cfg80211 scan database after it resume");
+		hdd_debug("Update cfg80211 scan database after it resume");
 		return;
 	}
 	qdf_spin_unlock(&pHddCtx->sched_scan_lock);
@@ -2645,7 +2645,7 @@ hdd_sched_scan_callback(void *callbackContext,
 	ret = wlan_hdd_cfg80211_update_bss(pHddCtx->wiphy, pAdapter, 0);
 
 	if (ret < 0) {
-		hdd_notice("NO SCAN result");
+		hdd_debug("NO SCAN result");
 	} else {
 		/*
 		 * Acquire wakelock to handle the case where APP's tries to
@@ -2659,7 +2659,7 @@ hdd_sched_scan_callback(void *callbackContext,
 	}
 
 	cfg80211_sched_scan_results(pHddCtx->wiphy);
-	hdd_notice("cfg80211 scan result database updated");
+	hdd_debug("cfg80211 scan result database updated");
 }
 
 /**
@@ -2674,7 +2674,7 @@ hdd_sched_scan_callback(void *callbackContext,
  */
 static QDF_STATUS wlan_hdd_is_pno_allowed(hdd_adapter_t *adapter)
 {
-	hdd_notice("dev_mode=%d, conn_state=%d, session ID=%d",
+	hdd_debug("dev_mode=%d, conn_state=%d, session ID=%d",
 		adapter->device_mode,
 		adapter->sessionCtx.station.conn_info.connState,
 		adapter->sessionId);
@@ -2714,7 +2714,7 @@ static void hdd_config_sched_scan_plan(tpSirPNOScanReq pno_req,
 			request->scan_plans[0].iterations;
 		pno_req->slow_scan_period =
 			request->scan_plans[1].interval * MSEC_PER_SEC;
-		hdd_notice("Base scan interval: %d sec, scan cycles: %d, slow scan interval %d",
+		hdd_debug("Base scan interval: %d sec, scan cycles: %d, slow scan interval %d",
 			   request->scan_plans[0].interval,
 			   request->scan_plans[0].iterations,
 			   request->scan_plans[1].interval);
@@ -2745,7 +2745,7 @@ static void hdd_config_sched_scan_plan(tpSirPNOScanReq pno_req,
 	pno_req->slow_scan_period =
 		hdd_ctx->config->pno_slow_scan_multiplier *
 		pno_req->fast_scan_period;
-	hdd_notice("Base scan interval: %d sec PNOScanTimerRepeatValue: %d",
+	hdd_debug("Base scan interval: %d sec PNOScanTimerRepeatValue: %d",
 		   (request->interval / 1000),
 		   hdd_ctx->config->configPNOScanTimerRepeatValue);
 }
@@ -2886,7 +2886,7 @@ static int __wlan_hdd_cfg80211_sched_scan_start(struct wiphy *wiphy,
 						&& (CHANNEL_STATE_DFS ==
 						cds_get_channel_state(
 						    channels_allowed[indx]))) {
-						hdd_notice("Dropping DFS channel : %d",
+						hdd_debug("Dropping DFS channel : %d",
 							   channels_allowed[indx]);
 						num_ignore_dfs_ch++;
 						break;
@@ -2905,13 +2905,13 @@ static int __wlan_hdd_cfg80211_sched_scan_start(struct wiphy *wiphy,
 				}
 			}
 		}
-		hdd_notice("Channel-List: %s ", chList);
+		hdd_debug("Channel-List: %s ", chList);
 
 		/* If all channels are DFS and dropped,
 		 * then ignore the PNO request
 		 */
 		if (!num_ch) {
-			hdd_notice("Channel list empty due to filtering of DSRC,DFS channels");
+			hdd_debug("Channel list empty due to filtering of DSRC,DFS channels");
 			ret = -EINVAL;
 			goto error;
 		}
@@ -2965,7 +2965,7 @@ static int __wlan_hdd_cfg80211_sched_scan_start(struct wiphy *wiphy,
 			j++;
 		}
 	}
-	hdd_notice("Number of hidden networks being Configured = %d",
+	hdd_debug("Number of hidden networks being Configured = %d",
 		  request->n_ssids);
 
 	/*
@@ -2987,13 +2987,13 @@ static int __wlan_hdd_cfg80211_sched_scan_start(struct wiphy *wiphy,
 	 */
 	hdd_config_sched_scan_plan(pPnoRequest, request, pHddCtx);
 
-	hdd_info("Base scan interval: %d sec PNOScanTimerRepeatValue: %d",
+	hdd_debug("Base scan interval: %d sec PNOScanTimerRepeatValue: %d",
 			(pPnoRequest->fast_scan_period / 1000),
 			config->configPNOScanTimerRepeatValue);
 
 	pPnoRequest->modePNO = SIR_PNO_MODE_IMMEDIATE;
 
-	hdd_info("SessionId %d, enable %d, modePNO %d",
+	hdd_debug("SessionId %d, enable %d, modePNO %d",
 		pAdapter->sessionId, pPnoRequest->enable, pPnoRequest->modePNO);
 
 	wlan_hdd_update_scan_rand_attrs((void *)pPnoRequest, (void *)request,
@@ -3010,7 +3010,7 @@ static int __wlan_hdd_cfg80211_sched_scan_start(struct wiphy *wiphy,
 		goto error;
 	}
 
-	hdd_notice("PNO scanRequest offloaded");
+	hdd_debug("PNO scanRequest offloaded");
 
 error:
 	qdf_mem_free(pPnoRequest);
@@ -3102,7 +3102,7 @@ int wlan_hdd_sched_scan_stop(struct net_device *dev)
 		goto exit;
 	}
 
-	hdd_notice("PNO scan disabled");
+	hdd_debug("PNO scan disabled");
 
 exit:
 	EXIT();
@@ -3136,13 +3136,13 @@ static int __wlan_hdd_cfg80211_sched_scan_stop(struct net_device *dev)
 	 * to the first one leading to a crash.
 	 */
 	if (cds_is_driver_recovering()) {
-		hdd_err("Recovery in Progress. State: 0x%x Ignore!!!",
+		hdd_info("Recovery in Progress. State: 0x%x Ignore!!!",
 			 cds_get_driver_state());
 		return 0;
 	}
 
 	if (cds_is_load_or_unload_in_progress()) {
-		hdd_err("Unload/Load in Progress, state: 0x%x.  Ignore!!!",
+		hdd_info("Unload/Load in Progress, state: 0x%x.  Ignore!!!",
 			cds_get_driver_state());
 		return 0;
 	}
@@ -3266,7 +3266,7 @@ void hdd_cleanup_scan_queue(hdd_context_t *hdd_ctx)
 				hdd_cfg80211_scan_done(adapter, req, aborted);
 			else
 				hdd_vendor_scan_callback(adapter, req, aborted);
-			hdd_info("removed Scan id: %d, req = %p",
+			hdd_debug("removed Scan id: %d, req = %p",
 					hdd_scan_req->scan_id, req);
 		}
 		qdf_mem_free(hdd_scan_req);