Commit 6f092c98 authored by Dmitry Antipov's avatar Dmitry Antipov Committed by Kalle Valo
Browse files

wifi: ath11k: simplify ath11k_mac_validate_vht_he_fixed_rate_settings() 



In ath11k_mac_validate_vht_he_fixed_rate_settings() ar->ab->peers
list is not altered so list_for_each_entry() should be safe.

Compile tested only.

Signed-off-by: default avatarDmitry Antipov <dmantipov@yandex.ru>
Signed-off-by: default avatarKalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20230726092113.78794-1-dmantipov@yandex.ru
parent 72c8caf9

drivers/net/wireless/ath/ath11k/mac.c

+2 −2
Original line number Diff line number Diff line
@@ -8255,7 +8255,7 @@ ath11k_mac_validate_vht_he_fixed_rate_settings(struct ath11k *ar, enum nl80211_b 
					       const struct cfg80211_bitrate_mask *mask)

{

	bool he_fixed_rate = false, vht_fixed_rate = false;

	struct ath11k_peer *peer, *tmp;

	struct ath11k_peer *peer;

	const u16 *vht_mcs_mask, *he_mcs_mask;

	struct ieee80211_link_sta *deflink;

	u8 vht_nss, he_nss;
@@ -8278,7 +8278,7 @@ ath11k_mac_validate_vht_he_fixed_rate_settings(struct ath11k *ar, enum nl80211_b 


	rcu_read_lock();

	spin_lock_bh(&ar->ab->base_lock);

	list_for_each_entry_safe(peer, tmp, &ar->ab->peers, list) {

	list_for_each_entry(peer, &ar->ab->peers, list) {

		if (peer->sta) {

			deflink = &peer->sta->deflink;