From: Philippe Ombredanne Subject: Re: [PATCH 1/2] padata: Remove FSF address Date: Thu, 21 Dec 2017 08:34:37 +0100 Message-ID: References: <1513800921-12452-1-git-send-email-thrust73@gmail.com> <20171220211728.GA2383@linux-Precision-WorkStation-T5500> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: steffen.klassert@secunet.com, linux-crypto@vger.kernel.org, LKML To: Cheah Kok Cheong Return-path: Received: from mail-wm0-f65.google.com ([74.125.82.65]:34887 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750965AbdLUHfT (ORCPT ); Thu, 21 Dec 2017 02:35:19 -0500 Received: by mail-wm0-f65.google.com with SMTP id f9so14007100wmh.0 for ; Wed, 20 Dec 2017 23:35:18 -0800 (PST) In-Reply-To: <20171220211728.GA2383@linux-Precision-WorkStation-T5500> Sender: linux-crypto-owner@vger.kernel.org List-ID: Dear CheahKC, On Wed, Dec 20, 2017 at 10:17 PM, Cheah Kok Cheong wrote: > 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 wrote: >> > Remove FSF address otherwise checkpatch will flag my next patch. >> > >> > Signed-off-by: Cheah Kok Cheong >> > --- >> > 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 >> > -- >> > 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. If you could work it out directly that would be best. There are a lot of files in the kernel and each one needs a careful review even with best license detection script (which . So tagging files on the go when they are updated is the best way to help! Thank you. -- Cordially Philippe Ombredanne