diff mbox

staging: refresh TODO for rtl8723au

Message ID 1458321324-16906-1-git-send-email-xose.vazquez@gmail.com (mailing list archive)
State Not Applicable
Delegated to: Kalle Valo
Headers show

Commit Message

Xose Vazquez Perez March 18, 2016, 5:15 p.m. UTC
People should not waste time and energy working on this staging driver.
A replacement(rtl8xxxu) using the kernel wireless stack already was merged
in the 4.3 kernel.

Cc: Jes Sorensen <Jes.Sorensen@redhat.com>
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Linux Driver Project Developer List <driverdev-devel@linuxdriverproject.org>
Cc: Linux Wireless <linux-wireless@vger.kernel.org>
Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
---
 drivers/staging/rtl8723au/TODO | 3 +++
 1 file changed, 3 insertions(+)

Comments

Jes Sorensen March 18, 2016, 5:42 p.m. UTC | #1
Xose Vazquez Perez <xose.vazquez@gmail.com> writes:
> People should not waste time and energy working on this staging driver.
> A replacement(rtl8xxxu) using the kernel wireless stack already was merged
> in the 4.3 kernel.
>
> Cc: Jes Sorensen <Jes.Sorensen@redhat.com>
> Cc: Larry Finger <Larry.Finger@lwfinger.net>
> Cc: Kalle Valo <kvalo@codeaurora.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Linux Driver Project Developer List <driverdev-devel@linuxdriverproject.org>
> Cc: Linux Wireless <linux-wireless@vger.kernel.org>
> Signed-off-by: Xose Vazquez Perez <xose.vazquez@gmail.com>
> ---
>  drivers/staging/rtl8723au/TODO | 3 +++
>  1 file changed, 3 insertions(+)

Acked-by: Jes Sorensen <Jes.Sorensen@redhat.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joe Perches March 18, 2016, 6:28 p.m. UTC | #2
On Fri, 2016-03-18 at 13:42 -0400, Jes Sorensen wrote:
> Xose Vazquez Perez <xose.vazquez@gmail.com> writes:
> > People should not waste time and energy working on this staging driver.
> > A replacement(rtl8xxxu) using the kernel wireless stack already was merged
> > in the 4.3 kernel.
[]
> >  drivers/staging/rtl8723au/TODO | 3 +++
> Acked-by: Jes Sorensen <Jes.Sorensen@redhat.com>

why not git rm drivers/staging/rtl8723au/ ?

How about changing MAINTAINERS from:

STAGING - REALTEK RTL8723U WIRELESS DRIVER
M:	Larry Finger <Larry.Finger@lwfinger.net>
M:	Jes Sorensen <Jes.Sorensen@redhat.com>
L:	linux-wireless@vger.kernel.org
S:	Maintained
F:	drivers/staging/rtl8723au/

to

S:	Obsolete

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jes Sorensen March 19, 2016, 3:58 a.m. UTC | #3
Joe Perches <joe@perches.com> writes:
> On Fri, 2016-03-18 at 13:42 -0400, Jes Sorensen wrote:
>> Xose Vazquez Perez <xose.vazquez@gmail.com> writes:
>> > People should not waste time and energy working on this staging driver.
>> > A replacement(rtl8xxxu) using the kernel wireless stack already was merged
>> > in the 4.3 kernel.
> []
>> >  drivers/staging/rtl8723au/TODO | 3 +++
>> Acked-by: Jes Sorensen <Jes.Sorensen@redhat.com>
>
> why not git rm drivers/staging/rtl8723au/ ?

Because you don't just pull the rug out under people. I plan to mark it
obsolete and then remove it in a follow-on release.

Jes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joe Perches March 19, 2016, 4:09 a.m. UTC | #4
On Fri, 2016-03-18 at 23:58 -0400, Jes Sorensen wrote:
> Joe Perches <joe@perches.com> writes:
> > On Fri, 2016-03-18 at 13:42 -0400, Jes Sorensen wrote:
> > > Xose Vazquez Perez <xose.vazquez@gmail.com> writes:
> > > > 
> > > > People should not waste time and energy working on this staging driver.
> > > > A replacement(rtl8xxxu) using the kernel wireless stack already was merged
> > > > in the 4.3 kernel.
> > []
> > > 
> > > > 
> > > >  drivers/staging/rtl8723au/TODO | 3 +++
> > > Acked-by: Jes Sorensen <Jes.Sorensen@redhat.com>
> > why not git rm drivers/staging/rtl8723au/ ?
> Because you don't just pull the rug out under people. I plan to mark it
> obsolete and then remove it in a follow-on release.

Duh.

I note you removed the obsolete suggestion which
is probably more valuable then updating the TODO.

Updating the TODO to something like:

Actual bug fixes only, no other changes accepted.
This driver is going to be removed by 4.8 or so

is also probably more valuable than keeping all the
current items and just adding a note that another
driver exists.

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/staging/rtl8723au/TODO b/drivers/staging/rtl8723au/TODO
index f5d57d3..42b86e4 100644
--- a/drivers/staging/rtl8723au/TODO
+++ b/drivers/staging/rtl8723au/TODO
@@ -9,5 +9,8 @@  TODO:
 - merge Realtek's bugfixes and new features into the driver
 - switch to use MAC80211
 
+A mac80211 driver for this hardware already was integrated at
+drivers/net/wireless/realtek/rtl8xxxu/
+
 Please send any patches to Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
 Jes Sorensen <Jes.Sorensen@redhat.com>, and Larry Finger <Larry.Finger@lwfinger.net>.