Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757085AbdLWQ1C (ORCPT ); Sat, 23 Dec 2017 11:27:02 -0500 Received: from gloria.sntech.de ([95.129.55.99]:59434 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752279AbdLWQ1A (ORCPT ); Sat, 23 Dec 2017 11:27:00 -0500 From: Heiko Stuebner To: Philippe Ombredanne Cc: Emmanuel Vadot , zhangqing@rock-chips.com, Rob Herring , Mark Rutland , Greg Kroah-Hartman , Thomas Gleixner , Kate Stewart , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , LKML Subject: Re: [PATCH] clk: rockchip: Switch dt-binding headers for rk3328 to GPL/X11 Date: Sat, 23 Dec 2017 17:26:51 +0100 Message-ID: <2822025.zCoiRFqhzu@phil> User-Agent: KMail/5.2.3 (Linux/4.13.0-1-amd64; KDE/5.37.0; x86_64; ; ) In-Reply-To: References: <20171223152254.11617-1-manu@freebsd.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1583 Lines: 41 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 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 [...] > > 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