From: Kalle Valo <kvalo@codeaurora.org>
To: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
Brett Rudley <brudley@broadcom.com>,
Arend van Spriel <arend@broadcom.com>,
Hante Meuleman <meuleman@broadcom.com>,
Fabian Frederick <fabf@skynet.be>,
"linux-wireless\@vger.kernel.org"
<linux-wireless@vger.kernel.org>,
brcm80211-dev-list@broadcom.com,
Network Development <netdev@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions
Date: Mon, 05 Jan 2015 12:49:59 +0200 [thread overview]
Message-ID: <871tn933jc.fsf@kamboji.qca.qualcomm.com> (raw)
In-Reply-To: <CAKXHbyN6LH_pFw1uoLXRzgJv-NcjbTLJvMZXfwYXDwuk9QDszA@mail.gmail.com> (Rickard Strandqvist's message of "Sun, 4 Jan 2015 13:43:59 +0100")
Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes:
> As I hope you can see I have made some changes regarding the
> subject-line. Thought it was an advantage to be able to see which file
> I actually removed something from. There seems to be a big focus on
> getting right on subject-line right in recent weeks.
>
> I wonder why there is a script that takes a file name, and respond
> with an appropriate subject line?
I don't think you can really automate this as some drivers do this a bit
differently. You always need to manually check the commit log.
> But ok, I change my script accordingly. Should I submit the patch again?
Yes, please resubmit.
--
Kalle Valo
next prev parent reply other threads:[~2015-01-05 10:50 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-04 0:47 [PATCH] brcm80211: brcmsmac: dma: Remove some unused functions Rickard Strandqvist
2015-01-04 6:21 ` Larry Finger
2015-01-04 12:43 ` Rickard Strandqvist
2015-01-05 10:49 ` Kalle Valo [this message]
2015-01-05 11:06 ` Arend van Spriel
2015-01-06 23:33 ` Rickard Strandqvist
2015-01-07 6:29 ` Julia Lawall
2015-01-07 8:58 ` Arend van Spriel
2015-01-09 17:58 ` Rickard Strandqvist
2015-01-07 8:57 ` Arend van Spriel
2015-01-05 10:34 ` Kalle Valo
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=871tn933jc.fsf@kamboji.qca.qualcomm.com \
--to=kvalo@codeaurora.org \
--cc=Larry.Finger@lwfinger.net \
--cc=arend@broadcom.com \
--cc=brcm80211-dev-list@broadcom.com \
--cc=brudley@broadcom.com \
--cc=fabf@skynet.be \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=meuleman@broadcom.com \
--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).