Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210204124357.3817-1-brgl@bgdev.pl> <YBv61eNnVksYq9mr@kroah.com>
 <CAMpxmJXcVOHbozzhQQW2GPg-6ivCWL3pQmyC-oU_YW5NX=gK3w@mail.gmail.com>
 <YBwGHWSV1/ZZm7u/@kroah.com> <CAMpxmJUof3DN2YHBOQm1Z-hrtS1cj6in5CAHowce++36XtqsnA@mail.gmail.com>
 <YBwSuAjNu4FtXSn/@kroah.com>
In-Reply-To: <YBwSuAjNu4FtXSn/@kroah.com>
From:   Bartosz Golaszewski <brgl@bgdev.pl>
Date:   Wed, 10 Feb 2021 14:05:27 +0100
Message-ID: <CAMRc=MeBFCdcBWnm4nWTM3E7AnVhLmxQz_1C_YF3smA6ffXrDA@mail.gmail.com>
Subject: Re: [PATCH] gpio: uapi: use the preferred SPDX license identifier
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Bartosz Golaszewski <bgolaszewski@baylibre.com>,
        Kent Gibson <warthog618@gmail.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        linux-gpio <linux-gpio@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, Feb 4, 2021 at 4:29 PM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> On Thu, Feb 04, 2021 at 04:17:51PM +0100, Bartosz Golaszewski wrote:
> > On Thu, Feb 4, 2021 at 3:35 PM Greg Kroah-Hartman
> > <gregkh@linuxfoundation.org> wrote:
> > >
> > > On Thu, Feb 04, 2021 at 03:15:50PM +0100, Bartosz Golaszewski wrote:
> > > > On Thu, Feb 4, 2021 at 2:47 PM Greg Kroah-Hartman
> > > > <gregkh@linuxfoundation.org> wrote:
> > > > >
> > > > > On Thu, Feb 04, 2021 at 01:43:57PM +0100, Bartosz Golaszewski wrote:
> > > > > > From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > > > > >
> > > > > > GPL-2.0 license identifier is deprecated, let's use the preferred
> > > > > > identifier: GPL-2.0-only.
> > > > > >
> > > > > > Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > > > > > ---
> > > > > > Hi Kent, Greg,
> > > > > >
> > > > > > I started working on making libgpiod licensing reuse-compliant and noticed
> > > > > > that the reuse-tool is telling me that the GPL-2.0 SPDX identifier in the
> > > > > > GPIO uapi header is deprecated. Since I'm required to copy the header
> > > > > > verbatim into libgpiod's repository, I think we need to fix that at source
> > > > > > first.
> > > > > >
> > > > > >  include/uapi/linux/gpio.h | 2 +-
> > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > >
> > > > > > diff --git a/include/uapi/linux/gpio.h b/include/uapi/linux/gpio.h
> > > > > > index e4eb0b8c5cf9..3e01ededbf36 100644
> > > > > > --- a/include/uapi/linux/gpio.h
> > > > > > +++ b/include/uapi/linux/gpio.h
> > > > > > @@ -1,4 +1,4 @@
> > > > > > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
> > > > > > +/* SPDX-License-Identifier: GPL-2.0-only WITH Linux-syscall-note */
> > > > >
> > > > > No, there is no need to convert the kernel to the "latest" spdx level,
> > > > > when we started out there was no "-only" nonsense (hint no other license
> > > > > has that crud), and "GPL-2.0" is a totally valid summary of the license.
> > > > >
> > > > > So please don't go changing it all in-kernel, that way lies madness.
> > > > > Let's finish fixing up ALL kernel files before worrying about what SPDX
> > > > > "version" we are at.
> > > > >
> > > >
> > > > But then, the reuse script is telling me:
> > > >
> > > >     Unfortunately, your project is not compliant with version 3.0 of
> > > > the REUSE Specification :-(
> > > >
> > > > because I'm using a deprecated license. :(
> > >
> > > Go yell at the REUSE people, there's no requirement to have the most
> > > recent version of SPDX, is there?   :)
> > >
> > > And if that's the only thing wrong with the project when running 'reuse
> > > lint' then you should be happy, no one will complain at all.
> > >
> > > thanks,
> > >
> > > greg k-h
> >
> > Ok, I get it but let me try one last time: there's absolutely no harm
> > in merging this patch for the next release, is there? Currently there
> > are around 15000 instances of "GPL-2.0-only" in the kernel vs 30000
> > "GPL-2.0" - so the former is not that uncommon.
>
> It's up to the owner/maintainer of this file in the end, but I don't
> want to see patches doing this for many more files, as it's pointless
> churn as far as the kernel is concerned.
>

Since it's up to me to decide - I'll apply this but I'll clarify in
the commit message why user-space may want this change.

Bart