public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Peter Wu <peter@lekensteyn.nl>,
	John W Linville <linville@tuxdriver.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Chaoming_Li <chaoming_li@realsil.com.cn>,
	linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org,
	lekensteyn@gmail.com
Subject: Re: [PATCH 0/3] rtlwifi (and staging rtl8821ae) cleanups
Date: Thu, 20 Mar 2014 14:48:22 -0500	[thread overview]
Message-ID: <532B4606.3030303@lwfinger.net> (raw)
In-Reply-To: <1395341544-21464-1-git-send-email-peter@lekensteyn.nl>

On 03/20/2014 01:52 PM, Peter Wu wrote:
> Hi,
>
> The first patch of these series was NAKed some weeks ago because it would
> conflict with the new rtl8723be driver. Since those changes are in wireless-next
> now, I have adjusted that patch to take care of rtl8723be.
>
> The other two patches to the staging driver is similar to the rtlwifi changes.
> I am not sure who to send these two patches to. Greg is the staging maintainer,
> but John takes wireless patches, so I have included both of you.
>
> All patches are compile-tested on top of v3.14-rc2-938-g67b3bd4.
>
> I also found the following (unrelated to my patches) warning while compiling the
> staging driver, perhaps it needs a closer look?

Peter,

Your Patch 1/3 applies cleanly and I think it is OK, but if you are going to do 
cleanups on the wireless tree, you should be working from the latest version of 
wireless-testing git repo. We are on 3.14-rc7 and there are several recent 
commits that might have conflicted.

As for the changes to staging/rtl8821ae (No. 2 & 3), once Linus starts the merge 
that will precede the release of 3.15-rc2, the entire code for rtl8821ae will be 
replaced. A cleaned-up version was nearly ready for the regular wireless tree in 
time to make 3.15, but it did not get enough testing. As a result, that driver 
will be in testing for one more cycle. By 3.16, it should reside in the regular 
tree and be out of staging.

Pushing patches that affect both the regular tree and staging is complicated and 
should be avoided whenever possible.

Larry




  parent reply	other threads:[~2014-03-20 19:48 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-20 18:52 [PATCH 0/3] rtlwifi (and staging rtl8821ae) cleanups Peter Wu
2014-03-20 18:52 ` [PATCH v2 1/3] rtlwifi: remove unused allow_all_destaddr functions Peter Wu
2014-03-20 18:52 ` [PATCH 2/3] staging/rtl8821ae: avoid accessing RCR directly Peter Wu
2014-03-20 18:52 ` [PATCH 3/3] staging/rtl8821ae: remove unused allow_all_destaddr function Peter Wu
2014-03-20 19:48 ` Larry Finger [this message]
2014-03-20 20:05   ` [PATCH 0/3] rtlwifi (and staging rtl8821ae) cleanups Greg Kroah-Hartman
2014-03-20 21:01   ` Peter Wu

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=532B4606.3030303@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=chaoming_li@realsil.com.cn \
    --cc=gregkh@linuxfoundation.org \
    --cc=lekensteyn@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=peter@lekensteyn.nl \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox