[PATCH v2] watchdog: add SPDX identifiers for watchdog subsystem

Guenter Roeck linux at roeck-us.net
Tue Feb 20 07:13:43 PST 2018


On 02/20/2018 05:21 AM, Marcus Folkesson wrote:
> Hello William,
> 
> On Tue, Feb 20, 2018 at 07:49:55AM -0500, William Breathitt Gray wrote:
>> On Tue, Feb 20, 2018 at 11:45:31AM +0100, Marcus Folkesson wrote:
>>> - Add SPDX identifier
>>> - Remove boiler plate license text
>>> - If MODULE_LICENSE and boiler plate does not match, go for boiler plate
>>>   license
>>>
>>> Signed-off-by: Marcus Folkesson <marcus.folkesson at gmail.com>
>>> Acked-by: Adam Thomson <Adam.Thomson.Opensource at diasemi.com>
>>> Acked-by: Charles Keepax <ckeepax at opensource.cirrus.com>
>>> Acked-by: Nicolas Ferre <nicolas.ferre at microchip.com>
>>> Acked-by: Michal Simek <michal.simek at xilinx.com>
>>> ---
>>>
>>> Notes:
>>>     v2:
>>>     	- Put back removed copyright texts for meson_gxbb_wdt and coh901327_wdt
>>>     	- Change to BSD-3-Clause for meson_gxbb_wdt
>>>     v1: Please have an extra look at meson_gxbb_wdt.c
>>
>> [...]
>>
>>> diff --git a/drivers/watchdog/ebc-c384_wdt.c b/drivers/watchdog/ebc-c384_wdt.c
>>> index 2170b275ea01..c173b6f5c866 100644
>>> --- a/drivers/watchdog/ebc-c384_wdt.c
>>> +++ b/drivers/watchdog/ebc-c384_wdt.c
>>> @@ -1,15 +1,8 @@
>>> +// SPDX-License-Identifier: GPL-2.0
>>> /*
>>>   * Watchdog timer driver for the WinSystems EBC-C384
>>>   * Copyright (C) 2016 William Breathitt Gray
>>>   *
>>> - * This program is free software; you can redistribute it and/or modify
>>> - * it under the terms of the GNU General Public License, version 2, as
>>> - * published by the Free Software Foundation.
>>> - *
>>> - * This program is distributed in the hope that it will be useful, but
>>> - * WITHOUT ANY WARRANTY; without even the implied warranty of
>>> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>>> - * General Public License for more details.
>>>   */
>>> #include <linux/device.h>
>>> #include <linux/dmi.h>
> 
> Thank you for your feedback!
>>
>> I have no problem with adding a SPDX line to the top of this file, but
>> use "SPDX-License-Identifier: GPL-2.0-only" as I was very intentional
>> with the selection of GPL version 2 only when I published this code.
> 
> SPDX-License-Identifier: GPL-2.0
> Is GPL-2.0 only [1], so it respects your choice of license.
> 
>>
>> Furthermore, please do not remove the existing copyright text; although
> 

It is not a matter if you CAN keep a copyright. You MUST NOT remove a copyright.
As long as you do, the series is

Nacked-by: Guenter Roeck <linux at roeck-us.net>

Guenter

> The copyright text:
> 
>   Copyright (C) 2016 William Breathitt Gray
> 
> Is still in the file.
> 
> 
>> it's just boilerplate for some, I was careful with the selection of
>> these words, and I worry the SPDX line only -- despite its useful
>> conciseness -- may lead to misunderstandings about my intentioned
>> license for this code.
> 
> I'm not sure I understand your concerns here, the SPDX identifier is
> a shorthand for the GPL 2.0 only license. See [1] - Linux kernel licensing rules.
> 
> One of the biggest benefits with SPDX identifier is that it is hard to verify
> boiler plate licenses due to formatting, types, different formulations and so on.
> 
> If still worrying, I think we could keep the license text as
> well.
> 
>>
>> For the time being, I can't Ack this patch with the changes it makes
>> currently:
>>
>>          Nacked-by: William Breathitt Gray <vilhelm.gray at gmail.com>
>>
>> William Breathitt Gray
> 
> Please let me know if I got you wrong at some point.
> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
> 
> Best regards
> Marcus Folkesson
> 




More information about the Linux-mediatek mailing list