* [PATCH] auxdisplay: Replace licenses with SPDX identifiers
@ 2018-02-17 19:39 Miguel Ojeda
2018-02-17 21:27 ` Willy Tarreau
` (4 more replies)
0 siblings, 5 replies; 12+ messages in thread
From: Miguel Ojeda @ 2018-02-17 19:39 UTC (permalink / raw)
To: w, geert, triad, robin, paul.burton, linux-kernel
Cc: Willy Tarreau <w@1wt.eu>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Linus Walleij <triad@df.lth.se>
Cc: Robin van der Gracht <robin@protonic.nl>
Cc: Paul Burton <paul.burton@mips.com>
Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
---
Please let me know if you agree for your files and I will queue it up.
Thanks!
drivers/auxdisplay/arm-charlcd.c | 2 +-
drivers/auxdisplay/cfag12864b.c | 16 +---------------
drivers/auxdisplay/cfag12864bfb.c | 16 +---------------
drivers/auxdisplay/charlcd.c | 6 +-----
drivers/auxdisplay/hd44780.c | 6 +-----
drivers/auxdisplay/ht16k33.c | 10 +---------
drivers/auxdisplay/img-ascii-lcd.c | 6 +-----
drivers/auxdisplay/ks0108.c | 16 +---------------
drivers/auxdisplay/panel.c | 6 +-----
include/linux/cfag12864b.h | 16 +---------------
include/linux/ks0108.h | 16 +---------------
samples/auxdisplay/cfag12864b-example.c | 16 +---------------
12 files changed, 12 insertions(+), 120 deletions(-)
diff --git a/drivers/auxdisplay/arm-charlcd.c b/drivers/auxdisplay/arm-charlcd.c
index b3176ee92b90..ec5bd065d0d5 100644
--- a/drivers/auxdisplay/arm-charlcd.c
+++ b/drivers/auxdisplay/arm-charlcd.c
@@ -1,10 +1,10 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Driver for the on-board character LCD found on some ARM reference boards
* This is basically an Hitachi HD44780 LCD with a custom IP block to drive it
* http://en.wikipedia.org/wiki/HD44780_Character_LCD
* Currently it will just display the text "ARM Linux" and the linux version
*
- * License terms: GNU General Public License (GPL) version 2
* Author: Linus Walleij <triad@df.lth.se>
*/
#include <linux/init.h>
diff --git a/drivers/auxdisplay/cfag12864b.c b/drivers/auxdisplay/cfag12864b.c
index 41ce4bd96813..6bd2f65e116a 100644
--- a/drivers/auxdisplay/cfag12864b.c
+++ b/drivers/auxdisplay/cfag12864b.c
@@ -1,26 +1,12 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: cfag12864b.c
* Version: 0.1.0
* Description: cfag12864b LCD driver
- * License: GPLv2
* Depends: ks0108
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-31
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
#include <linux/init.h>
diff --git a/drivers/auxdisplay/cfag12864bfb.c b/drivers/auxdisplay/cfag12864bfb.c
index a3874034e2ce..898d14ca8ac8 100644
--- a/drivers/auxdisplay/cfag12864bfb.c
+++ b/drivers/auxdisplay/cfag12864bfb.c
@@ -1,26 +1,12 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: cfag12864bfb.c
* Version: 0.1.0
* Description: cfag12864b LCD framebuffer driver
- * License: GPLv2
* Depends: cfag12864b
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-31
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
#include <linux/init.h>
diff --git a/drivers/auxdisplay/charlcd.c b/drivers/auxdisplay/charlcd.c
index 642afd88870b..dc187a7a1a67 100644
--- a/drivers/auxdisplay/charlcd.c
+++ b/drivers/auxdisplay/charlcd.c
@@ -1,13 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0+
/*
* Character LCD driver for Linux
*
* Copyright (C) 2000-2008, Willy Tarreau <w@1wt.eu>
* Copyright (C) 2016-2017 Glider bvba
- *
- * 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/atomic.h>
diff --git a/drivers/auxdisplay/hd44780.c b/drivers/auxdisplay/hd44780.c
index 036eec404289..78d8f1986fec 100644
--- a/drivers/auxdisplay/hd44780.c
+++ b/drivers/auxdisplay/hd44780.c
@@ -1,13 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0+
/*
* HD44780 Character LCD driver for Linux
*
* Copyright (C) 2000-2008, Willy Tarreau <w@1wt.eu>
* Copyright (C) 2016-2017 Glider bvba
- *
- * 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/delay.h>
diff --git a/drivers/auxdisplay/ht16k33.c b/drivers/auxdisplay/ht16k33.c
index fbfa5b4cc567..a43276c76fc6 100644
--- a/drivers/auxdisplay/ht16k33.c
+++ b/drivers/auxdisplay/ht16k33.c
@@ -1,18 +1,10 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* HT16K33 driver
*
* Author: Robin van der Gracht <robin@protonic.nl>
*
* Copyright: (C) 2016 Protonic Holland.
- *
- * 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/kernel.h>
diff --git a/drivers/auxdisplay/img-ascii-lcd.c b/drivers/auxdisplay/img-ascii-lcd.c
index 9180b9bd5821..85d964c4b7b2 100644
--- a/drivers/auxdisplay/img-ascii-lcd.c
+++ b/drivers/auxdisplay/img-ascii-lcd.c
@@ -1,11 +1,7 @@
+// SPDX-License-Identifier: GPL-2.0+
/*
* Copyright (C) 2016 Imagination Technologies
* Author: Paul Burton <paul.burton@mips.com>
- *
- * 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 <generated/utsrelease.h>
diff --git a/drivers/auxdisplay/ks0108.c b/drivers/auxdisplay/ks0108.c
index 816de9eaac26..abfe3fa9e6f4 100644
--- a/drivers/auxdisplay/ks0108.c
+++ b/drivers/auxdisplay/ks0108.c
@@ -1,26 +1,12 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: ks0108.c
* Version: 0.1.0
* Description: ks0108 LCD Controller driver
- * License: GPLv2
* Depends: parport
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-31
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
diff --git a/drivers/auxdisplay/panel.c b/drivers/auxdisplay/panel.c
index ea7869c0d7f9..ab8f2a374864 100644
--- a/drivers/auxdisplay/panel.c
+++ b/drivers/auxdisplay/panel.c
@@ -1,13 +1,9 @@
+// SPDX-License-Identifier: GPL-2.0+
/*
* Front panel driver for Linux
* Copyright (C) 2000-2008, Willy Tarreau <w@1wt.eu>
* Copyright (C) 2016-2017 Glider bvba
*
- * 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 code drives an LCD module (/dev/lcd), and a keypad (/dev/keypad)
* connected to a parallel printer port.
*
diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
index b454dfce60d9..aa960efc32f6 100644
--- a/include/linux/cfag12864b.h
+++ b/include/linux/cfag12864b.h
@@ -1,25 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: cfag12864b.h
* Version: 0.1.0
* Description: cfag12864b LCD driver header
- * License: GPLv2
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-12
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
#ifndef _CFAG12864B_H_
diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
index cb311798e0bc..2a1c985fedea 100644
--- a/include/linux/ks0108.h
+++ b/include/linux/ks0108.h
@@ -1,25 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: ks0108.h
* Version: 0.1.0
* Description: ks0108 LCD Controller driver header
- * License: GPLv2
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-31
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
#ifndef _KS0108_H_
diff --git a/samples/auxdisplay/cfag12864b-example.c b/samples/auxdisplay/cfag12864b-example.c
index e7823ffb1ca0..85571e90191f 100644
--- a/samples/auxdisplay/cfag12864b-example.c
+++ b/samples/auxdisplay/cfag12864b-example.c
@@ -1,25 +1,11 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* Filename: cfag12864b-example.c
* Version: 0.1.0
* Description: cfag12864b LCD userspace example program
- * License: GPLv2
*
* Author: Copyright (C) Miguel Ojeda Sandonis
* Date: 2006-10-31
- *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
- *
*/
/*
--
2.14.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
@ 2018-02-17 21:27 ` Willy Tarreau
2018-02-17 22:02 ` Linus Walleij
` (3 subsequent siblings)
4 siblings, 0 replies; 12+ messages in thread
From: Willy Tarreau @ 2018-02-17 21:27 UTC (permalink / raw)
To: Miguel Ojeda; +Cc: geert, triad, robin, paul.burton, linux-kernel
On Sat, Feb 17, 2018 at 08:39:55PM +0100, Miguel Ojeda wrote:
> Cc: Willy Tarreau <w@1wt.eu>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Linus Walleij <triad@df.lth.se>
> Cc: Robin van der Gracht <robin@protonic.nl>
> Cc: Paul Burton <paul.burton@mips.com>
> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> ---
> Please let me know if you agree for your files and I will queue it up.
> Thanks!
Ack from me.
Willy
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
2018-02-17 21:27 ` Willy Tarreau
@ 2018-02-17 22:02 ` Linus Walleij
2018-02-18 19:04 ` Philippe Ombredanne
` (2 subsequent siblings)
4 siblings, 0 replies; 12+ messages in thread
From: Linus Walleij @ 2018-02-17 22:02 UTC (permalink / raw)
To: Miguel Ojeda; +Cc: w, geert, Linus Walleij, robin, paul.burton, linux kernel
2018-02-17 20:39 GMT+01:00 Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>:
> Cc: Willy Tarreau <w@1wt.eu>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Linus Walleij <triad@df.lth.se>
> Cc: Robin van der Gracht <robin@protonic.nl>
> Cc: Paul Burton <paul.burton@mips.com>
> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> ---
> Please let me know if you agree for your files and I will queue it up.
> Thanks!
Acked-by: Linus Walleij <triad@dflund.se>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
2018-02-17 21:27 ` Willy Tarreau
2018-02-17 22:02 ` Linus Walleij
@ 2018-02-18 19:04 ` Philippe Ombredanne
2018-02-18 19:37 ` Geert Uytterhoeven
2018-02-18 19:56 ` Miguel Ojeda
2018-02-19 7:44 ` Robin van der Gracht
2018-02-19 7:55 ` Geert Uytterhoeven
4 siblings, 2 replies; 12+ messages in thread
From: Philippe Ombredanne @ 2018-02-18 19:04 UTC (permalink / raw)
To: Miguel Ojeda; +Cc: w, Geert Uytterhoeven, triad, robin, Paul Burton, LKML
Miguel,
On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
<miguel.ojeda.sandonis@gmail.com> wrote:
> Cc: Willy Tarreau <w@1wt.eu>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Linus Walleij <triad@df.lth.se>
> Cc: Robin van der Gracht <robin@protonic.nl>
> Cc: Paul Burton <paul.burton@mips.com>
> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> ---
<snip>
> diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
> index b454dfce60d9..aa960efc32f6 100644
> --- a/include/linux/cfag12864b.h
> +++ b/include/linux/cfag12864b.h
> @@ -1,25 +1,11 @@
> +// SPDX-License-Identifier: GPL-2.0
Per the doc [1] you should be using instead this in a .h:
/* SPDX-License-Identifier: GPL-2.0 */
I know this can be surprising. This has been discussed on list quite
lot and the doc has some rationale.
<snip>
> diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
> index cb311798e0bc..2a1c985fedea 100644
> --- a/include/linux/ks0108.h
> +++ b/include/linux/ks0108.h
> @@ -1,25 +1,11 @@
> +// SPDX-License-Identifier: GPL-2.0
Same comment as above.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
--
Cordially
Philippe Ombredanne
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-18 19:04 ` Philippe Ombredanne
@ 2018-02-18 19:37 ` Geert Uytterhoeven
2018-02-18 20:18 ` Joe Perches
2018-02-18 19:56 ` Miguel Ojeda
1 sibling, 1 reply; 12+ messages in thread
From: Geert Uytterhoeven @ 2018-02-18 19:37 UTC (permalink / raw)
To: Philippe Ombredanne
Cc: Miguel Ojeda, Willy Tarreau, Linus Walleij, robin, Paul Burton, LKML
Hi Philippe,
On Sun, Feb 18, 2018 at 8:04 PM, Philippe Ombredanne
<pombredanne@nexb.com> wrote:
> On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
> <miguel.ojeda.sandonis@gmail.com> wrote:
>> Cc: Willy Tarreau <w@1wt.eu>
>> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
>> Cc: Linus Walleij <triad@df.lth.se>
>> Cc: Robin van der Gracht <robin@protonic.nl>
>> Cc: Paul Burton <paul.burton@mips.com>
>> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
>> ---
>
> <snip>
>
>> diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
>> index b454dfce60d9..aa960efc32f6 100644
>> --- a/include/linux/cfag12864b.h
>> +++ b/include/linux/cfag12864b.h
>> @@ -1,25 +1,11 @@
>> +// SPDX-License-Identifier: GPL-2.0
>
> Per the doc [1] you should be using instead this in a .h:
> /* SPDX-License-Identifier: GPL-2.0 */
>
> I know this can be surprising. This has been discussed on list quite
> lot and the doc has some rationale.
>
> <snip>
>
>> diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
>> index cb311798e0bc..2a1c985fedea 100644
>> --- a/include/linux/ks0108.h
>> +++ b/include/linux/ks0108.h
>> @@ -1,25 +1,11 @@
>> +// SPDX-License-Identifier: GPL-2.0
>
> Same comment as above.
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
>
Obviously we need a check in scripts/checkpatch.pl, to catch this
before submission...
Care to send a patch?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-18 19:04 ` Philippe Ombredanne
2018-02-18 19:37 ` Geert Uytterhoeven
@ 2018-02-18 19:56 ` Miguel Ojeda
1 sibling, 0 replies; 12+ messages in thread
From: Miguel Ojeda @ 2018-02-18 19:56 UTC (permalink / raw)
To: Philippe Ombredanne
Cc: Willy Tarreau, Geert Uytterhoeven, Linus Walleij, robin,
Paul Burton, LKML
On Sun, Feb 18, 2018 at 8:04 PM, Philippe Ombredanne
<pombredanne@nexb.com> wrote:
> Miguel,
>
> On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
> <miguel.ojeda.sandonis@gmail.com> wrote:
>> Cc: Willy Tarreau <w@1wt.eu>
>> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
>> Cc: Linus Walleij <triad@df.lth.se>
>> Cc: Robin van der Gracht <robin@protonic.nl>
>> Cc: Paul Burton <paul.burton@mips.com>
>> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
>> ---
>
> <snip>
>
>> diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
>> index b454dfce60d9..aa960efc32f6 100644
>> --- a/include/linux/cfag12864b.h
>> +++ b/include/linux/cfag12864b.h
>> @@ -1,25 +1,11 @@
>> +// SPDX-License-Identifier: GPL-2.0
>
> Per the doc [1] you should be using instead this in a .h:
> /* SPDX-License-Identifier: GPL-2.0 */
>
> I know this can be surprising. This has been discussed on list quite
> lot and the doc has some rationale.
Thanks for taking the time to check the patch. It was a mistake on my
side -- I had actually read the docs but I recalled Linus talking
about moving to // at some point (and somehow my brain recorded that
the discussion was about the headers :-), so I thought I was actually
doing the right thing by using // already. Don't worry, I will fix it
when I put it in the queue.
Cheers,
Miguel
>
> <snip>
>
>> diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
>> index cb311798e0bc..2a1c985fedea 100644
>> --- a/include/linux/ks0108.h
>> +++ b/include/linux/ks0108.h
>> @@ -1,25 +1,11 @@
>> +// SPDX-License-Identifier: GPL-2.0
>
> Same comment as above.
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
>
> --
> Cordially
> Philippe Ombredanne
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-18 19:37 ` Geert Uytterhoeven
@ 2018-02-18 20:18 ` Joe Perches
2018-02-20 7:41 ` Philippe Ombredanne
0 siblings, 1 reply; 12+ messages in thread
From: Joe Perches @ 2018-02-18 20:18 UTC (permalink / raw)
To: Geert Uytterhoeven, Philippe Ombredanne
Cc: Miguel Ojeda, Willy Tarreau, Linus Walleij, robin, Paul Burton, LKML
On Sun, 2018-02-18 at 20:37 +0100, Geert Uytterhoeven wrote:
> Hi Philippe,
>
> On Sun, Feb 18, 2018 at 8:04 PM, Philippe Ombredanne
> <pombredanne@nexb.com> wrote:
> > On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
> > <miguel.ojeda.sandonis@gmail.com> wrote:
> > > Cc: Willy Tarreau <w@1wt.eu>
> > > Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> > > Cc: Linus Walleij <triad@df.lth.se>
> > > Cc: Robin van der Gracht <robin@protonic.nl>
> > > Cc: Paul Burton <paul.burton@mips.com>
> > > Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> > > ---
> >
> > <snip>
> >
> > > diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
> > > index b454dfce60d9..aa960efc32f6 100644
> > > --- a/include/linux/cfag12864b.h
> > > +++ b/include/linux/cfag12864b.h
> > > @@ -1,25 +1,11 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> >
> > Per the doc [1] you should be using instead this in a .h:
> > /* SPDX-License-Identifier: GPL-2.0 */
> >
> > I know this can be surprising. This has been discussed on list quite
> > lot and the doc has some rationale.
> >
> > <snip>
> >
> > > diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
> > > index cb311798e0bc..2a1c985fedea 100644
> > > --- a/include/linux/ks0108.h
> > > +++ b/include/linux/ks0108.h
> > > @@ -1,25 +1,11 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> >
> > Same comment as above.
> >
> > [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
> >
>
> Obviously we need a check in scripts/checkpatch.pl, to catch this
> before submission...
>
> Care to send a patch?
There's one already in progress.
https://lkml.org/lkml/2018/2/8/712
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
` (2 preceding siblings ...)
2018-02-18 19:04 ` Philippe Ombredanne
@ 2018-02-19 7:44 ` Robin van der Gracht
2018-02-19 7:48 ` Robin van der Gracht
2018-02-19 7:55 ` Geert Uytterhoeven
4 siblings, 1 reply; 12+ messages in thread
From: Robin van der Gracht @ 2018-02-19 7:44 UTC (permalink / raw)
To: Miguel Ojeda; +Cc: w, geert, triad, paul.burton, linux-kernel
On Sat, 17 Feb 2018 20:39:55 +0100
Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> wrote:
> Cc: Willy Tarreau <w@1wt.eu>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Linus Walleij <triad@df.lth.se>
> Cc: Robin van der Gracht <robin@protonic.nl>
> Cc: Paul Burton <paul.burton@mips.com>
> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> ---
> Please let me know if you agree for your files and I will queue it up.
> Thanks!
...
>
> #include <linux/delay.h>
> diff --git a/drivers/auxdisplay/ht16k33.c b/drivers/auxdisplay/ht16k33.c
> index fbfa5b4cc567..a43276c76fc6 100644
> --- a/drivers/auxdisplay/ht16k33.c
> +++ b/drivers/auxdisplay/ht16k33.c
> @@ -1,18 +1,10 @@
> +// SPDX-License-Identifier: GPL-2.0
> /*
> * HT16K33 driver
> *
> * Author: Robin van der Gracht <robin@protonic.nl>
> *
> * Copyright: (C) 2016 Protonic Holland.
> - *
> - * 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.
> */
>
Signed-off-by: Robin van der Gracht <robin@protonic.nl>
Best regards, / vriendelijke groet,
--
Robin van der Gracht
Protonic Holland
tel.: +31 (0) 229 212928
fax.: +31 (0) 229 210930
Factorij 36 / 1689 AL Zwaag
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-19 7:44 ` Robin van der Gracht
@ 2018-02-19 7:48 ` Robin van der Gracht
0 siblings, 0 replies; 12+ messages in thread
From: Robin van der Gracht @ 2018-02-19 7:48 UTC (permalink / raw)
To: Miguel Ojeda; +Cc: w, geert, triad, paul.burton, linux-kernel
On Mon, 19 Feb 2018 08:44:17 +0100
Robin van der Gracht <robin@protonic.nl> wrote:
> On Sat, 17 Feb 2018 20:39:55 +0100
> Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> wrote:
>
> > Cc: Willy Tarreau <w@1wt.eu>
> > Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> > Cc: Linus Walleij <triad@df.lth.se>
> > Cc: Robin van der Gracht <robin@protonic.nl>
> > Cc: Paul Burton <paul.burton@mips.com>
> > Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
> > ---
> > Please let me know if you agree for your files and I will queue it up.
> > Thanks!
>
> ...
>
> >
> > #include <linux/delay.h>
> > diff --git a/drivers/auxdisplay/ht16k33.c b/drivers/auxdisplay/ht16k33.c
> > index fbfa5b4cc567..a43276c76fc6 100644
> > --- a/drivers/auxdisplay/ht16k33.c
> > +++ b/drivers/auxdisplay/ht16k33.c
> > @@ -1,18 +1,10 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > /*
> > * HT16K33 driver
> > *
> > * Author: Robin van der Gracht <robin@protonic.nl>
> > *
> > * Copyright: (C) 2016 Protonic Holland.
> > - *
> > - * 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.
> > */
> >
>
> Signed-off-by: Robin van der Gracht <robin@protonic.nl>
Change to:
Acked-by: Robin van der Gracht <robin@protonic.nl>
Best regards, / vriendelijke groet,
--
Robin van der Gracht
Protonic Holland
tel.: +31 (0) 229 212928
fax.: +31 (0) 229 210930
Factorij 36 / 1689 AL Zwaag
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
` (3 preceding siblings ...)
2018-02-19 7:44 ` Robin van der Gracht
@ 2018-02-19 7:55 ` Geert Uytterhoeven
4 siblings, 0 replies; 12+ messages in thread
From: Geert Uytterhoeven @ 2018-02-19 7:55 UTC (permalink / raw)
To: Miguel Ojeda
Cc: Willy Tarreau, Linus Walleij, robin, Paul Burton,
Linux Kernel Mailing List
On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
<miguel.ojeda.sandonis@gmail.com> wrote:
> Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
For:
> drivers/auxdisplay/hd44780.c | 6 +-----
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-18 20:18 ` Joe Perches
@ 2018-02-20 7:41 ` Philippe Ombredanne
2018-03-13 0:20 ` Miguel Ojeda
0 siblings, 1 reply; 12+ messages in thread
From: Philippe Ombredanne @ 2018-02-20 7:41 UTC (permalink / raw)
To: Joe Perches, Geert Uytterhoeven
Cc: Miguel Ojeda, Willy Tarreau, Linus Walleij, robin, Paul Burton, LKML
On Sun, Feb 18, 2018 at 9:18 PM, Joe Perches <joe@perches.com> wrote:
> On Sun, 2018-02-18 at 20:37 +0100, Geert Uytterhoeven wrote:
>> Hi Philippe,
>>
>> On Sun, Feb 18, 2018 at 8:04 PM, Philippe Ombredanne
>> <pombredanne@nexb.com> wrote:
>> > On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
>> > <miguel.ojeda.sandonis@gmail.com> wrote:
>> > > Cc: Willy Tarreau <w@1wt.eu>
>> > > Cc: Geert Uytterhoeven <geert@linux-m68k.org>
>> > > Cc: Linus Walleij <triad@df.lth.se>
>> > > Cc: Robin van der Gracht <robin@protonic.nl>
>> > > Cc: Paul Burton <paul.burton@mips.com>
>> > > Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
>> > > ---
>> >
>> > <snip>
>> >
>> > > diff --git a/include/linux/cfag12864b.h b/include/linux/cfag12864b.h
>> > > index b454dfce60d9..aa960efc32f6 100644
>> > > --- a/include/linux/cfag12864b.h
>> > > +++ b/include/linux/cfag12864b.h
>> > > @@ -1,25 +1,11 @@
>> > > +// SPDX-License-Identifier: GPL-2.0
>> >
>> > Per the doc [1] you should be using instead this in a .h:
>> > /* SPDX-License-Identifier: GPL-2.0 */
>> >
>> > I know this can be surprising. This has been discussed on list quite
>> > lot and the doc has some rationale.
>> >
>> > <snip>
>> >
>> > > diff --git a/include/linux/ks0108.h b/include/linux/ks0108.h
>> > > index cb311798e0bc..2a1c985fedea 100644
>> > > --- a/include/linux/ks0108.h
>> > > +++ b/include/linux/ks0108.h
>> > > @@ -1,25 +1,11 @@
>> > > +// SPDX-License-Identifier: GPL-2.0
>> >
>> > Same comment as above.
>> >
>> > [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst
>> >
>>
>> Obviously we need a check in scripts/checkpatch.pl, to catch this
>> before submission...
>>
>> Care to send a patch?
>
> There's one already in progress.
> https://lkml.org/lkml/2018/2/8/712
>
And I will try to help there as much as I can. Time to dust off my Perl!
--
Cordially
Philippe Ombredanne
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] auxdisplay: Replace licenses with SPDX identifiers
2018-02-20 7:41 ` Philippe Ombredanne
@ 2018-03-13 0:20 ` Miguel Ojeda
0 siblings, 0 replies; 12+ messages in thread
From: Miguel Ojeda @ 2018-03-13 0:20 UTC (permalink / raw)
To: paul.burton, Paul Burton, paulburton89
Cc: Joe Perches, Geert Uytterhoeven, Willy Tarreau, Linus Walleij,
Robin van der Gracht, Philippe Ombredanne, LKML
On Tue, Feb 20, 2018 at 8:41 AM, Philippe Ombredanne
<pombredanne@nexb.com> wrote:
> On Sun, Feb 18, 2018 at 9:18 PM, Joe Perches <joe@perches.com> wrote:
>> On Sun, 2018-02-18 at 20:37 +0100, Geert Uytterhoeven wrote:
>>> Hi Philippe,
>>>
>>> On Sun, Feb 18, 2018 at 8:04 PM, Philippe Ombredanne
>>> <pombredanne@nexb.com> wrote:
>>> > On Sat, Feb 17, 2018 at 8:39 PM, Miguel Ojeda
>>> > <miguel.ojeda.sandonis@gmail.com> wrote:
>>> > > Cc: Willy Tarreau <w@1wt.eu>
>>> > > Cc: Geert Uytterhoeven <geert@linux-m68k.org>
>>> > > Cc: Linus Walleij <triad@df.lth.se>
>>> > > Cc: Robin van der Gracht <robin@protonic.nl>
>>> > > Cc: Paul Burton <paul.burton@mips.com>
>>> > > Signed-off-by: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>
>>> > > ---
Pinging Paul Burton in a couple of emails more...
Cheers,
Miguel
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-03-13 0:20 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-17 19:39 [PATCH] auxdisplay: Replace licenses with SPDX identifiers Miguel Ojeda
2018-02-17 21:27 ` Willy Tarreau
2018-02-17 22:02 ` Linus Walleij
2018-02-18 19:04 ` Philippe Ombredanne
2018-02-18 19:37 ` Geert Uytterhoeven
2018-02-18 20:18 ` Joe Perches
2018-02-20 7:41 ` Philippe Ombredanne
2018-03-13 0:20 ` Miguel Ojeda
2018-02-18 19:56 ` Miguel Ojeda
2018-02-19 7:44 ` Robin van der Gracht
2018-02-19 7:48 ` Robin van der Gracht
2018-02-19 7:55 ` Geert Uytterhoeven
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).