2018-03-01 10:12:27

by Steve Twiss

[permalink] [raw]
Subject: RE: [PATCH v4] watchdog: add SPDX identifiers for watchdog subsystem

On 28 February 2018 15:02, Marcus Folkesson wrote:

> Subject: [PATCH v4] watchdog: add SPDX identifiers for watchdog subsystem
>
> - Add SPDX identifier
> - Remove boiler plate license text
> - If MODULE_LICENSE and boiler plate does not match, go for boiler plate license

Acked-by: Steve Twiss <[email protected]>

Hi Marcus,

Acked for the files below.

Regards,
Steve

[...]

> drivers/watchdog/da9052_wdt.c | 6 +---
> drivers/watchdog/da9055_wdt.c | 6 +---
> drivers/watchdog/da9062_wdt.c | 10 +------
> drivers/watchdog/da9063_wdt.c | 5 +---

[...]

> diff --git a/drivers/watchdog/da9052_wdt.c
> b/drivers/watchdog/da9052_wdt.c
> index d6d5006efa71..e263bad99574 100644
> --- a/drivers/watchdog/da9052_wdt.c
> +++ b/drivers/watchdog/da9052_wdt.c
> @@ -1,3 +1,4 @@
> +// SPDX-License-Identifier: GPL-2.0+
> /*
> * System monitoring driver for DA9052 PMICs.
> *
> @@ -5,11 +6,6 @@
> *
> * Author: Anthony Olech <[email protected]>
> *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - *
> */
>
> #include <linux/module.h>
> diff --git a/drivers/watchdog/da9055_wdt.c
> b/drivers/watchdog/da9055_wdt.c
> index 50bdd1022186..26a5b2984094 100644
> --- a/drivers/watchdog/da9055_wdt.c
> +++ b/drivers/watchdog/da9055_wdt.c
> @@ -1,3 +1,4 @@
> +// SPDX-License-Identifier: GPL-2.0+
> /*
> * System monitoring driver for DA9055 PMICs.
> *
> @@ -5,11 +6,6 @@
> *
> * Author: David Dajun Chen <[email protected]>
> *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - *
> */
>
> #include <linux/module.h>
> diff --git a/drivers/watchdog/da9062_wdt.c
> b/drivers/watchdog/da9062_wdt.c
> index 814dff6045a4..a001782bbfdb 100644
> --- a/drivers/watchdog/da9062_wdt.c
> +++ b/drivers/watchdog/da9062_wdt.c
> @@ -1,16 +1,8 @@
> +// SPDX-License-Identifier: GPL-2.0+
> /*
> * Watchdog device driver for DA9062 and DA9061 PMICs
> * Copyright (C) 2015 Dialog Semiconductor Ltd.
> *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License
> - * as published by the Free Software Foundation; either version 2
> - * of the License, or (at your option) any later version.
> - *
> - * 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/kernel.h>
> diff --git a/drivers/watchdog/da9063_wdt.c
> b/drivers/watchdog/da9063_wdt.c
> index 2a20fc163ed0..b17ac1bb1f28 100644
> --- a/drivers/watchdog/da9063_wdt.c
> +++ b/drivers/watchdog/da9063_wdt.c
> @@ -1,3 +1,4 @@
> +// SPDX-License-Identifier: GPL-2.0+
> /*
> * Watchdog driver for DA9063 PMICs.
> *
> @@ -5,10 +6,6 @@
> *
> * Author: Mariusz Wojtasik <[email protected]>
> *
> - * This program is free software; you can redistribute it and/or modify it
> - * under the terms of the GNU General Public License as published by the
> - * Free Software Foundation; either version 2 of the License, or (at your
> - * option) any later version.
> */
>
> #include <linux/kernel.h>