linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x
@ 2025-07-21  6:26 Jeff Chen
  2025-07-21 13:25 ` Francesco Dolcini
  0 siblings, 1 reply; 4+ messages in thread
From: Jeff Chen @ 2025-07-21  6:26 UTC (permalink / raw)
  To: linux-wireless, johannes
  Cc: linux-kernel, briannorris, francesco, s.hauer, tsung-hsien.hsieh,
	jeff.chen_1, brian.hsu

This series adds a new full-MAC Wi-Fi driver `nxpwifi` to support NXP IW61x
chip family. These chips are tri-radio single-chip solutions with Wi-Fi 6
(1x1, 2.4/5 GHz), Bluetooth 5.4, and IEEE 802.15.4. Communication with the
external host is via SDIO interface. The driver is tested on i.MX8M Mini 
EVK in both STA and AP mode.

This driver is not based on mac80211. It derives from mwifiex, but due to
significant differences in firmware architecture, host command interface,
and supported features, it was not feasible to extend mwifiex without risk
of regressions. Thus, a new driver is introduced. Future NXP Wi-Fi chipsets
will also be supported under `nxpwifi`.

The driver passes checkpatch. WPA2/WPA3 personal/enterprise is currently
handled in host via wpa_supplicant/hostapd.

v4 adds support for TWT, monitor mode, WPA3 Enterprise Suite-B, and various
fixes and improvements. See below for full changelog.

Data sheet is available at:
https://www.nxp.com/docs/en/data-sheet/IW612.pdf

Change history:

v4:
   - Added support for TWT (STA mode)
   - Added support for Monitor mode
   - Added support for WPA3 Enterprise Suite-B
   - Bug fix: In BGN/AN HT40 mode, throughput was 50% lower than expected
   - Bug fix: In STA 2.4G HE40 mode, throughput was 80% lower than expected
   - Use wiphy work instead of general workqueue for cfg80211 ops
   - Introduced RCU protection for ba, rx_reorder, and sta_lists
   - Used per-TID spinlock for tx_ba / rx_reorder lists to improve
     concurrency
   - Replaced mutex_lock with wiphy_lock
   - Reverted "use tasklet for Rx" (back to workqueue)
   - Refactored HE capability handling for better clarity and
     maintainability
   - Used standard kernel helpers for MAC address handling
   - Replaced proprietary nxpwifi_ieee80211 with standard ieee80211_mgmt
     structure
   - Used u32_get_bits() for VHT capability field access
   - Replaced LOW_PRIO_TID with TC_PRIO_BESTEFFORT for skb priority
   - Removed static WPA/RSN OUI table; now derived from cipher suite ID
   - Removed redundant helper functions: has_vendor_hdr() and has_ieee_hdr(

v3:
   - Enable 802.11ax (Wi-Fi 6) for both AP and STA mode.
   - Extend driver verion string with hotfix number.
   - Remove Rx mlme work.
   - Remove all useless check of netif_carrier_ok().
   - Merge decl.h to cfg.h.
   - Remove unnecessary check for wiphy parameters setting.
   - Synchronize following commits from Mwifiex: 
     wifi: mwifiex: replace open-coded module_sdio_driver() 
     wifi: mwifiex: Fix interface type change 
     wifi: mwifiex: Do not return unused priv in mwifiex_get_priv_by_id() 
     wifi: mwifiex: increase max_num_akm_suites 
     wifi: mwifiex: duplicate static structs used in driver instances 
     wifi: mwifiex: keep mwifiex_cfg80211_ops constant 
     wifi: mwifiex: Fix uninitialized variable in 
                    mwifiex_cfg80211_authenticate() 
     wifi: mwifiex: remove unnecessary checks for valid priv 
     wifi: mwifiex: Fix memcpy() field-spanning write warning in
                    mwifiex_cmd_802_11_scan_ext() 
     wifi: mwifiex: Use IRQF_NO_AUTOEN flag in request_irq() 

v2:
   - Rename ioctl.h and sta_ioctl.c to cfg.h and sta_cfg.c.
   - Remove useless header file semaphore.h.
   - Use static variable for cookie.
   - Modify nxpwifi_register to use ERR_PTR, IS_ERR and PTR_ERR.
   - Use error number for error code.
   - Remove unnecessary private ie definitions.
   - Remove mutex async_mutex and related code.
   - Remove unnecessary work queue.
   - Add the support for PSK SHA256.
   - Use tasklet for Rx.
   - Remove unused functions.
   - Remove compile warning.

Jeff Chen (22):
  wifi: nxpwifi: add 802.11n files
  wifi: nxpwifi: add 802.11ac files
  wifi: nxpwifi: add 802.11ax files
  wifi: nxpwifi: add 802.11h file
  wifi: nxpwifi: add WMM files
  wifi: nxpwifi: add scan.c
  wifi: nxpwifi: add join.c
  wifi: nxpwifi: add cfp.c
  wifi: nxpwifi: add configuration files
  wifi: nxpwifi: implement cfg80211 ops
  wifi: nxpwifi: add host command file
  wifi: nxpwifi: add command and event files
  wifi: nxpwifi: add data path files
  wifi: nxpwifi: add debugfs file
  wifi: nxpwifi: add ethtool.c
  wifi: nxpwifi: add utility files
  wifi: nxpwifi: add initialization file
  wifi: nxpwifi: add core files
  wifi: nxpwifi: add sdio bus driver files
  wifi: nxpwifi: modify sdio_ids.h
  wifi: nxpwifi: add makefile and kconfig files
  wifi: nxpwifi: modify MAINTAINERS file

 MAINTAINERS                                   |    9 +
 drivers/net/wireless/Kconfig                  |    1 +
 drivers/net/wireless/Makefile                 |    1 +
 drivers/net/wireless/nxp/Kconfig              |   17 +
 drivers/net/wireless/nxp/Makefile             |    3 +
 drivers/net/wireless/nxp/nxpwifi/11ac.c       |  289 ++
 drivers/net/wireless/nxp/nxpwifi/11ac.h       |   32 +
 drivers/net/wireless/nxp/nxpwifi/11ax.c       |  590 +++
 drivers/net/wireless/nxp/nxpwifi/11ax.h       |   72 +
 drivers/net/wireless/nxp/nxpwifi/11h.c        |  346 ++
 drivers/net/wireless/nxp/nxpwifi/11n.c        |  895 ++++
 drivers/net/wireless/nxp/nxpwifi/11n.h        |  167 +
 drivers/net/wireless/nxp/nxpwifi/11n_aggr.c   |  275 ++
 drivers/net/wireless/nxp/nxpwifi/11n_aggr.h   |   21 +
 .../net/wireless/nxp/nxpwifi/11n_rxreorder.c  |  938 ++++
 .../net/wireless/nxp/nxpwifi/11n_rxreorder.h  |   71 +
 drivers/net/wireless/nxp/nxpwifi/Kconfig      |   22 +
 drivers/net/wireless/nxp/nxpwifi/Makefile     |   39 +
 drivers/net/wireless/nxp/nxpwifi/cfg.h        |  984 ++++
 drivers/net/wireless/nxp/nxpwifi/cfg80211.c   | 4067 +++++++++++++++++
 drivers/net/wireless/nxp/nxpwifi/cfg80211.h   |   19 +
 drivers/net/wireless/nxp/nxpwifi/cfp.c        |  475 ++
 drivers/net/wireless/nxp/nxpwifi/cmdevt.c     | 1284 ++++++
 drivers/net/wireless/nxp/nxpwifi/cmdevt.h     |   98 +
 drivers/net/wireless/nxp/nxpwifi/debugfs.c    | 1230 +++++
 drivers/net/wireless/nxp/nxpwifi/ethtool.c    |   58 +
 drivers/net/wireless/nxp/nxpwifi/fw.h         | 2401 ++++++++++
 drivers/net/wireless/nxp/nxpwifi/ie.c         |  501 ++
 drivers/net/wireless/nxp/nxpwifi/init.c       |  684 +++
 drivers/net/wireless/nxp/nxpwifi/join.c       |  911 ++++
 drivers/net/wireless/nxp/nxpwifi/main.c       | 1718 +++++++
 drivers/net/wireless/nxp/nxpwifi/main.h       | 1850 ++++++++
 drivers/net/wireless/nxp/nxpwifi/scan.c       | 2805 ++++++++++++
 drivers/net/wireless/nxp/nxpwifi/sdio.c       | 2656 +++++++++++
 drivers/net/wireless/nxp/nxpwifi/sdio.h       |  340 ++
 drivers/net/wireless/nxp/nxpwifi/sta_cfg.c    | 1319 ++++++
 drivers/net/wireless/nxp/nxpwifi/sta_cmd.c    | 3449 ++++++++++++++
 drivers/net/wireless/nxp/nxpwifi/sta_event.c  |  868 ++++
 drivers/net/wireless/nxp/nxpwifi/sta_rx.c     |  250 +
 drivers/net/wireless/nxp/nxpwifi/sta_tx.c     |  208 +
 drivers/net/wireless/nxp/nxpwifi/txrx.c       |  358 ++
 drivers/net/wireless/nxp/nxpwifi/uap_cmd.c    | 1234 +++++
 drivers/net/wireless/nxp/nxpwifi/uap_event.c  |  490 ++
 drivers/net/wireless/nxp/nxpwifi/uap_txrx.c   |  506 ++
 drivers/net/wireless/nxp/nxpwifi/util.c       | 1580 +++++++
 drivers/net/wireless/nxp/nxpwifi/util.h       |  132 +
 drivers/net/wireless/nxp/nxpwifi/wmm.c        | 1379 ++++++
 drivers/net/wireless/nxp/nxpwifi/wmm.h        |   78 +
 include/linux/mmc/sdio_ids.h                  |    3 +
 49 files changed, 37723 insertions(+)
 create mode 100644 drivers/net/wireless/nxp/Kconfig
 create mode 100644 drivers/net/wireless/nxp/Makefile
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11ac.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11ac.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11ax.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11ax.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11h.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n_aggr.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n_aggr.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n_rxreorder.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/11n_rxreorder.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/Kconfig
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/Makefile
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cfg.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cfg80211.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cfg80211.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cfp.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cmdevt.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/cmdevt.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/debugfs.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/ethtool.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/fw.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/ie.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/init.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/join.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/main.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/main.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/scan.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sdio.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sdio.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sta_cfg.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sta_cmd.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sta_event.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sta_rx.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/sta_tx.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/txrx.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/uap_cmd.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/uap_event.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/uap_txrx.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/util.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/util.h
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/wmm.c
 create mode 100644 drivers/net/wireless/nxp/nxpwifi/wmm.h

-- 
2.34.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x
  2025-07-21  6:26 Jeff Chen
@ 2025-07-21 13:25 ` Francesco Dolcini
  0 siblings, 0 replies; 4+ messages in thread
