wifi: iwlwifi: mvm: allow always calling iwl_mvm_get_bss_vif()

Allow calling iwl_mvm_get_bss_vif() without getting an error
message if multiple vifs are active etc., this is much easier
than having to check beforehand (which would be effectively
the same code as the function itself.)

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
Link: https://patch.msgid.link/20241028135215.a343824e2c61.Ib92534f17c63630aa2eb0604fd80dbae20077487@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/utils.c b/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
index 1d1364d03f028d50a8fb056c29219c8b97a4b7a1..dbbec81559fd9e4e4bd9d4b55ec697572946d8a4 100644 (file)
--- a/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/utils.c
@@ -679,10 +679,8 @@ struct ieee80211_vif *iwl_mvm_get_bss_vif(struct iwl_mvm *mvm)
                mvm->hw, IEEE80211_IFACE_ITER_NORMAL,
                iwl_mvm_bss_iface_iterator, &bss_iter_data);
 
-       if (bss_iter_data.error) {
-               IWL_ERR(mvm, "More than one managed interface active!\n");
+       if (bss_iter_data.error)
                return ERR_PTR(-EINVAL);
-       }
 
        return bss_iter_data.vif;
 }