linux-kernel - Re: [PATCH 4/9] iio: cros_ec: Switch to SPDX identifier.

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: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Sun, 10 Jun 2018 14:40:31 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Enric Balletbo i Serra <enric.balletbo@...labora.com>
Cc:     linux-kernel@...r.kernel.org,
        Gwendal Grignou <gwendal@...omium.org>, kernel@...labora.com,
        Paolo Cretaro <paolocretaro@...il.com>,
        linux-iio@...r.kernel.org, Hartmut Knaack <knaack.h@....de>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>
Subject: Re: [PATCH 4/9] iio: cros_ec: Switch to SPDX identifier.

On Tue,  5 Jun 2018 11:22:04 +0200
Enric Balletbo i Serra <enric.balletbo@...labora.com> wrote:

> Adopt the SPDX license identifier headers to ease license compliance
> management.
> 
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@...labora.com>
This code has seen quite a few contributors, I'd like at least
some of them to agree they are happy with this change.

There hasn't been much general resistance in general to SPDX
headers replacing text, but there has been some, so I've been
asking this for all the IIO related SPDX patches.

Thanks,

Jonathan

> ---
> 
>  drivers/iio/accel/cros_ec_accel_legacy.c      | 23 ++++--------------
>  .../common/cros_ec_sensors/cros_ec_sensors.c  | 24 ++++++-------------
>  .../cros_ec_sensors/cros_ec_sensors_core.c    | 18 ++++----------
>  .../cros_ec_sensors/cros_ec_sensors_core.h    | 12 ++--------
>  drivers/iio/light/cros_ec_light_prox.c        | 18 ++++----------
>  drivers/iio/pressure/cros_ec_baro.c           | 18 ++++----------
>  6 files changed, 26 insertions(+), 87 deletions(-)
> 
> diff --git a/drivers/iio/accel/cros_ec_accel_legacy.c b/drivers/iio/accel/cros_ec_accel_legacy.c
> index 063e89eff791..e7350ddec328 100644
> --- a/drivers/iio/accel/cros_ec_accel_legacy.c
> +++ b/drivers/iio/accel/cros_ec_accel_legacy.c
> @@ -1,21 +1,8 @@
> -/*
> - * Driver for older Chrome OS EC accelerometer
> - *
> - * Copyright 2017 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.
> - *
> - * This driver uses the memory mapper cros-ec interface to communicate
> - * with the Chrome OS EC about accelerometer data.
> - * Accelerometer access is presented through iio sysfs.
> - */
> +// SPDX-License-Identifier: GPL-2.0+
> +// Driver for older Chrome OS EC accelerometer
> +//
> +// Copyright (C) 2017 Google, Inc.
> +// Author: Gwendal Grignou <gwendal@...omium.org>
>  
>  #include <linux/delay.h>
>  #include <linux/device.h>
> diff --git a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c
> index 705cb3e72663..3dbc90baf6bb 100644
> --- a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c
> +++ b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c
> @@ -1,20 +1,10 @@
> -/*
> - * cros_ec_sensors - Driver for Chrome OS Embedded Controller sensors.
> - *
> - * Copyright (C) 2016 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.
> - *
> - * This driver uses the cros-ec interface to communicate with the Chrome OS
> - * EC about sensors data. Data access is presented through iio sysfs.
> - */
> +// SPDX-License-Identifier: GPL-2.0
> +// Driver for Chrome OS Embedded Controller sensors.
> +//
> +// Copyright (C) 2016 Google, Inc.
> +//
> +// This driver uses the cros-ec interface to communicate with the ChromeOS
> +// EC about sensors data. Data access is presented through iio sysfs.
>  
>  #include <linux/delay.h>
>  #include <linux/device.h>
> diff --git a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c
> index a620eb5ce202..05221994197c 100644
> --- a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c
> +++ b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c
> @@ -1,17 +1,7 @@
> -/*
> - * cros_ec_sensors_core - Common function for Chrome OS EC sensor driver.
> - *
> - * Copyright (C) 2016 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.
> - */
> +// SPDX-License-Identifier: GPL-2.0
> +// Common functions for ChromeOS EC sensor driver.
> +//
> +// Copyright (C) 2016 Google, Inc.
>  
>  #include <linux/delay.h>
>  #include <linux/device.h>
> diff --git a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.h b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.h
> index 2edf68dc7336..a9935489030e 100644
> --- a/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.h
> +++ b/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.h
> @@ -1,16 +1,8 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
>  /*
>   * ChromeOS EC sensor hub
>   *
> - * Copyright (C) 2016 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.
> + * Copyright (C) 2016 Google, Inc.
>   */
>  
>  #ifndef __CROS_EC_SENSORS_CORE_H
> diff --git a/drivers/iio/light/cros_ec_light_prox.c b/drivers/iio/light/cros_ec_light_prox.c
> index 8e8a0e7f78d1..8635e1c18cd5 100644
> --- a/drivers/iio/light/cros_ec_light_prox.c
> +++ b/drivers/iio/light/cros_ec_light_prox.c
> @@ -1,17 +1,7 @@
> -/*
> - * cros_ec_light_prox - Driver for light and prox sensors behing CrosEC.
> - *
> - * Copyright (C) 2017 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.
> - */
> +// SPDX-License-Identifier: GPL-2.0
> +// Driver for light and prox sensors behind ChromeOS EC.
> +//
> +// Copyright (C) 2017 Google, Inc.
>  
>  #include <linux/delay.h>
>  #include <linux/device.h>
> diff --git a/drivers/iio/pressure/cros_ec_baro.c b/drivers/iio/pressure/cros_ec_baro.c
> index 4599fde4dd25..8e134d65d18d 100644
> --- a/drivers/iio/pressure/cros_ec_baro.c
> +++ b/drivers/iio/pressure/cros_ec_baro.c
> @@ -1,17 +1,7 @@
> -/*
> - * cros_ec_baro - Driver for barometer sensor behind CrosEC.
> - *
> - * Copyright (C) 2017 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.
> - */
> +// SPDX-License-Identifier: GPL-2.0
> +// Driver for barometer sensor behind ChromeOS EC.
> +//
> +// Copyright (C) 2017 Google, Inc.
>  
>  #include <linux/delay.h>
>  #include <linux/device.h>