diff mbox series

ethernet: renesas: convert to SPDX identifiers

Message ID 875zziqdem.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show
Series ethernet: renesas: convert to SPDX identifiers | expand

Commit Message

Kuninori Morimoto Sept. 7, 2018, 2:02 a.m. UTC
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

This patch updates license to use SPDX-License-Identifier
instead of verbose license text.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 drivers/net/ethernet/renesas/Kconfig    | 1 +
 drivers/net/ethernet/renesas/Makefile   | 1 +
 drivers/net/ethernet/renesas/ravb_ptp.c | 6 +-----
 3 files changed, 3 insertions(+), 5 deletions(-)

Comments

Sergei Shtylyov Sept. 7, 2018, 9:05 a.m. UTC | #1
Hello!

On 9/7/2018 5:02 AM, Kuninori Morimoto wrote:

> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> 
> This patch updates license to use SPDX-License-Identifier
> instead of verbose license text.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
>   drivers/net/ethernet/renesas/Kconfig    | 1 +
>   drivers/net/ethernet/renesas/Makefile   | 1 +
>   drivers/net/ethernet/renesas/ravb_ptp.c | 6 +-----
>   3 files changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/net/ethernet/renesas/Kconfig b/drivers/net/ethernet/renesas/Kconfig
> index f3f7477..bb0ebdf 100644
> --- a/drivers/net/ethernet/renesas/Kconfig
> +++ b/drivers/net/ethernet/renesas/Kconfig
> @@ -1,3 +1,4 @@
> +# SPDX-License-Identifier: GPL-2.0
>   #
>   # Renesas device configuration
>   #

    There was no license at all on this file...

> diff --git a/drivers/net/ethernet/renesas/Makefile b/drivers/net/ethernet/renesas/Makefile
> index a05102a..f21ab8c 100644
> --- a/drivers/net/ethernet/renesas/Makefile
> +++ b/drivers/net/ethernet/renesas/Makefile
> @@ -1,3 +1,4 @@
> +# SPDX-License-Identifier: GPL-2.0
>   #
>   # Makefile for the Renesas device drivers.
>   #

    Likewise.

> diff --git a/drivers/net/ethernet/renesas/ravb_ptp.c b/drivers/net/ethernet/renesas/ravb_ptp.c
> index eede70e..0721b5c 100644
> --- a/drivers/net/ethernet/renesas/ravb_ptp.c
> +++ b/drivers/net/ethernet/renesas/ravb_ptp.c
> @@ -1,13 +1,9 @@
> +// SPDX-License-Identifier: GPL-2.0+
>   /* PTP 1588 clock using the Renesas Ethernet AVB
>    *
>    * Copyright (C) 2013-2015 Renesas Electronics Corporation
>    * Copyright (C) 2015 Renesas Solutions Corp.
>    * Copyright (C) 2015-2016 Cogent Embedded, Inc. <source@cogentembedded.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 "ravb.h"

    This part looks valid, don't know why this file was missed by Wolfram who 
did the previous patch...

MBR, Sergei
Geert Uytterhoeven Sept. 7, 2018, 9:11 a.m. UTC | #2
Hi Sergei,

On Fri, Sep 7, 2018 at 11:05 AM Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> On 9/7/2018 5:02 AM, Kuninori Morimoto wrote:
> > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> >
> > This patch updates license to use SPDX-License-Identifier
> > instead of verbose license text.
> >
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> > ---
> >   drivers/net/ethernet/renesas/Kconfig    | 1 +
> >   drivers/net/ethernet/renesas/Makefile   | 1 +
> >   drivers/net/ethernet/renesas/ravb_ptp.c | 6 +-----
> >   3 files changed, 3 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/renesas/Kconfig b/drivers/net/ethernet/renesas/Kconfig
> > index f3f7477..bb0ebdf 100644
> > --- a/drivers/net/ethernet/renesas/Kconfig
> > +++ b/drivers/net/ethernet/renesas/Kconfig
> > @@ -1,3 +1,4 @@
> > +# SPDX-License-Identifier: GPL-2.0
> >   #
> >   # Renesas device configuration
> >   #
>
>     There was no license at all on this file...

So it's licensed under GPL-2.0.
Cfr. e.g. commit 7328c8f48d1895b3 ("PCI: Add SPDX GPL-2.0 when no license
was specified")

Gr{oetje,eeting}s,

                        Geert
Sergei Shtylyov Sept. 7, 2018, 9:16 a.m. UTC | #3
On 9/7/2018 12:11 PM, Geert Uytterhoeven wrote:

>> On 9/7/2018 5:02 AM, Kuninori Morimoto wrote:
>>> From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>>>
>>> This patch updates license to use SPDX-License-Identifier
>>> instead of verbose license text.
>>>
>>> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>>> ---
>>>    drivers/net/ethernet/renesas/Kconfig    | 1 +
>>>    drivers/net/ethernet/renesas/Makefile   | 1 +
>>>    drivers/net/ethernet/renesas/ravb_ptp.c | 6 +-----
>>>    3 files changed, 3 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/net/ethernet/renesas/Kconfig b/drivers/net/ethernet/renesas/Kconfig
>>> index f3f7477..bb0ebdf 100644
>>> --- a/drivers/net/ethernet/renesas/Kconfig
>>> +++ b/drivers/net/ethernet/renesas/Kconfig
>>> @@ -1,3 +1,4 @@
>>> +# SPDX-License-Identifier: GPL-2.0
>>>    #
>>>    # Renesas device configuration
>>>    #
>>
>>      There was no license at all on this file...
> 
> So it's licensed under GPL-2.0.
> Cfr. e.g. commit 7328c8f48d1895b3 ("PCI: Add SPDX GPL-2.0 when no license
> was specified")

    OK, but then does the patch description match what is being done?

> Gr{oetje,eeting}s,
> 
>                          Geert

MBR, Sergei
diff mbox series

Patch

diff --git a/drivers/net/ethernet/renesas/Kconfig b/drivers/net/ethernet/renesas/Kconfig
index f3f7477..bb0ebdf 100644
--- a/drivers/net/ethernet/renesas/Kconfig
+++ b/drivers/net/ethernet/renesas/Kconfig
@@ -1,3 +1,4 @@ 
+# SPDX-License-Identifier: GPL-2.0
 #
 # Renesas device configuration
 #
diff --git a/drivers/net/ethernet/renesas/Makefile b/drivers/net/ethernet/renesas/Makefile
index a05102a..f21ab8c 100644
--- a/drivers/net/ethernet/renesas/Makefile
+++ b/drivers/net/ethernet/renesas/Makefile
@@ -1,3 +1,4 @@ 
+# SPDX-License-Identifier: GPL-2.0
 #
 # Makefile for the Renesas device drivers.
 #
diff --git a/drivers/net/ethernet/renesas/ravb_ptp.c b/drivers/net/ethernet/renesas/ravb_ptp.c
index eede70e..0721b5c 100644
--- a/drivers/net/ethernet/renesas/ravb_ptp.c
+++ b/drivers/net/ethernet/renesas/ravb_ptp.c
@@ -1,13 +1,9 @@ 
+// SPDX-License-Identifier: GPL-2.0+
 /* PTP 1588 clock using the Renesas Ethernet AVB
  *
  * Copyright (C) 2013-2015 Renesas Electronics Corporation
  * Copyright (C) 2015 Renesas Solutions Corp.
  * Copyright (C) 2015-2016 Cogent Embedded, Inc. <source@cogentembedded.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 "ravb.h"