Re: [PATCH 2/2] wifi: iwlwifi: mvm: disable MLO for the time being

From: Kalle Valo <kvalo@kernel.org>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless@vger.kernel.org,
	 Miri Korenblit <miriam.rachel.korenblit@intel.com>,
	 ath12k@lists.infradead.org
Subject: Re: [PATCH 2/2] wifi: iwlwifi: mvm: disable MLO for the time being
Date: Thu, 14 Mar 2024 18:45:02 +0200	[thread overview]
Message-ID: <87le6kep9t.fsf@kernel.org> (raw)
In-Reply-To: <db6a60e0dbd66f307a5890d958f67029ad5bcb4f.camel@sipsolutions.net> (Johannes Berg's message of "Thu, 14 Mar 2024 17:40:21 +0100")

Johannes Berg <johannes@sipsolutions.net> writes:

> On Thu, 2024-03-14 at 18:39 +0200, Kalle Valo wrote:
>> Johannes Berg <johannes@sipsolutions.net> writes:
>> 
>> > On Thu, 2024-03-14 at 18:08 +0200, Kalle Valo wrote:
>> > > 
>> > > > -		hw->wiphy->flags |= WIPHY_FLAG_SUPPORTS_MLO;
>> > > > +		hw->wiphy->flags |= WIPHY_FLAG_DISABLE_WEXT;
>> > > 
>> > > I think we should add the same for ath12k, right?
>> > > 
>> > 
>> > It's really up to you, I think. I'm hoping to even get this into stable,
>> > FWIW.
>> 
>> ath12k doesn't support MLO yet but I feel that disabling wext already
>> now would be consistent from user's point of view.
>
> Ah, you're in the situation where wext is still enabled now, but will
> disappear when you enable MLO.

Exactly.

> Agree then that it would make sense to disable wext right now so it's
> not a software upgrade for the same hardware that removes it.

Good, thanks for confirming.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches