MIME-Version: 1.0
In-Reply-To: <20171215114427.32059-1-klaus.goger@theobroma-systems.com>
References: <20171215114427.32059-1-klaus.goger@theobroma-systems.com>
From: Philippe Ombredanne <pombredanne@nexb.com>
Date: Fri, 15 Dec 2017 14:45:34 +0100
Message-ID: <CAOFm3uHba4kBD81thpxSpDBsDS4Y5m7cN2jfYt_uT5atdSDmqg@mail.gmail.com>
Subject: Re: [PATCH 0/2] Use SPDX-License-Identifier for rockchip devicetree files
To: Klaus Goger <klaus.goger@theobroma-systems.com>
Cc: linux-rockchip@lists.infradead.org, Heiko Stuebner <heiko@sntech.de>,
        Romain Perier <romain.perier@collabora.com>,
        Arnd Bergmann <arnd@arndb.de>, Andre Przywara <andre.przywara@arm.com>,
        Simon Xue <xxm@rock-chips.com>, Liang Chen <cl@rock-chips.com>,
        Linus Walleij <linus.walleij@linaro.org>,
        Rob Herring <robh+dt@kernel.org>, Icenowy Zheng <icenowy@aosc.io>,
        Shawn Lin <shawn.lin@rock-chips.com>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Huibin Hong <huibin.hong@rock-chips.com>,
        David Wu <david.wu@rock-chips.com>,
        Finley Xiao <finley.xiao@rock-chips.com>,
        Jacob Chen <jacob-chen@iotwrt.com>,
        William Wu <william.wu@rock-chips.com>,
        Brian Norris <briannorris@chromium.org>,
        Jaehoon Chung <jh80.chung@samsung.com>,
        Maxime Ripard <maxime.ripard@free-electrons.com>,
        Douglas Anderson <dianders@chromium.org>,
        Rocky Hao <rocky.hao@rock-chips.com>,
        Will Deacon <will.deacon@arm.com>,
        Jeffy Chen <jeffy.chen@rock-chips.com>,
        Matthias Kaehlcke <mka@chromium.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@vger.kernel.org>,
        Pierre-Hugues Husson <phh@phh.me>,
        Emil Renner Berthing <kernel@esmil.dk>,
        Russell King <linux@armlinux.org.uk>,
        "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@lists.infradead.org>,
        Jianqun Xu <jay.xu@rock-chips.com>,
        Sugar Zhang <sugar.zhang@rock-chips.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Mark Yao <mark.yao@rock-chips.com>,
        "David S. Miller" <davem@davemloft.net>,
        Heinrich Schuchardt <xypron.glpk@gmx.de>,
        Andy Yan <andy.yan@rock-chips.com>, Caesar Wang <wxt@rock-chips.com>,
        Joseph Chen <chenjh@rock-chips.com>,
        Mark Rutland <mark.rutland@arm.com>, Chen-Yu Tsai <wens@csie.org>,
        Kever Yang <kever.yang@rock-chips.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1408
Lines: 34

Klaus,

On Fri, Dec 15, 2017 at 12:44 PM, Klaus Goger
<klaus.goger@theobroma-systems.com> wrote:
> This patch series replaces all the license text in rockchip devicetree
> files text with a proper SPDX-License-Identifier.
> It follows the guidelines submitted[1] by Thomas Gleixner that are not
> yet merged.
>
> These series also fixes the issue with contradicting statements in most
> licenses. The introduction text claims to be GPL or X11[2] but the
> following verbatim copy of the license is actually a MIT[3] license.
> The X11 license includes a advertise clause and trademark information
> related to the X Consortium. As these X Consortium specfic points are
> irrelevant for us we stick with the actuall license text.
>
> [1] https://patchwork.kernel.org/patch/10091607/
> [2] https://spdx.org/licenses/X11.html
> [3] https://spdx.org/licenses/MIT.html

FWIW, the X11 license name was not always something clearly defined.
SPDX calls it clearly MIT which is the most widely accepted name for
the corresponding text. And this is also what we have in Thomas doc
patches that should be the kernel reference.

Also, as a general note, you want to make sure that such as patch set
is not merged by mistake until you have collected an explicit review
or ack from all the copyright holders involved.

May be calling it an "RFC" could be best until you have these acks?

-- 
Cordially
Philippe Ombredanne