From: Francesco Dolcini @ 2025-07-21 13:25 UTC (permalink / raw)
  To: Jeff Chen
  Cc: linux-wireless, johannes, linux-kernel, briannorris, francesco,
	s.hauer, tsung-hsien.hsieh, brian.hsu

Hello Jeff,

On Mon, Jul 21, 2025 at 02:26:19PM +0800, Jeff Chen wrote:
> This driver is not based on mac80211. It derives from mwifiex, but due to
> significant differences in firmware architecture, host command interface,
> and supported features, it was not feasible to extend mwifiex without risk
> of regressions. Thus, a new driver is introduced. Future NXP Wi-Fi chipsets
> will also be supported under `nxpwifi`.

What changed compared to the last time nxpwifi was discussed and from v3?

My last status is the following

 1 - it is possible to extend mwifiex to support iw61x [1]
 2 - duplicating mwifiex was not considered a good idea
 3 - keeping mwifiex well maintained is considered important

Francesco

[1] https://lore.kernel.org/all/20240826072648.167004-1-s.hauer@pengutronix.de/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x
@ 2025-07-22 13:00 Gallissot Mathieu
  2025-08-20 10:52 ` [EXT] " Jeff Chen
  0 siblings, 1 reply; 4+ messages in thread
From: Gallissot Mathieu @ 2025-07-22 13:00 UTC (permalink / raw)
  To: jeff.chen_1@nxp.com
  Cc: brian.hsu@nxp.com, briannorris@chromium.org, francesco@dolcini.it,
	johannes@sipsolutions.net, linux-kernel@vger.kernel.org,
	linux-wireless@vger.kernel.org, s.hauer@pengutronix.de,
	tsung-hsien.hsieh@nxp.com

Hi,

I've been try to compile this patch on 6.12.33 (commit id e03ced99c437f4a7992b8fa3d97d598f55453fd0), I've got the following error messages. Which version of kernel does this patch applies?

drivers/net/wireless/nxp/nxpwifi/cmdevt.c:8:10: fatal error: asm/unaligned.h: No such file or directory
    8 | #include <asm/unaligned.h>
      |          ^~~~~~~~~~~~~~~~~
compilation terminated.
make[7]: *** [scripts/Makefile.build:287: drivers/net/wireless/nxp/nxpwifi/cmdevt.o] Error 1
make[7]: *** Waiting for unfinished jobs....
  CC [M]  drivers/net/wireless/ti/wlcore/scan.o
  CC [M]  drivers/net/wireless/ti/wlcore/sysfs.o
  CC [M]  drivers/net/wireless/rsi/rsi_91x_sdio_ops.o
  CC [M]  drivers/net/wireless/ti/wlcore/vendor_cmd.o
drivers/net/wireless/nxp/nxpwifi/main.c: In function 'nxpwifi_unregister':
drivers/net/wireless/nxp/nxpwifi/main.c:114:9: error: implicit declaration of function 'del_timer_syn'; did you mean 'dev_mc_sync'? [-Werror=implicit-function-declaration]
  114 |         del_timer_sync(&adapter->cmd_timer);
      |         ^~~~~~~~~~~~~~
      |         dev_mc_sync
  CC [M]  drivers/net/wireless/rsi/rsi_91x_usb.o
drivers/net/wireless/nxp/nxpwifi/main.c: In function 'nxpwifi_main_process':
drivers/net/wireless/nxp/nxpwifi/main.c:274:25: error: implicit declaration of function 'del_timer'; did you mean 'add_timer'? [-Werror=implicit-function-declaration]
  274 |                         del_timer(&adapter->wakeup_timer);
      |                         ^~~~~~~~~
      |                         add_timer
In file included from ./include/linux/export.h:5,
                 from ./include/linux/linkage.h:7,
                 from ./include/linux/preempt.h:10,
                 from ./include/linux/spinlock.h:56,
                 from ./include/linux/swap.h:5,
                 from ./include/linux/suspend.h:5,
                 from drivers/net/wireless/nxp/nxpwifi/main.c:8:
drivers/net/wireless/nxp/nxpwifi/main.c: In function 'nxpwifi_hard_start_xmit':
drivers/net/wireless/nxp/nxpwifi/main.c:838:50: error: 'SKBTX_WIFI_STATUS' undeclared (first use in this function); did you mean 'SOCK_WIFI_STATUS'?
  838 |                      skb_shinfo(skb)->tx_flags & SKBTX_WIFI_STATUS &&
      |                                                  ^~~~~~~~~~~~~~~~~
./include/linux/compiler.h:77:45: note: in definition of macro 'unlikely'
   77 | # define unlikely(x)    __builtin_expect(!!(x), 0)
      |                                             ^
drivers/net/wireless/nxp/nxpwifi/main.c:838:50: note: each undeclared identifier is reported only once for each function it appears in
  838 |                      skb_shinfo(skb)->tx_flags & SKBTX_WIFI_STATUS &&
      |                                                  ^~~~~~~~~~~~~~~~~
./include/linux/compiler.h:77:45: note: in definition of macro 'unlikely'
   77 | # define unlikely(x)    __builtin_expect(!!(x), 0)
      |                                             ^
cc1: some warnings being treated as errors
drivers/net/wireless/nxp/nxpwifi/init.c: In function 'wakeup_timer_fn':
drivers/net/wireless/nxp/nxpwifi/init.c:43:43: error: implicit declaration of function 'from_timer'; did you mean 'mod_timer'? [-Werror=implicit-function-declaration]
   43 |         struct nxpwifi_adapter *adapter = from_timer(adapter, t, wakeup_timer);
      |                                           ^~~~~~~~~~
      |                                           mod_timer
drivers/net/wireless/nxp/nxpwifi/init.c:43:66: error: 'wakeup_timer' undeclared (first use in this function); did you mean 'wakeup_timer_fn'?
   43 |         struct nxpwifi_adapter *adapter = from_timer(adapter, t, wakeup_timer);
      |                                                                  ^~~~~~~~~~~~
      |                                                                  wakeup_timer_fn
drivers/net/wireless/nxp/nxpwifi/init.c:43:66: note: each undeclared identifier is reported only once for each function it appears in
make[7]: *** [scripts/Makefile.build:287: drivers/net/wireless/nxp/nxpwifi/main.o] Error 1
drivers/net/wireless/nxp/nxpwifi/init.c: In function 'nxpwifi_adapter_cleanup':
drivers/net/wireless/nxp/nxpwifi/init.c:361:9: error: implicit declaration of function 'del_timer'; did you mean 'add_timer'? [-Werror=implicit-function-declaration]
  361 |         del_timer(&adapter->wakeup_timer);
      |         ^~~~~~~~~
      |         add_timer
  CC [M]  drivers/net/wireless/rsi/rsi_91x_usb_ops.o
  CC [M]  drivers/net/wireless/ti/wlcore/sdio.o
drivers/net/wireless/nxp/nxpwifi/init.c: In function 'nxpwifi_shutdown_drv':
drivers/net/wireless/nxp/nxpwifi/init.c:577:17: error: implicit declaration of function 'del_timer_sync'; did you mean 'dev_mc_sync'? [-Werror=implicit-function-declaration]
  577 |                 del_timer_sync(&adapter->cmd_timer);
      |                 ^~~~~~~~~~~~~~
      |                 dev_mc_sync
cc1: some warnings being treated as errors

Thanks,
Mathieu

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [EXT] Re: [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x
  2025-07-22 13:00 [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x Gallissot Mathieu
@ 2025-08-20 10:52 ` Jeff Chen
  0 siblings, 0 replies; 4+ messages in thread
From: Jeff Chen @ 2025-08-20 10:52 UTC (permalink / raw)
  To: Gallissot Mathieu
  Cc: Brian Hsu, briannorris@chromium.org, francesco@dolcini.it,
	johannes@sipsolutions.net, linux-kernel@vger.kernel.org,
	linux-wireless@vger.kernel.org, s.hauer@pengutronix.de,
	Pete Hsieh

Hi Mathieu,

Apologies for the late reply, and thank you for catching the build errors.

I've fixed the build errors and posted the updated v5 patch series.
You can find the series here on Patchwork:
https://patchwork.kernel.org/project/linux-wireless/list/?series=988195

Thanks and regards,

Jeff

> Hi,
> 
> I've been try to compile this patch on 6.12.33 (commit id
> e03ced99c437f4a7992b8fa3d97d598f55453fd0), I've got the following error
> messages. Which version of kernel does this patch applies?
> 
> Thanks,
> Mathieu

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-08-20 10:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-22 13:00 [PATCH v4 00/22] wifi: nxpwifi: create nxpwifi to support iw61x Gallissot Mathieu
2025-08-20 10:52 ` [EXT] " Jeff Chen
  -- strict thread matches above, loose matches on Subject: below --
2025-07-21  6:26 Jeff Chen
2025-07-21 13:25 ` Francesco Dolcini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).