All of lore.kernel.org
 help / color / mirror / Atom feed
From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Aaro Koskinen <aaro.koskinen@iki.fi>,
	Marcus Folkesson <marcus.folkesson@gmail.com>,
	Wim Van Sebroeck <wim@linux-watchdog.org>,
	Joel Stanley <joel@jms.id.au>,
	Nicolas Ferre <nicolas.ferre@microchip.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Ray Jui <rjui@broadcom.com>,
	Scott Branden <sbranden@broadcom.com>,
	bcm-kernel-feedback-list@broadcom.com,
	Eric Anholt <eric@anholt.net>,
	Stefan Wahren <stefan.wahren@i2se.com>,
	Support Opensource <support.opensource@diasemi.com>,
	Baruch Siach <baruch@tkos.co.il>,
	Jimmy Vance <jimmy.vance@hpe.com>,
	Keguang Zhang <keguang.zhang@gmail.com>,
	Tomas Winkler <tomas.winkler@intel.com>,
	Johannes Thumshirn <morbidrsa@gmail.com>,
	Carlo Caione <carlo@caione.org>,
	Kevin Hilman <khilman@baylibre.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Michal Simek <michal.simek@xilinx.com>,
	Vladimir Zapolskiy <vz@mleia.com>,
	Sylvain Lemieux <slemieux.tyco@gmail.com>,
	Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Zwane Mwaikambo <zwanem@gmail.com>,
	Jim Cromie <jim.cromie@gmail.com>, Barry Song <baohua@kernel.org>,
	Patrice Chotard <patrice.chotard@st.com>,
	Maxime Ripard <maxime.ripard@bootlin.com>,
	Chen-Yu Tsai <wens@csie.org>,
	Marc Gonzalez <marc.w.gonzalez@free.fr>,
	Mans Rullgard <mans@mansr.com>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Masahiro Yamada <yamada.masahiro@socionext.com>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Paul Mackerras <paulus@samba.org>,
	Michael Ellerman <mpe@ellerman.id.au>,
	Jun Nie <jun.nie@linaro.org>, Baoyou Xie <baoyou.xie@linaro.org>,
	Shawn Guo <shawnguo@kernel.org>,
	linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org,
	Philippe Ombredanne <pombredanne@nexb.com>
Subject: Re: [PATCH v5] watchdog: add SPDX identifiers for watchdog subsystem
Date: Sun, 4 Mar 2018 20:33:24 -0500	[thread overview]
Message-ID: <20180305013308.GA11983@sophia> (raw)
In-Reply-To: <9db48467-7727-87b7-5c55-9c53c46a901f@roeck-us.net>

On Sun, Mar 04, 2018 at 05:11:00PM -0800, Guenter Roeck wrote:
>On 03/04/2018 03:53 PM, Aaro Koskinen wrote:
>> Hi,
>> 
>> On Thu, Mar 01, 2018 at 09:54:50AM +0100, Marcus Folkesson wrote:
>>> diff --git a/drivers/watchdog/retu_wdt.c b/drivers/watchdog/retu_wdt.c
>>> index 39cd51df2ffc..258dfcf9cbda 100644
>>> --- a/drivers/watchdog/retu_wdt.c
>>> +++ b/drivers/watchdog/retu_wdt.c
>>> @@ -1,3 +1,4 @@
>>> +// SPDX-License-Identifier: GPL-2.0+
>> 
>> Why "+" and not just GPL-2.0?
>> 
>
>My non-attorney feedback, which doesn't really mean anything, is that
>MODULE_LICENSE says GPL. There appears to be no indication in the file
>suggesting that the license would be limited or restricted to GPL v2.
>
>This makes me wonder if we should drop this patch entirely. I am not an
>attorney, and if we start getting into license disputes I rather revert
>to doing nothing than getting into trouble by signing off to something
>that isn't legally clean.
>
>Guenter

Regardless of the legal aspect of these changes, I believe the ultimate
decision to change license text should be handled by the authors of the
code. License text is a statement of intention by the authors' for the
use of their provided code. A change of license text thus is by its very
nature -- even if meant to be compatible -- a change of the authors'
intention. Therefore, it would make sense that license text changes
should be provided directly from the authors.

Rather than seeking ACKs on an encompassing patch such as this, I would
suggest that a more appropriate way to get these SPDX changes merged is
to request the authors of these pieces of code to submit the relevant
license text change patches themselves directly. This would remove the
ambiquity of what the authors' intentions actually are, as well as ease
the legal fears some have with the current attempt.

William Breathitt Gray

  parent reply	other threads:[~2018-03-05  1:33 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-01  8:54 [PATCH v5] watchdog: add SPDX identifiers for watchdog subsystem Marcus Folkesson
     [not found] ` <20180304235319.55gulkoxk4vcaoug@darkstar.musicnaut.iki.fi>
     [not found]   ` <9db48467-7727-87b7-5c55-9c53c46a901f@roeck-us.net>
2018-03-05  1:33     ` William Breathitt Gray [this message]
2018-03-05 19:30     ` Aaro Koskinen
2018-03-05 22:08     ` Marcus Folkesson
2018-03-06 10:13       ` Alexandre Belloni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180305013308.GA11983@sophia \
    --to=vilhelm.gray@gmail.com \
    --cc=aaro.koskinen@iki.fi \
    --cc=alexandre.belloni@bootlin.com \
    --cc=baohua@kernel.org \
    --cc=baoyou.xie@linaro.org \
    --cc=baruch@tkos.co.il \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=benh@kernel.crashing.org \
    --cc=carlo@caione.org \
    --cc=eric@anholt.net \
    --cc=f.fainelli@gmail.com \
    --cc=jim.cromie@gmail.com \
    --cc=jimmy.vance@hpe.com \
    --cc=joel@jms.id.au \
    --cc=jonathanh@nvidia.com \
    --cc=jun.nie@linaro.org \
    --cc=keguang.zhang@gmail.com \
    --cc=kgene@kernel.org \
    --cc=khilman@baylibre.com \
    --cc=krzk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=mans@mansr.com \
    --cc=marc.w.gonzalez@free.fr \
    --cc=marcus.folkesson@gmail.com \
    --cc=matthias.bgg@gmail.com \
    --cc=maxime.ripard@bootlin.com \
    --cc=michal.simek@xilinx.com \
    --cc=morbidrsa@gmail.com \
    --cc=mpe@ellerman.id.au \
    --cc=nicolas.ferre@microchip.com \
    --cc=patrice.chotard@st.com \
    --cc=paulus@samba.org \
    --cc=pombredanne@nexb.com \
    --cc=rjui@broadcom.com \
    --cc=sbranden@broadcom.com \
    --cc=shawnguo@kernel.org \
    --cc=slemieux.tyco@gmail.com \
    --cc=stefan.wahren@i2se.com \
    --cc=support.opensource@diasemi.com \
    --cc=thierry.reding@gmail.com \
    --cc=tomas.winkler@intel.com \
    --cc=vz@mleia.com \
    --cc=wens@csie.org \
    --cc=wim@linux-watchdog.org \
    --cc=yamada.masahiro@socionext.com \
    --cc=zwanem@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.