qcacmn: Fix the inconsistency in updating to NOL list

Consider AP is brought up in channel 100 VHT80 mode. When "bangradar" cmd
is issued, all the HT20 channels 100, 104, 108, 112 must be added to NOL.
It is found that only chans 100, 104 are added to NOL. This is because
dfs_find_radar_affected_subchans() does not find out all the radar affected
HT20 (4 in this eg) sub-channels.

dfs_find_radar_affected_subchans() is invoked as dfs_use_nol_subchannel
_marking is set to true by default. Also, dfs_find_radar_affected_subchans
finds the frequency offsets of the radar_found channel from
radar_found->freq_offset variable which is updated only in case of real
HW radar. For bangradar, this freq_offset takes to some scrap value.
As the radar found frequency offsets are wrong, the radar found channels
are computed incorrectly and it leads to chaos in NOL addition.

dfs_find_radar_affected_subchans() is used to compute radar found
frequency offsets only in case of real HW radar. For bangradar,
dfs_get_bonding_channels() is used to compute the radar found sub-channels.

Change-Id: Ie190e9266bcb696a07e389736df1258409abf08b
CRs-Fixed: 2313201

umac/dfs/core/src/filtering/dfs_process_radarevent.c

@@ -594,7 +594,6 @@ static inline int dfs_handle_bangradar(
 		 * channel.
 		 */
 		*rs = &dfs->dfs_radar[dfs->dfs_curchan_radindex];
-		dfs->dfs_bangradar = 0; /* Reset */
 		dfs_debug(dfs, WLAN_DEBUG_DFS, "bangradar");
 		*retval = 1;
 		return 1;
@@ -615,7 +614,6 @@ static inline int dfs_handle_bangradar(
 			dfs_debug(dfs, WLAN_DEBUG_DFS,
 					"Do not process the second segment bangradar");
 		}
-		dfs->dfs_second_segment_bangradar = 0; /* Reset */
 		return 1;
 	}
 

umac/dfs/core/src/misc/dfs_process_radar_found_ind.c

@@ -492,6 +492,12 @@ int dfs_second_segment_radar_disable(struct wlan_dfs *dfs)
 	return 0;
 }
 
+static inline void dfs_reset_bangradar(struct wlan_dfs *dfs)
+{
+	dfs->dfs_bangradar  = 0;
+	dfs->dfs_second_segment_bangradar = 0;
+}
+
 QDF_STATUS dfs_process_radar_ind(struct wlan_dfs *dfs,
 				 struct radar_found_info *radar_found)
 {
@@ -523,6 +529,7 @@ QDF_STATUS dfs_process_radar_ind(struct wlan_dfs *dfs,
 			 dfs->dfs_curchan->dfs_ch_freq);
 
 	if (!dfs->dfs_use_nol) {
+		dfs_reset_bangradar(dfs);
 		dfs_send_csa_to_current_chan(dfs);
 		return QDF_STATUS_SUCCESS;
 	}
@@ -539,7 +546,8 @@ QDF_STATUS dfs_process_radar_ind(struct wlan_dfs *dfs,
 		dfs->dfs_enhanced_bangradar = 0;
 	}
 
-	if (dfs->dfs_use_nol_subchannel_marking)
+	if (dfs->dfs_use_nol_subchannel_marking &&
+	    !(dfs->dfs_bangradar || dfs->dfs_second_segment_bangradar))
 		num_channels = dfs_find_radar_affected_subchans(dfs,
 								radar_found,
 								channels);
@@ -548,6 +556,7 @@ QDF_STATUS dfs_process_radar_ind(struct wlan_dfs *dfs,
 							radar_found->segment_id,
 							channels);
 
+	dfs_reset_bangradar(dfs);
 	status = dfs_radar_add_channel_list_to_nol(dfs, channels, num_channels);
 	if (QDF_IS_STATUS_ERROR(status)) {
 		dfs_err(dfs, WLAN_DEBUG_DFS,