public inbox for linux-crypto@vger.kernel.org
 help / color / mirror / Atom feed
From: Cheah Kok Cheong <thrust73@gmail.com>
To: Philippe Ombredanne <pombredanne@nexb.com>
Cc: steffen.klassert@secunet.com, linux-crypto@vger.kernel.org,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] padata: Remove FSF address
Date: Thu, 21 Dec 2017 05:17:28 +0800	[thread overview]
Message-ID: <20171220211728.GA2383@linux-Precision-WorkStation-T5500> (raw)
In-Reply-To: <CAOFm3uGvuyHcKQ9vvZ1=U3D3Qp8N4npU-JSkWmyr-=9aYv=vzw@mail.gmail.com>

On Wed, Dec 20, 2017 at 09:20:48PM +0100, Philippe Ombredanne wrote:
> On Wed, Dec 20, 2017 at 9:15 PM, Cheah Kok Cheong <thrust73@gmail.com> wrote:
> > Remove FSF address otherwise checkpatch will flag my next patch.
> >
> > Signed-off-by: Cheah Kok Cheong <thrust73@gmail.com>
> > ---
> >  kernel/padata.c | 4 ----
> >  1 file changed, 4 deletions(-)
> >
> > diff --git a/kernel/padata.c b/kernel/padata.c
> > index 57c0074..9d91909 100644
> > --- a/kernel/padata.c
> > +++ b/kernel/padata.c
> > @@ -14,10 +14,6 @@
> >   * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> >   * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
> >   * more details.
> > - *
> > - * You should have received a copy of the GNU General Public License along with
> > - * this program; if not, write to the Free Software Foundation, Inc.,
> > - * 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.
> >   */
> >
> >  #include <linux/export.h>
> > --
> > 2.7.4
> >
> 
> 
> Why not use instead the simpler SPDX ids?
> -- 
> Cordially
> Philippe Ombredanne

Hi Philippe,
  Adding the SPDX id can be the subject of a separate patch.
Believe you are part of the team doing an audit in this matter.
I shall leave adding the SPDX id to you guys the professionals.
Looks like you guys are using script to do this en masse.

But if you insist and our maintainer is agreeable, I can send
another patch to add the SPDX id.

Thanks.
Brgds,
CheahKC 

  reply	other threads:[~2017-12-20 21:17 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-20 20:15 [PATCH 1/2] padata: Remove FSF address Cheah Kok Cheong
2017-12-20 20:15 ` [PATCH 2/2] padata: Remove redundant export.h header Cheah Kok Cheong
2017-12-20 20:20 ` [PATCH 1/2] padata: Remove FSF address Philippe Ombredanne
2017-12-20 21:17   ` Cheah Kok Cheong [this message]
2017-12-21  7:34     ` Philippe Ombredanne
2017-12-21 11:05       ` Cheah Kok Cheong
2017-12-21 11:35         ` [PATCH] padata: add SPDX identifier Cheah Kok Cheong
2017-12-28  6:53           ` Herbert Xu
2017-12-30  8:21             ` Steffen Klassert
2018-01-05 11:14           ` Herbert Xu

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=20171220211728.GA2383@linux-Precision-WorkStation-T5500 \
    --to=thrust73@gmail.com \
    --cc=linux-crypto@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pombredanne@nexb.com \
    --cc=steffen.klassert@secunet.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox