RE: [EXT] Re: [PATCH] wifi: mwifiex: avoid AP and STA running on different channel
From: David Lin
Date: Fri Aug 30 2024 - 06:59:06 EST
> From: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> Sent: Friday, August 30, 2024 5:19 PM
> To: Francesco Dolcini <francesco@xxxxxxxxxx>
> Cc: David Lin <yu-hao.lin@xxxxxxx>; linux-wireless@xxxxxxxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; briannorris@xxxxxxxxxxxx; kvalo@xxxxxxxxxx;
> Pete Hsieh <tsung-hsien.hsieh@xxxxxxx>
> Subject: [EXT] Re: [PATCH] wifi: mwifiex: avoid AP and STA running on
> different channel
>
> On Fri, Aug 30, 2024 at 10:10:30AM +0200, Francesco Dolcini wrote:
> > +Sascha, that just sent a patch to handle the same issue.
> >
> > On Fri, Aug 30, 2024 at 11:06:30AM +0800, David Lin wrote:
> > > Current firmware doesn't support AP and STA running on different
> > > channels simultaneously.
> > > FW crash would occur in such case.
> > > This patch avoids the issue by disabling AP and STA to run on
> > > different channels.
> >
> > Is this a generic issue of specific of some firmware version? Asking
> > since the driver as you know is supporting multiple Wi-Fi device.
>
> The driver does, unfortunately the hardware does not.
>
> I learned this the hard way. When an accesspoint is running on uap0 and
> you want to connect to an accesspoint on mlan0 then it won't work when
> that accesspoint is on a different channel.
>
> Likewise, when you are creating an accesspoint on uap0 using channel A and
> another one on channel B then you'll notice that both accesspoints will end
> up using channel A.
>
> It took me a while to find that out. In the end I found the same channel
> check in the nxpwifi driver.
>
> Sascha
>
Yes, this patch is backported from nxpwifi (same as another patch for AP DFS mode).
Because Mwifiex supports connect/disconnect and separate auth/assoc, I think I will
clean up the code and create patch v2 later.
David