Browse Source

qcacmn: Print an error message when Agile spectral frequency is 0

When agile spectral frequency is 0 and user tries to start agile spectral
scan, 'A parameter is not initialized' message is thrown from user space
but that message is not enough to identify that agile spectral frequency
is not initialized. To address this, print an error message from driver
when Agile spectral frequency is 0.

CRs-Fixed: 2921000
Change-Id: Ibbb144a57774ad274b31015c7a04aa6097daf09f
Shiva Krishna Pittala 4 years ago
parent
commit
ec05e79403
1 changed files with 8 additions and 3 deletions
  1. 8 3
      target_if/spectral/target_if_spectral.c

+ 8 - 3
target_if/spectral/target_if_spectral.c

@@ -4833,11 +4833,16 @@ target_if_start_spectral_scan(struct wlan_objmgr_pdev *pdev,
 		op_ch_width = ch_width[SPECTRAL_SCAN_MODE_NORMAL];
 		agile_ch_width = ch_width[SPECTRAL_SCAN_MODE_AGILE];
 
-		if (!spectral->params[smode].ss_frequency.cfreq1 ||
-		    (agile_ch_width == CH_WIDTH_80P80MHZ &&
-		    !spectral->params[smode].ss_frequency.cfreq2)) {
+		if (!spectral->params[smode].ss_frequency.cfreq1) {
 			*err = SPECTRAL_SCAN_ERR_PARAM_NOT_INITIALIZED;
 			qdf_spin_unlock(&spectral->spectral_lock);
+			spectral_err("Agile Spectral cfreq1 is 0");
+			return QDF_STATUS_E_FAILURE;
+		} else if (agile_ch_width == CH_WIDTH_80P80MHZ &&
+			   !spectral->params[smode].ss_frequency.cfreq2) {
+			*err = SPECTRAL_SCAN_ERR_PARAM_NOT_INITIALIZED;
+			qdf_spin_unlock(&spectral->spectral_lock);
+			spectral_err("Agile Spectral cfreq2 is 0");
 			return QDF_STATUS_E_FAILURE;
 		}