wifi: iwlwifi: mvm: Correctly set link configuration

[ Upstream commit 35b9281fb710ea9fa47dca56774f4a9606fe9154 ]

In case the link puncturing is changed such that the channel
is no longer punctured, configure the FW correctly indicating
the EHT parameters changed (with a 0 punctured map).

Allow EHT parameters configuration only when the link really
supports EHT.

Fixes: 55eb1c5fa4b2 ("wifi: iwlwifi: mvm: add support for the new LINK command")
Signed-off-by: Ilan Peer <ilan.peer@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20231004123422.2666ef86e032.I4b0e95722660acc5345ceefba7e8866a69572e8e@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/link.c b/drivers/net/wireless/intel/iwlwifi/mvm/link.c
index ace82e2..4f8d2a3 100644 (file)
--- a/drivers/net/wireless/intel/iwlwifi/mvm/link.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/link.c
@@ -194,11 +194,14 @@ int iwl_mvm_link_changed(struct iwl_mvm *mvm, struct ieee80211_vif *vif,
                flags_mask |= LINK_FLG_MU_EDCA_CW;
        }
 
-       if (link_conf->eht_puncturing && !iwlwifi_mod_params.disable_11be)
-               cmd.puncture_mask = cpu_to_le16(link_conf->eht_puncturing);
-       else
-               /* This flag can be set only if the MAC has eht support */
-               changes &= ~LINK_CONTEXT_MODIFY_EHT_PARAMS;
+       if (changes & LINK_CONTEXT_MODIFY_EHT_PARAMS) {
+               if (iwlwifi_mod_params.disable_11be ||
+                   !link_conf->eht_support)
+                       changes &= ~LINK_CONTEXT_MODIFY_EHT_PARAMS;
+               else
+                       cmd.puncture_mask =
+                               cpu_to_le16(link_conf->eht_puncturing);
+       }
 
        cmd.bss_color = link_conf->he_bss_color.color;
 

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c
index 778a311..a3a2e9a 100644 (file)
--- a/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/mld-mac80211.c
@@ -653,7 +653,7 @@ iwl_mvm_mld_link_info_changed_station(struct iwl_mvm *mvm,
        }
 
        /* Update EHT Puncturing info */
-       if (changes & BSS_CHANGED_EHT_PUNCTURING && vif->cfg.assoc && has_eht)
+       if (changes & BSS_CHANGED_EHT_PUNCTURING && vif->cfg.assoc)
                link_changes |= LINK_CONTEXT_MODIFY_EHT_PARAMS;
 
        if (link_changes) {