From: "Jason A. Donenfeld" Subject: Re: [PATCH net-next v5 04/20] zinc: ChaCha20 x86_64 implementation Date: Wed, 19 Sep 2018 04:14:31 +0200 Message-ID: References: <20180918161646.19105-1-Jason@zx2c4.com> <20180918161646.19105-5-Jason@zx2c4.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: LKML , Netdev , Linux Crypto Mailing List , David Miller , Greg Kroah-Hartman , Samuel Neves , Andrew Lutomirski , Jean-Philippe Aumasson , Andy Polyakov , mingo@redhat.com, X86 ML To: Thomas Gleixner Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org Hi Thomas, On Wed, Sep 19, 2018 at 12:30 AM Thomas Gleixner wrote: > I'm a bit confused by this SOB chain. So above you write that it's from > Andy Polakovs implementation and Samuel did the changes. But here it seems > you are the main author. If Samuel just did some modifications then you > want to use the Co-developed-by tag along with his SOB. Thanks, I'll use that tag. > > Also I'd recommend to add a Originally-by or Based-on-code-from: Andy > Polyakov tag. Both are not formal tags but widely in use for attributions. Great idea. > > > +++ b/lib/zinc/chacha20/chacha20-x86_64-glue.h > > @@ -0,0 +1,100 @@ > > +/* SPDX-License-Identifier: MIT > > Please put that into a separate one liner comment. Also this should be > 'GPL-2.0[+] or MIT' I think. I had that originally, but changed it to just MIT, since MIT is a subset of GPL-2.0. And looking at tree repo, it appears this is what others do too. Jason