From: Kalle Valo <kvalo@qca.qualcomm.com>
To: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
Chaoming Li <chaoming_li@realsil.com.cn>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Fengguang Wu <fengguang.wu@intel.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
Network Development <netdev@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] net: wireless: rtlwifi: btcoexist: halbtc8821a2ant: Remove some unused functions
Date: Mon, 5 Jan 2015 12:28:48 +0200 [thread overview]
Message-ID: <87d26t34in.fsf@kamboji.qca.qualcomm.com> (raw)
In-Reply-To: <CAKXHbyOYO29uKA127qyYrCfpgQ2zoqBS9rAJME513oZ8JVdOug@mail.gmail.com> (Rickard Strandqvist's message of "Sat, 3 Jan 2015 13:03:02 +0100")
Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes:
> I know that you have been told that including "net: wireless:" in the
> subject line is discouraged. Please do not do this again. The staging
> directory is different as GregKH uses "staging:" in the subject to route
> patches, but wireless does not.
>
> As to the patch, NACK for the simple reason that I am currently working on
> a number of changes to btcoexist. Some of these routines may end up being
> removed, but others will not. Having your patch remove them, and one of
> mine adding them back just constitutes a lot of churning of the source. In
> addition, it greatly increases the probability of the source trees becoming
> unsynchronized and getting merge conflicts.
>
> I do not recognize that there has been no diskution on the subject of "net:
> wireless:"
I mentioned about this in one your earlier patches, but I guess you
didn't get the mail because I got a bounce:
https://lkml.org/lkml/2014/12/24/222
--
Kalle Valo
next prev parent reply other threads:[~2015-01-05 10:29 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-02 20:26 [PATCH] net: wireless: rtlwifi: btcoexist: halbtc8821a2ant: Remove some unused functions Rickard Strandqvist
2015-01-03 6:05 ` Larry Finger
[not found] ` <CAKXHbyOYO29uKA127qyYrCfpgQ2zoqBS9rAJME513oZ8JVdOug@mail.gmail.com>
2015-01-05 1:45 ` Larry Finger
2015-01-05 4:05 ` Julian Calaby
2015-01-05 10:28 ` Kalle Valo [this message]
2015-01-06 18:14 ` Kalle Valo
2015-01-09 17:23 ` Rickard Strandqvist
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=87d26t34in.fsf@kamboji.qca.qualcomm.com \
--to=kvalo@qca.qualcomm.com \
--cc=Larry.Finger@lwfinger.net \
--cc=chaoming_li@realsil.com.cn \
--cc=fengguang.wu@intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=rickard_strandqvist@spectrumdigital.se \
/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;
as well as URLs for NNTP newsgroup(s).