mbox series

[v3,00/12] mwifiex: two fixes and cleanup

Message ID 20241202-mwifiex-cleanup-1-v3-0-317a6ce0dd5b@pengutronix.de (mailing list archive)
Headers show
Series mwifiex: two fixes and cleanup | expand

Message

Sascha Hauer Dec. 2, 2024, 12:57 p.m. UTC
These are a few patches broken out from [1]. Kalle requested to limit
the number of patches per series to approximately 12 and Francesco to
move the fixes to the front of the series, so here we go.

First two patches are fixes. First one is for host mlme support which
currently is in wireless-next, so no stable tag needed, second one has a
stable tag.

The remaining patches except the last one I have chosen to upstream
first. I'll continue with the other patches after having this series
in shape and merged.

The last one is a new patch not included in [1].

Sascha

[1] https://lore.kernel.org/all/20240820-mwifiex-cleanup-v1-0-320d8de4a4b7@pengutronix.de/

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
Changes in v3:
- Remove Cc: stable tag from 02/12 wifi: mwifiex: fix MAC address handling
- Add better reasons for setting the locally admistered bit in 02/12
  wifi: mwifiex: fix MAC address handling
- Link to v2: https://lore.kernel.org/r/20240918-mwifiex-cleanup-1-v2-0-2d0597187d3c@pengutronix.de

Changes in v2:
- Add refence to 7bff9c974e1a in commit message of "wifi: mwifiex: drop
  asynchronous init waiting code"
- Add extra sentence about bss_started in "wifi: mwifiex: move common
  settings out of switch/case"
- Kill now unused MWIFIEX_BSS_TYPE_ANY
- Collect reviewed-by tags from Francesco Dolcini
- Link to v1: https://lore.kernel.org/r/20240826-mwifiex-cleanup-1-v1-0-56e6f8e056ec@pengutronix.de

---
Sascha Hauer (12):
      wifi: mwifiex: add missing locking
      wifi: mwifiex: fix MAC address handling
      wifi: mwifiex: deduplicate code in mwifiex_cmd_tx_rate_cfg()
      wifi: mwifiex: use adapter as context pointer for mwifiex_hs_activated_event()
      wifi: mwifiex: drop unnecessary initialization
      wifi: mwifiex: make region_code_mapping_t const
      wifi: mwifiex: pass adapter to mwifiex_dnld_cmd_to_fw()
      wifi: mwifiex: simplify mwifiex_setup_ht_caps()
      wifi: mwifiex: fix indention
      wifi: mwifiex: make locally used function static
      wifi: mwifiex: move common settings out of switch/case
      wifi: mwifiex: drop asynchronous init waiting code

 drivers/net/wireless/marvell/mwifiex/cfg80211.c | 38 ++++------
 drivers/net/wireless/marvell/mwifiex/cfp.c      |  4 +-
 drivers/net/wireless/marvell/mwifiex/cmdevt.c   | 76 +++++++-------------
 drivers/net/wireless/marvell/mwifiex/decl.h     |  1 -
 drivers/net/wireless/marvell/mwifiex/init.c     | 19 ++---
 drivers/net/wireless/marvell/mwifiex/main.c     | 94 +++++++++----------------
 drivers/net/wireless/marvell/mwifiex/main.h     | 16 ++---
 drivers/net/wireless/marvell/mwifiex/sta_cmd.c  | 49 ++++---------
 drivers/net/wireless/marvell/mwifiex/txrx.c     |  3 +-
 drivers/net/wireless/marvell/mwifiex/util.c     | 22 +-----
 drivers/net/wireless/marvell/mwifiex/wmm.c      | 12 ++--
 11 files changed, 105 insertions(+), 229 deletions(-)
---
base-commit: 67a72043aa2e6f60f7bbe7bfa598ba168f16d04f
change-id: 20240826-mwifiex-cleanup-1-b5035c7faff6

Best regards,

Comments

Kalle Valo Dec. 2, 2024, 3:26 p.m. UTC | #1
Sascha Hauer <s.hauer@pengutronix.de> writes:

> These are a few patches broken out from [1]. Kalle requested to limit
> the number of patches per series to approximately 12 and Francesco to
> move the fixes to the front of the series, so here we go.
>
> First two patches are fixes. First one is for host mlme support which
> currently is in wireless-next, so no stable tag needed, second one has a
> stable tag.

Are you sure? Apparently we didn't take any patches to wireless-next
during this merge window (ie. last two weeks), so everything which is in
wireless-next should be in v6.13-rc1, unless I'm missing something of
course. Not a problem for me but wanted to point out anyway.
Sascha Hauer Dec. 2, 2024, 3:41 p.m. UTC | #2
On Mon, Dec 02, 2024 at 05:26:33PM +0200, Kalle Valo wrote:
> Sascha Hauer <s.hauer@pengutronix.de> writes:
> 
> > These are a few patches broken out from [1]. Kalle requested to limit
> > the number of patches per series to approximately 12 and Francesco to
> > move the fixes to the front of the series, so here we go.
> >
> > First two patches are fixes. First one is for host mlme support which
> > currently is in wireless-next, so no stable tag needed, second one has a
> > stable tag.
> 
> Are you sure? Apparently we didn't take any patches to wireless-next
> during this merge window (ie. last two weeks), so everything which is in
> wireless-next should be in v6.13-rc1, unless I'm missing something of
> course. Not a problem for me but wanted to point out anyway.

Forget this last sentence, I should have removed it. It was true for v1
of this series. 01/12 is already applied as 0d7c2194f17c7 ("wifi: mwifiex:
add missing locking for cfg80211 calls") and I should have dropped that
patch from this series.

Sascha