Re: [PATCH] ethernet: renesas: convert to SPDX identifiers

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

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