All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jes Sorensen <Jes.Sorensen@redhat.com>
To: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Xose Vazquez Perez <xose.vazquez@gmail.com>,
	driverdev-devel@linuxdriverproject.org,
	linux-wireless <linux-wireless@vger.kernel.org>,
	Greg KH <gregkh@linuxfoundation.org>,
	Kalle Valo <kvalo@codeaurora.org>
Subject: Re: staging/rtl8723au should be removed
Date: Fri, 23 Oct 2015 10:02:17 -0400	[thread overview]
Message-ID: <wrfjy4ethedy.fsf@redhat.com> (raw)
In-Reply-To: <562A179F.90201@lwfinger.net> (Larry Finger's message of "Fri, 23 Oct 2015 06:18:55 -0500")

Larry Finger <Larry.Finger@lwfinger.net> writes:
> On 10/23/2015 05:44 AM, Xose Vazquez Perez wrote:
>> Hi,
>>
>> Support for RTL8723AU devices was added to
>> wireless-drivers-next.git recently:
>> https://marc.info/?l=linux-wireless&m=144541406318463
>> commit:
>> https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_regs.h?id=26f1fad29ad973b0fb26a9ca3dcb2a73dde781aa
>
> The new driver is remarkable, but removal of the staging driver should
> not be done quite yet.
>
> The new one in the regular tree lacks a number of features. To quote
> from its commit message "This driver is still under development, but
> has proven to be very stable for me. It currently supports station
> mode only. It has support for OFDM and CCK rates. It does lack certain
> features found in the staging driver, such as power management, AMPDU,
> and 40MHz channel support. In addition it does not support AD-HOC, AP,
> and monitor mode support at this point."

I agree - it's too early to delete drivers/staging/rtl8723au at this
point. Just like it's too early to delete drivers/staging/rtl8192u.

There is monitor mode patches in my devel tree now, courtesy of Bruno
Randolf, so one of the features has gotten fixed. I also added 40MHz
support a while ago, maybe I forgot to remove that from the commit
message.

Cheers,
Jes

      reply	other threads:[~2015-10-23 14:02 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-23 10:44 staging/rtl8723au should be removed Xose Vazquez Perez
2015-10-23 11:18 ` Larry Finger
2015-10-23 14:02   ` Jes Sorensen [this message]

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=wrfjy4ethedy.fsf@redhat.com \
    --to=jes.sorensen@redhat.com \
    --cc=Larry.Finger@lwfinger.net \
    --cc=driverdev-devel@linuxdriverproject.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=kvalo@codeaurora.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=xose.vazquez@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.