linux-kernel - Re: [PATCH] clk: rockchip: Switch dt-binding headers for rk3328 to GPL/X11

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2822025.zCoiRFqhzu@phil>
Date:   Sat, 23 Dec 2017 17:26:51 +0100
From:   Heiko Stuebner <heiko@...ech.de>
To:     Philippe Ombredanne <pombredanne@...b.com>
Cc:     Emmanuel Vadot <manu@...ebsd.org>, zhangqing@...k-chips.com,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Kate Stewart <kstewart@...uxfoundation.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] clk: rockchip: Switch dt-binding headers for rk3328 to GPL/X11

Hi,

Am Samstag, 23. Dezember 2017, 17:19:58 CET schrieb Philippe Ombredanne:
> Dear Emmanuel,
> 
> On Sat, Dec 23, 2017 at 4:22 PM, Emmanuel Vadot <manu@...ebsd.org> wrote:
> > Since those files are also needed kernel side, switch their licences
> > to GPL/X11 so it can be used in BSD kernels.
> >
> > Signed-off-by: Emmanuel Vadot <manu@...ebsd.org>

[...]

> > diff --git a/include/dt-bindings/power/rk3328-power.h b/include/dt-bindings/power/rk3328-power.h
> > index 02e3d7fc1cce..301f30967b39 100644
> > --- a/include/dt-bindings/power/rk3328-power.h
> > +++ b/include/dt-bindings/power/rk3328-power.h
> > @@ -1,4 +1,4 @@
> > -/* SPDX-License-Identifier: GPL-2.0 */
> > +/* SPDX-License-Identifier: GPL-2.0 or X11 */
> >  #ifndef __DT_BINDINGS_POWER_RK3328_POWER_H__
> >  #define __DT_BINDINGS_POWER_RK3328_POWER_H__
> 
> What you call X11 is called MIT in SPDX and in Thomas doc patches [1],
> e.g. this tag is supposed to match the eyes-poking long legalese
> above, this should be instead:
> 
> +/* SPDX-License-Identifier: (GPL-2.0+ or MIT) */
> 
> Finally if the goal of this proposed license update is usage in
> FreeBSD and other BSD kernels, why use MIT as a second license? Would
> not a BSD be better and avoid license inflation on the BSD side?

I think it is likely meant to match the license used on the devicetree
files themselfs. For whatever reason the existing combination of
GPL+MIT was the preferred one, so the license inflation is already there
and it might be best to keep to the same combination for the headers
needed by those devicetree files?

Heiko