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
next prev parent 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.