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: Dmitry Antipov <dmantipov@yandex.ru>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20230726092113.78794-1-dmantipov@yandex.ru

diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index 8c77ade49437fff1b94d4ef266fe775c678e8c40..2aadf2c387b62dca81f473e7a05abcb497d04736 100644 (file)
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -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;