mirror of
https://github.com/torvalds/linux.git
synced 2026-06-04 12:35:52 +02:00
wifi: ath11k: Do not directly use scan_flags in struct scan_req_params
As discussed in [1] lets not use WMI_SCAN_XXX defines in combination with scan_flags directly when setting scan params in struct scan_req_params but use the underlying bitfield. This bitfield is then converted to WMI_SCAN_XXX when filling the WMI command to send to the firmware. [1] https://lore.kernel.org/all/871qae51wx.fsf@kernel.org/ Tested-on: QCN9074 PCI WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 Signed-off-by: Nicolas Escande <nico.escande@gmail.com> Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com> Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> Link: https://msgid.link/20240209113536.266822-2-nico.escande@gmail.com
This commit is contained in:
parent
04edb5dc68
commit
64493a7ff7
|
|
@ -4013,7 +4013,7 @@ static int ath11k_mac_op_hw_scan(struct ieee80211_hw *hw,
|
|||
req->ssids[i].ssid_len);
|
||||
}
|
||||
} else {
|
||||
arg->scan_flags |= WMI_SCAN_FLAG_PASSIVE;
|
||||
arg->scan_f_passive = 1;
|
||||
}
|
||||
|
||||
if (req->n_channels) {
|
||||
|
|
@ -9252,8 +9252,8 @@ static int ath11k_mac_op_remain_on_channel(struct ieee80211_hw *hw,
|
|||
arg->dwell_time_active = scan_time_msec;
|
||||
arg->dwell_time_passive = scan_time_msec;
|
||||
arg->max_scan_time = scan_time_msec;
|
||||
arg->scan_flags |= WMI_SCAN_FLAG_PASSIVE;
|
||||
arg->scan_flags |= WMI_SCAN_FILTER_PROBE_REQ;
|
||||
arg->scan_f_passive = 1;
|
||||
arg->scan_f_filter_prb_req = 1;
|
||||
arg->burst_duration = duration;
|
||||
|
||||
ret = ath11k_start_scan(ar, arg);
|
||||
|
|
|
|||
|
|
@ -2098,7 +2098,7 @@ void ath11k_wmi_start_scan_init(struct ath11k *ar,
|
|||
WMI_SCAN_EVENT_BSS_CHANNEL |
|
||||
WMI_SCAN_EVENT_FOREIGN_CHAN |
|
||||
WMI_SCAN_EVENT_DEQUEUED;
|
||||
arg->scan_flags |= WMI_SCAN_CHAN_STAT_EVENT;
|
||||
arg->scan_f_chan_stat_evnt = 1;
|
||||
|
||||
if (test_bit(WMI_TLV_SERVICE_PASSIVE_SCAN_START_TIME_ENHANCE,
|
||||
ar->ab->wmi_ab.svc_map))
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user