Commit 6a2a71e5 authored by Johannes Berg's avatar Johannes Berg
Browse files

wifi: iwlwifi: mvm: make a few warnings only trigger once



We're hitting these while starting to enable MLO in the
driver, but getting them each and every time isn't very
useful one way or the other. Make these warnings trigger
only once.

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarGregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20230329100039.f333741d3dcf.If063d4cfe8a583f0f980a1b0ae4e63e17ba4ddc9@changeid


Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 3ca22079
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -5591,7 +5591,7 @@ void iwl_mvm_mac_flush(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
			continue;

		/* make sure only TDLS peers or the AP are flushed */
		WARN_ON(i != mvmvif->deflink.ap_sta_id && !sta->tdls);
		WARN_ON_ONCE(i != mvmvif->deflink.ap_sta_id && !sta->tdls);

		if (drop) {
			if (iwl_mvm_flush_sta(mvm, mvmsta, false))
+2 −2
Original line number Diff line number Diff line
@@ -213,7 +213,7 @@ void iwl_mvm_sec_key_remove_ap(struct iwl_mvm *mvm,
	u32 sec_key_id = WIDE_ID(DATA_PATH_GROUP, SEC_KEY_CMD);
	u8 sec_key_ver = iwl_fw_lookup_cmd_ver(mvm->fw, sec_key_id, 0);

	if (WARN_ON(vif->type != NL80211_IFTYPE_STATION ||
	if (WARN_ON_ONCE(vif->type != NL80211_IFTYPE_STATION ||
			 mvmvif->deflink.ap_sta_id == IWL_MVM_INVALID_STA))
		return;