linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] hwmon: (ads7871) Switch to SPDX header
@ 2019-06-11  7:19 Lubomir Rintel
  2019-06-11 10:57 ` Guenter Roeck
  0 siblings, 1 reply; 2+ messages in thread
From: Lubomir Rintel @ 2019-06-11  7:19 UTC (permalink / raw)
  To: Jean Delvare, Guenter Roeck
  Cc: Paul Thomas, linux-hwmon, linux-kernel, Lubomir Rintel

The original license text had a typo ("publishhed") which would be
likely to confuse automated licensing auditing tools. Let's just switch
to SPDX instead of fixing the wording.

Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
---
 drivers/hwmon/ads7871.c | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/hwmon/ads7871.c b/drivers/hwmon/ads7871.c
index cd14c1501508..7ccc79f77f95 100644
--- a/drivers/hwmon/ads7871.c
+++ b/drivers/hwmon/ads7871.c
@@ -1,17 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  *  ads7871 - driver for TI ADS7871 A/D converter
  *
  *  Copyright (c) 2010 Paul Thomas <pthomas8589@gmail.com>
  *
- *  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.
- *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License version 2 or
- *  later as publishhed by the Free Software Foundation.
- *
  *	You need to have something like this in struct spi_board_info
  *	{
  *		.modalias	= "ads7871",
-- 
2.21.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] hwmon: (ads7871) Switch to SPDX header
  2019-06-11  7:19 [PATCH] hwmon: (ads7871) Switch to SPDX header Lubomir Rintel
@ 2019-06-11 10:57 ` Guenter Roeck
  0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2019-06-11 10:57 UTC (permalink / raw)
  To: Lubomir Rintel, Jean Delvare; +Cc: Paul Thomas, linux-hwmon, linux-kernel

On 6/11/19 12:19 AM, Lubomir Rintel wrote:
> The original license text had a typo ("publishhed") which would be
> likely to confuse automated licensing auditing tools. Let's just switch
> to SPDX instead of fixing the wording.
> 
If you have a look into the upstream kernel, you may notice that
this has already been done. Same for the other patch.

Guenter

> Signed-off-by: Lubomir Rintel <lkundrak@v3.sk>
> ---
>   drivers/hwmon/ads7871.c | 10 +---------
>   1 file changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/drivers/hwmon/ads7871.c b/drivers/hwmon/ads7871.c
> index cd14c1501508..7ccc79f77f95 100644
> --- a/drivers/hwmon/ads7871.c
> +++ b/drivers/hwmon/ads7871.c
> @@ -1,17 +1,9 @@
> +// SPDX-License-Identifier: GPL-2.0+
>   /*
>    *  ads7871 - driver for TI ADS7871 A/D converter
>    *
>    *  Copyright (c) 2010 Paul Thomas <pthomas8589@gmail.com>
>    *
> - *  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.
> - *
> - *  This program is free software; you can redistribute it and/or modify
> - *  it under the terms of the GNU General Public License version 2 or
> - *  later as publishhed by the Free Software Foundation.
> - *
>    *	You need to have something like this in struct spi_board_info
>    *	{
>    *		.modalias	= "ads7871",
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-06-11 10:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-11  7:19 [PATCH] hwmon: (ads7871) Switch to SPDX header Lubomir Rintel
2019-06-11 10:57 ` Guenter Roeck

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).