Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <20171102151629.GA22281@kroah.com>
References: <20171102151629.GA22281@kroah.com>
From:   Rob Herring <rob.herring@linaro.org>
Date:   Wed, 8 Nov 2017 17:07:46 -0600
Message-ID: <CABGGiswvTExoeQHro_ctpTEhdxf6_VQXdZJDbDvBhG6QndG04A@mail.gmail.com>
Subject: Re: [GIT PULL] License cleanup: add SPDX license identifiers to some
 kernel files
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     Linus Torvalds <torvalds@linux-foundation.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Kate Stewart <kstewart@linuxfoundation.org>,
        Philippe Ombredanne <pombredanne@nexb.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Nov 2, 2017 at 10:16 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> [resend without the full diffstat as lkml and some email systems didn't
>  like to see emails with 12k lines...]
>
> Hi,
>
> As discussed at the Maintainers Summit last week, here is a pull request
> that adds some SPDX license identifiers to three different classes of
> files:
>         - files with no license identifiers at all, but not uapi files
>         - uapi files with no license identifiers at all
>         - uapi files with existing license identifiers
>
> This "only" touched 1/6 of the files in the tree.  The remaining files
> will be dealt with on a subsystem-by-subsystem basis over the next few
> kernel releases.
>
> The full methodology of how these files were determined, and how the
> work was done is down below in the signed tag, and in the first commit
> of the series.
>
> These patches have a "new" timestamp, a few hours old, only because we
> have revised and rewritten the changelog text many times based on lots
> of people's inputs (lawyers included.)  The patches themselves are not
> "new" at all and were auto-generated as described below and are based on
> 4.14-rc6.
>
> Note, we had to use /* */ as the comment marker for the .h files, as
> there are just too many .h files being included into .S files to be able
> to try to identify which is which, so we could not use //, unlike the .c
> files.
>
> These have been through 0-day testing with no reported problems, as well
> as my build system and Thomas's build system.

I have some concerns about adding the SPDX tag on the dts/dtsi files.
These files are generally either GPL2 or dual GPL/MIT. The license
should normally be decided per platform and generally we don't have
cross-platform includes. So I'd think there could be some cases where
the intent was to match the rest of the platform's dts files, but the
license was omitted by mistake.

In any case, the platform maintainers should have a chance to comment
on their platforms and it seems this was rushed.

Rob

From 1582981634022725349@xxx Thu Nov 02 18:55:27 +0000 2017
X-GM-THRID: 1582967908286062949
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread