2023-10-20 03:48:19

by David Lin

[permalink] [raw]
Subject: [PATCH v6 0/6] wifi: mwifiex: added code to support host mlme.

Patch v6:

Correct mailing sequence.

Patch v5:

1. Add host base MLME support enable WPA3 functionalities in both STA and AP mode
2. This feature requires a firmware with corresponding WPA3 key API support
3. This feature and regression test have been covered for IW416
4. This feature is currently enabled and tested only for IW416
5. Changelogs since Patch v4
a. Enhancement: Add WPA3 AP mode support
b. Bug fix: In WPA3 STA mode, device gets disconnected from AP when group rekey occurs
c. Bug fix: STAUT doesn't send WMM IE in Association request when associate to WMM-AP

Patch v4:

1. Correct one bad code segment.
2. Add code to check firmware key api version for host_mlme.

Patch v3:

Clean and clear commit message.

Patch v2:

1. Checkpatch error: pwe[1] -> pwe[0].
2. Remove module parameter 'host_mlme'.
3. Won't advertise NL80211_FEATURE_SAE if host mlme is disabled.

David Lin (6):
wifi: mwifiex: added code to support host mlme.
wifi: mwifiex: fixed group rekey issue for WPA3.
wifi: mwifiex: fixed reassocation issue for WPA3.
wifi: mwifiex: fixed missing WMM IE for assoc req.
wifi: mwifiex: supported host mlme for AP mode.
wifi: mwifiex: added mac address for AP config.

.../net/wireless/marvell/mwifiex/cfg80211.c | 420 +++++++++++++++++-
drivers/net/wireless/marvell/mwifiex/cmdevt.c | 34 +-
drivers/net/wireless/marvell/mwifiex/decl.h | 12 +
drivers/net/wireless/marvell/mwifiex/fw.h | 36 ++
drivers/net/wireless/marvell/mwifiex/init.c | 4 +
drivers/net/wireless/marvell/mwifiex/ioctl.h | 6 +
drivers/net/wireless/marvell/mwifiex/join.c | 64 ++-
drivers/net/wireless/marvell/mwifiex/main.c | 10 +
drivers/net/wireless/marvell/mwifiex/main.h | 13 +
drivers/net/wireless/marvell/mwifiex/scan.c | 6 +
drivers/net/wireless/marvell/mwifiex/sdio.c | 13 +
drivers/net/wireless/marvell/mwifiex/sdio.h | 2 +
.../wireless/marvell/mwifiex/sta_cmdresp.c | 2 +
.../net/wireless/marvell/mwifiex/sta_event.c | 18 +-
.../net/wireless/marvell/mwifiex/sta_ioctl.c | 3 +-
drivers/net/wireless/marvell/mwifiex/sta_tx.c | 9 +-
.../net/wireless/marvell/mwifiex/uap_cmd.c | 182 ++++++++
drivers/net/wireless/marvell/mwifiex/util.c | 104 +++++
18 files changed, 919 insertions(+), 19 deletions(-)

--
2.25.1


Attachments:
winmail.dat (13.24 kB)

2023-10-24 00:02:53

by Jeff Johnson

[permalink] [raw]
Subject: Re: [PATCH v6 0/6] wifi: mwifiex: added code to support host mlme.

On 10/20/2023 8:24 AM, Francesco Dolcini wrote:
> On Fri, Oct 20, 2023 at 03:48:06AM +0000, David Lin wrote:
>> Patch v6:
>> Correct mailing sequence.
>
> You are still doing something wrong sending the patches over email, the
> various emails are not threaded as expected, git-send-email should do
> everything for you.
>
> No need to re-send now IMO, but please figure out what's wrong for the
> next time.
>
> To give you an idea here [1] you see this email alone.
>
> This [2] and this [3] are examples of a properly sent patch series (just random
> series from your NXP colleagues).
>
> You have also a winmail.dat file attached to every email in this series,
> not sure what it is, but I would try to not send it.
>
> Francesco
>
> [1] https://lore.kernel.org/all/PA4PR04MB96388A628D6AB8359D3DAAD0D1DBA@PA4PR04MB9638.eurprd04.prod.outlook.com/
> [2] https://lore.kernel.org/all/[email protected]/
> [3] https://lore.kernel.org/all/[email protected]/
>

