linux-kernel - Re: [PATCH v2 3/8] mfd: cros_ec: Add or fix SPDX-License-Identifier in all files.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 18 Jun 2018 07:37:50 +0100
From:   Lee Jones <lee.jones@...aro.org>
To:     Enric Balletbo i Serra <enric.balletbo@...labora.com>
Cc:     linux-kernel@...r.kernel.org,
        Gwendal Grignou <gwendal@...omium.org>, kernel@...labora.com
Subject: Re: [PATCH v2 3/8] mfd: cros_ec: Add or fix SPDX-License-Identifier
 in all files.

On Tue, 05 Jun 2018, Enric Balletbo i Serra wrote:

> And get rid of the license text that is no longer necessary. Also fix
> the license as sometimes doesn't match what the header with the value in
> the MODULE_LICENSE macro. Assuming that the desired license is GPL-2.0+,
> all the files are updated to this license version.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@...labora.com>
> ---
> 
> Changes in v2: None
> 
>  drivers/mfd/cros_ec.c                | 26 ++++++++------------------
>  drivers/mfd/cros_ec_dev.c            | 23 +++++------------------
>  drivers/mfd/cros_ec_dev.h            | 16 ++--------------
>  drivers/mfd/cros_ec_i2c.c            | 18 ++++--------------
>  drivers/mfd/cros_ec_spi.c            | 20 +++++---------------
>  include/linux/mfd/cros_ec.h          | 10 +---------
>  include/linux/mfd/cros_ec_commands.h | 10 +---------
>  include/linux/mfd/cros_ec_lpc_mec.h  | 14 +++-----------
>  include/linux/mfd/cros_ec_lpc_reg.h  | 14 +++-----------
>  9 files changed, 32 insertions(+), 119 deletions(-)
> 
> diff --git a/drivers/mfd/cros_ec.c b/drivers/mfd/cros_ec.c
> index 58e05069163e..5b6a1c541543 100644
> --- a/drivers/mfd/cros_ec.c
> +++ b/drivers/mfd/cros_ec.c
> @@ -1,21 +1,11 @@
> -/*
> - * ChromeOS EC multi-function device
> - *
> - * Copyright (C) 2012 Google, Inc
> - *
> - * This software is licensed under the terms of the GNU General Public
> - * License version 2, as published by the Free Software Foundation, and
> - * may be copied, distributed, and modified under those terms.
> - *
> - * 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.
> - *
> - * The ChromeOS EC multi function device is used to mux all the requests
> - * to the EC device for its multiple features: keyboard controller,
> - * battery charging and regulator control, firmware update.
> - */
> +// SPDX-License-Identifier: GPL-2.0+
> +// ChromeOS EC multi-function device.
> +//
> +// Copyright (C) 2012 Google, Inc
> +//
> +// The ChromeOS EC multi function device is used to mux all the requests
> +// to the EC device for its multiple features: keyboard controller,
> +// battery charging and regulator control, firmware update.

I must admit I haven't seen a great many patches like this.  Is C++
comments for the SPDX header *and* the Copyright text the done thing
now then?

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog