> From: Francesco Dolcini <francesco@xxxxxxxxxx> > Sent: Monday, August 14, 2023 6:27 PM > To: David Lin <yu-hao.lin@xxxxxxx> > Cc: Francesco Dolcini <francesco@xxxxxxxxxx>; linux-wireless@xxxxxxxxxxxxxxx; > Sharvari Harisangam <sharvari.harisangam@xxxxxxx>; Pete Hsieh > <tsung-hsien.hsieh@xxxxxxx> > Subject: [EXT] Re: [PATCH v3] wifi: mwifiex: added code to support host > mlme. > > Caution: This is an external email. Please take care when clicking links or > opening attachments. When in doubt, report the message using the 'Report > this email' button > > > On Mon, Aug 14, 2023 at 02:21:51AM +0000, David Lin wrote: > > > From: Francesco Dolcini <francesco@xxxxxxxxxx> > > > Sent: Friday, August 11, 2023 4:01 PM > > > To: David Lin <yu-hao.lin@xxxxxxx> > > > Cc: Francesco Dolcini <francesco@xxxxxxxxxx>; > > > linux-wireless@xxxxxxxxxxxxxxx; Sharvari Harisangam > > > <sharvari.harisangam@xxxxxxx>; Pete Hsieh > > > <tsung-hsien.hsieh@xxxxxxx> > > > Subject: Re: [EXT] Re: [PATCH v3] wifi: mwifiex: added code to > > > support host mlme. > > > On Fri, Aug 11, 2023 at 02:13:43AM +0000, David Lin wrote: > > > > > From: Francesco Dolcini <francesco@xxxxxxxxxx> > > > > > Sent: Thursday, August 10, 2023 10:25 PM > > > > > To: David Lin <yu-hao.lin@xxxxxxx> > > > > > Cc: Francesco Dolcini <francesco@xxxxxxxxxx>; > > > > > linux-wireless@xxxxxxxxxxxxxxx; Sharvari Harisangam > > > > > <sharvari.harisangam@xxxxxxx>; Pete Hsieh > > > > > <tsung-hsien.hsieh@xxxxxxx> > > > > > Subject: Re: [EXT] Re: [PATCH v3] wifi: mwifiex: added code to > > > > > support host mlme. > > > > > > > > > > Caution: This is an external email. Please take care when > > > > > clicking links or opening attachments. When in doubt, report the > > > > > message using the 'Report this email' button > > > > > > > > > > > > > > > On Thu, Aug 10, 2023 at 10:51:36AM +0000, David Lin wrote: > > > > > > > From: Francesco Dolcini <francesco@xxxxxxxxxx> On Thu, Aug > > > > > > > 10, > > > > > > > 2023 at 02:39:21AM +0000, David Lin wrote: > > > > > > > > 3. Firmware version 2 is needed for this feature. > > > > > > > That would mean that we could just enable it for other Wi-Fi > > > > > > > chip variant that has firmware version 2, after testing it? > > > > > > Yes. But I only tested with IW416. > > > > > > > > > > Can/should we probe the firmware version at runtime, instead of > > > > > hard-coding this? > > > > > > > > Firmware version is reported by firmware after firmware is > downloaded. > > > What's the problem with that? You cannot really do anything till the > > > firmware is loaded? > > > > > > If I understand correctly the possibility to use this feature > > > requires its counterpart to be available on the device firmware, > > > this should be the criteria to use to decide if this functionality should be > enabled or not. > > > Hard-coding this depending on the actual device does not look > > > correct to me. > > > > > > > If firmware is old and supports firmware v1, you should not enable > > host mlme for the firmware. Firmware version is detected at runtime > > when firmware is loaded. > > What I am trying to say is that in mwifiex we should just decide to enable or > not host mlme at runtime after we know if the firmware support this > functionality. > O.K. I can add the checking in PATCH v4. Thanks for your suggestion. > Francesco
