All of lore.kernel.org
 help / color / mirror / Atom feed
From: Brian Norris <briannorris@chromium.org>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: linux-wireless@vger.kernel.org, David Lin <yu-hao.lin@nxp.com>,
	Francesco Dolcini <francesco@dolcini.it>,
	Johannes Berg <johannes@sipsolutions.net>,
	kernel@pengutronix.de
Subject: Re: Future of mwifiex driver
Date: Mon, 3 Mar 2025 17:45:09 -0800	[thread overview]
Message-ID: <Z8ZbJYmxgnvd7Q1O@google.com> (raw)
In-Reply-To: <Z8WM9jn1QFscWZBQ@pengutronix.de>

Hi Sascha,

On Mon, Mar 03, 2025 at 12:05:26PM +0100, Sascha Hauer wrote:
> I am worried about the future of the mwifiex driver. NXP has an ongoing
> effort of forking the driver to support their new chips, but the forked
> driver lacks support for the old chips supported by the current mwifiex
> driver.
[...] 
> I have a series here [1] doing some cleanup work which I'd still like to
> get forward.
[...]
> [1] https://lore.kernel.org/linux-wireless/87ldwyumvq.fsf@kernel.org/

I'll apologize for that one stalling out a bit. IIRC, 11 of 12 patches
looked great, but I got stuck on the "fix MAC address handling" patch,
because it's a lot tougher to guarantee it doesn't break some use cases
while fixing things. But really, it's probably mostly a bandwidth thing
for me, as I really don't have many cycles to spend on things (and
especially when it gets beyond "obvious cleanup" and requires
substantial testing and/or reasoning).

> Any thoughts?

In no particular order:

1. even if NXP (or you, or anyone) does great work, I'm not going to be
   a super helpful maintainer. I simply don't have time to review and
   test substantial contributions.
2. I get the feeling linux-wireless may have problems like #1 in
   general. Johannes can't fill the entire gap Kalle left, for one.
   (Feel free to correct me if I'm wrong! Or if other excellent people
   have stepped up on review/maintenance.)
3. Other drivers may look somewhat similar, and yet fork for good
   reasons (like, firmware API revisions; or 802.11 generations; or some
   cross-section of both). I'm looking at rtw88/rtw89 (that I was
   involved with quite a bit), or ath10k/ath11k/ath12k/(have we made it
   to 13 yet?). So forking even with quite a bit of similarity isn't
   necessarily inherently wrong.
4. A key difference between #3 and mwifiex is, like you say, that
   mwifiex has a pretty low quality baseline. If I were maintaining it
   from the beginning, I probably wouldn't have accepted it.
5. I'm open to good people stepping up to fill in #1 -- i.e., include
   more maintainers that have a stake in larger contributions. Frankly,
   my only motivation here is to ensure that existing hardware supported
   by mwifiex doesn't get worse.

So all in all, I think I probably agree with you. But speaking openly, I
don't think I can be a large part of the solution here.

Regards,
Brian

  reply	other threads:[~2025-03-04  1:45 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-03 11:05 Future of mwifiex driver Sascha Hauer
2025-03-04  1:45 ` Brian Norris [this message]
2025-03-07  8:48   ` Johannes Berg
2025-03-07  9:47     ` Sascha Hauer
2025-03-19  1:14     ` Brian Norris
2025-03-06 10:17 ` Francesco Dolcini
2025-03-07 10:10   ` Sascha Hauer
2025-03-19 10:32     ` Francesco Dolcini
2025-03-26 12:19       ` Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z8ZbJYmxgnvd7Q1O@google.com \
    --to=briannorris@chromium.org \
    --cc=francesco@dolcini.it \
    --cc=johannes@sipsolutions.net \
    --cc=kernel@pengutronix.de \
    --cc=linux-wireless@vger.kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=yu-hao.lin@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.