On Thu, Jan 17, 2019 at 10:26:07PM -0200, Fabio Estevam wrote: > On Thu, Jan 17, 2019 at 9:17 PM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote: > > > --- a/include/crypto/morus1280_glue.h > > +++ b/include/crypto/morus1280_glue.h > > @@ -1,15 +1,10 @@ > > -/* SPDX-License-Identifier: GPL-2.0 */ > > +/* SPDX-License-Identifier: GPL-2.0-or-later */ > > Shouldn't this be: /* SPDX-License-Identifier: GPL-2.0+ */ ? > > At least this is the form documented in Documentation/process/license-rules.rst: > > " License identifiers for licenses like [L]GPL with the 'or later' option > are constructed by using a "+" for indicating the 'or later' option.:: > > // SPDX-License-Identifier: GPL-2.0+" Either works, please see LICENSES/preferred/GPL-2.0 Personally, I think we should stick with the "+" version as that is the much more prevalent use in the kernel at the moment, but it's Thomas's call, not mine. thanks, greg k-h