If you are new to contributing to the kernel and don't have an
established workflow I strongly recommend using b4. It removes a lot of
steps from the standard workflow.
<https://b4.docs.kernel.org/en/latest/contributor/overview.html>

2023-10-24 06:07:27

by David Lin

[permalink] [raw]
Subject: RE: [EXT] Re: [PATCH v6 0/6] wifi: mwifiex: added code to support host mlme.



> -----Original Message-----
> From: Jeff Johnson <[email protected]>
> Sent: Tuesday, October 24, 2023 8:03 AM
> To: Francesco Dolcini <[email protected]>; David Lin <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; Sharvari Harisangam
> <[email protected]>; Pete Hsieh <[email protected]>
> Subject: [EXT] Re: [PATCH v6 0/6] 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 10/20/2023 8:24 AM, Francesco Dolcini wrote:
> > On Fri, Oct 20, 2023 at 03:48:06AM +0000, David Lin wrote:
> >> Patch v6:
> >> Correct mailing sequence.
> >
> > You are still doing something wrong sending the patches over email,
> > the various emails are not threaded as expected, git-send-email should
> > do everything for you.
> >
> > No need to re-send now IMO, but please figure out what's wrong for the
> > next time.
> >
> > To give you an idea here [1] you see this email alone.
> >
> > This [2] and this [3] are examples of a properly sent patch series
> > (just random series from your NXP colleagues).
> >
> > You have also a winmail.dat file attached to every email in this
> > series, not sure what it is, but I would try to not send it.
> >
> > Francesco
> >
> > [1]
> > https://lore/
> > .kernel.org%2Fall%2FPA4PR04MB96388A628D6AB8359D3DAAD0D1DBA%40
> PA4PR04MB
> >
> 9638.eurprd04.prod.outlook.com%2F&data=05%7C01%7Cyu-hao.lin%40nxp.co
> m%
> >
> 7Cfa233ec6406043013edb08dbd4248bb1%7C686ea1d3bc2b4c6fa92cd99c5c30
> 1635%
> >
> 7C0%7C0%7C638337025651088842%7CUnknown%7CTWFpbGZsb3d8eyJWIjoi
> MC4wLjAwM
> >
> DAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C
> &sdat
> > a=j%2Fdb7i1jZc5Y%2Fc0s9JUHVmdamBx0Unu9i532CrrEngo%3D&reserved=0
> > [2]
> > https://lore/
> > .kernel.org%2Fall%2F1697794232-2607-1-git-send-email-shengjiu.wang%40n
> >
> xp.com%2F&data=05%7C01%7Cyu-hao.lin%40nxp.com%7Cfa233ec640604301
> 3edb08
> >
> dbd4248bb1%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6383370
> 2565108
> >
> 8842%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMz
> IiLCJBT
> >
> iI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=6bh0m%2FKqIX6u
> IvmTLr2K
> > 6YoEs8feeJsrldxgiiDf%2Fn4%3D&reserved=0
> > [3]
> > https://lore/
> > .kernel.org%2Fall%2F20230811101232.844769-1-neeraj.sanjaykale%40nxp.co
> >
> m%2F&data=05%7C01%7Cyu-hao.lin%40nxp.com%7Cfa233ec6406043013edb0
> 8dbd42
> >
> 48bb1%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638337025651
> 088842%
> >
> 7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJB
> TiI6Ik
> >
> 1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=mleWOeKLaj39Bratl9
> E%2Fn9KsL
> > zx71aippjx50VNt0QI%3D&reserved=0
> >
>
> If you are new to contributing to the kernel and don't have an established
> workflow I strongly recommend using b4. It removes a lot of steps from the
> standard workflow.

Thanks. I will reference it.

> <https://b4.docs/.
> kernel.org%2Fen%2Flatest%2Fcontributor%2Foverview.html&data=05%7C01%
> 7Cyu-hao.lin%40nxp.com%7Cfa233ec6406043013edb08dbd4248bb1%7C686ea
> 1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638337025651088842%7CUnkn
> own%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1h
> aWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=wAe4ueeaorIDgVIYpkf6x
> 4PpZ5epJynpQOzUiqaGLAs%3D&reserved=0>