From: Greg KH <gregkh@linuxfoundation.org>
To: Luka Gejak <luka.gejak@linux.dev>
Cc: Ping-Ke Shih <pkshih@realtek.com>,
linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
Michael Straube <straube.linux@gmail.com>,
Peter Robinson <pbrobinson@gmail.com>,
Bitterblue Smith <rtl8821cerfe2@gmail.com>
Subject: Re: [PATCH rtw-next v8 5/7] wifi: rtw88: 8723b: add the RTL8723B chip driver
Date: Wed, 14 Oct 2026 15:04:25 +0200 [thread overview]
Message-ID: <2026101452-chapter-friday-dada@gregkh> (raw)
In-Reply-To: <4464409846710dfa13134265f0f99523714cfa92@linux.dev>
On Mon, Oct 12, 2026 at 11:31:57AM +0000, Luka Gejak wrote:
> October 10, 2026 at 12:18 PM, "Greg KH" <gregkh@linuxfoundation.org> wrote:
>
>
> >
> > On Wed, Oct 07, 2026 at 10:35:25AM +0000, Luka Gejak wrote:
> >
> > >
> [...]
> > >
> > > Hi Greg,
> > >
> > > I have sent the patch, and I think you are going to like this
> > > number: 66186 deletions
> > >
> > Very happy :)
> >
> > >
> > > Not sure what people are going to use to learn though. It seems to
> > > me everyone is repeating the same patches.
> > >
> > Yeah, we need to dig up some more out-of-tree kernel code to add it for
> > them to churn on. But really, this is a good thing, it's nice to see
> > real support for this hardware finally.
>
> Hi Greg,
>
> As both of you and I know there are not many young people in the
> kernel community and the steep learning curve is a big reason for that.
Really? We average about 200-250 new contributors every kernel release
until recently when it's shot up to 600-700 if not more. So we don't
seem to have a lack of new contributors for some reason...
> I was not sure whether you were joking or not about the comment
> above so I decided to find that appropriate replacement for staging
> rtl8723bs. After searching I found this interesting xradio driver,
> which is a mac80211 based driver for some orange pies and other devices.
> It is around 18,000 lines of code and upon check it out I fixed bunch
> of security issues per se and lockups that caused kernel to hang and it
> is decently stable. I'm not gonna claim some extreme stability but it
> is at least usable and it has many coding standard issues. It is a good
> candidate because newcomers would be learning the modern architecture
> and on top of it there is a lot of janitor work so people can learn
> the workflow and etc. Link is [1] so you can see for yourself and tell
> me what you think about it.
If you think it is worth adding, sure, but normally we need someone to
actually care about it getting into the tree (i.e. real users
somewhere). Also look at the bottom of the readme, shouldn't the
existing in-kernel driver just be tweaked for this hardware instead?
thanks,
greg k-h
next prev parent reply other threads:[~2026-10-14 13:04 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-06 13:06 [PATCH rtw-next v8 0/7] wifi: rtw88: add RTL8723B/RTL8723BS support Luka Gejak
2026-10-06 13:06 ` [PATCH rtw-next v8 1/7] wifi: rtw88: move the 88xxa CCK power detect setter to phy.c Luka Gejak
2026-10-08 2:52 ` Ping-Ke Shih
2026-10-06 13:06 ` [PATCH rtw-next v8 2/7] wifi: rtw88: 8723b: add the RTL8723B register definitions Luka Gejak
2026-10-06 13:06 ` [PATCH rtw-next v8 3/7] wifi: rtw88: 8723b: add the RTL8723B BB, RF and AGC tables Luka Gejak
2026-10-06 13:06 ` [PATCH rtw-next v8 4/7] wifi: rtw88: move the shared 8723x definitions to rtw8723x.h Luka Gejak
2026-10-07 2:38 ` Ping-Ke Shih
2026-10-06 13:06 ` [PATCH rtw-next v8 5/7] wifi: rtw88: 8723b: add the RTL8723B chip driver Luka Gejak
2026-10-07 2:57 ` Ping-Ke Shih
2026-10-07 8:24 ` Luka Gejak
2026-10-07 8:32 ` Ping-Ke Shih
2026-10-07 8:39 ` Luka Gejak
2026-10-07 8:54 ` Ping-Ke Shih
2026-10-07 10:06 ` Greg KH
2026-10-07 10:35 ` Luka Gejak
2026-10-10 10:18 ` Greg KH
2026-10-12 11:31 ` Luka Gejak
2026-10-14 13:04 ` Greg KH [this message]
2026-10-14 13:13 ` Luka Gejak
2026-10-09 12:40 ` Bitterblue Smith
2026-10-09 15:33 ` Luka Gejak
2026-10-06 13:07 ` [PATCH rtw-next v8 6/7] wifi: rtw88: 8723bs: add the RTL8723BS SDIO bind Luka Gejak
2026-10-06 13:07 ` [PATCH rtw-next v8 7/7] wifi: rtw88: 8723bs: enable building the RTL8723BS driver Luka Gejak
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=2026101452-chapter-friday-dada@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=luka.gejak@linux.dev \
--cc=pbrobinson@gmail.com \
--cc=pkshih@realtek.com \
--cc=rtl8821cerfe2@gmail.com \
--cc=straube.linux@gmail.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.