Browse Source

qcacmn: Fix configuration of Spectral module for Gen2

Spectral module configuration is failing on Gen2, fix it.

CRs-Fixed: 2187148
Change-Id: I424512d181e70d300a71210cf3e0593b2ddbf24f
Shiva Krishna Pittala 7 years ago
parent
commit
cd67294973

+ 8 - 8
target_if/spectral/target_if_spectral.c

@@ -1834,16 +1834,17 @@ static void
 target_if_spectral_detach(struct target_if_spectral *spectral)
 {
 	spectral_info("spectral detach");
-	qdf_spinlock_destroy(&spectral->param_info.osps_lock);
 
-	target_if_spectral_detach_simulation(spectral);
+	if (spectral) {
+		qdf_spinlock_destroy(&spectral->param_info.osps_lock);
 
-	spectral->nl_cb.destroy_netlink(spectral->pdev_obj);
+		target_if_spectral_detach_simulation(spectral);
 
-	qdf_spinlock_destroy(&spectral->spectral_lock);
-	qdf_spinlock_destroy(&spectral->noise_pwr_reports_lock);
+		spectral->nl_cb.destroy_netlink(spectral->pdev_obj);
+
+		qdf_spinlock_destroy(&spectral->spectral_lock);
+		qdf_spinlock_destroy(&spectral->noise_pwr_reports_lock);
 
-	if (spectral) {
 		qdf_mem_free(spectral);
 		spectral = NULL;
 	}
@@ -1971,7 +1972,7 @@ target_if_pdev_spectral_init(struct wlan_objmgr_pdev *pdev)
 		spectral->tag_sscan_fft_exp = TLV_TAG_SEARCH_FFT_REPORT_GEN3;
 		spectral->tlvhdr_size = SPECTRAL_PHYERR_TLVSIZE_GEN3;
 	} else
-#else
+#endif
 	{
 		spectral->spectral_gen = SPECTRAL_GEN2;
 		spectral->hdr_sig_exp = SPECTRAL_PHYERR_SIGNATURE_GEN2;
@@ -1980,7 +1981,6 @@ target_if_pdev_spectral_init(struct wlan_objmgr_pdev *pdev)
 		spectral->tag_sscan_fft_exp = TLV_TAG_SEARCH_FFT_REPORT_GEN2;
 		spectral->tlvhdr_size = sizeof(struct spectral_phyerr_tlv_gen2);
 	}
-#endif
 
 	if (target_if_spectral_attach_simulation(spectral) < 0)
 		return NULL;

+ 0 - 1
target_if/spectral/target_if_spectral_netlink.c

@@ -252,7 +252,6 @@ target_if_spectral_create_samp_msg(struct target_if_spectral *spectral,
 		target_if_spectral_process_noise_pwr_report(
 			spectral, spec_samp_msg);
 
-	spectral_debug("Recieved sample message");
 	if (spectral->send_phy_data(spectral->pdev_obj) == 0)
 		spectral->spectral_sent_msg++;