Re: [PATCH] iio: adc: Replace license text w/ SPDX identifier

From: Matheus Tavares Bernardino
Date: Sat Dec 01 2018 - 10:58:09 EST

On Thu, Nov 29, 2018 at 5:37 AM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
>
> On Wed, 28 Nov 2018 at 18:53, Matheus Tavares <matheus.bernardino@xxxxxx> wrote:
> >
> > From: Lucas Santos <ilucas.ms@xxxxxxxxx>
> >
> > This patch removes all license boilerplate texts from the .c and .h
> > files at drivers/iio/adc/ and, instead, adds the proper SPDX license
> > identifiers.
> >
> > Signed-off-by: Lucas Santos <ilucas.ms@xxxxxxxxx>
> > Signed-off-by: Matheus Tavares <matheus.bernardino@xxxxxx>
> > ---
> > drivers/iio/adc/ad7266.c | 3 +--
> > drivers/iio/adc/ad7291.c | 3 +--
> > drivers/iio/adc/ad7298.c | 3 +--
> > drivers/iio/adc/ad7766.c | 3 +--
> > drivers/iio/adc/ad7791.c | 3 +--
> > drivers/iio/adc/ad7793.c | 3 +--
> > drivers/iio/adc/ad7887.c | 3 +--
> > drivers/iio/adc/ad7923.c | 3 +--
> > drivers/iio/adc/ad799x.c | 5 +----
> > drivers/iio/adc/ad_sigma_delta.c | 3 +--
> > drivers/iio/adc/aspeed_adc.c | 6 +-----
> > drivers/iio/adc/at91-sama5d2_adc.c | 10 +---------
> > drivers/iio/adc/at91_adc.c | 3 +--
> > drivers/iio/adc/axp20x_adc.c | 5 +----
> > drivers/iio/adc/axp288_adc.c | 13 +------------
> > drivers/iio/adc/bcm_iproc_adc.c | 13 +------------
> > drivers/iio/adc/berlin2-adc.c | 5 +----
> > drivers/iio/adc/cc10001_adc.c | 6 +-----
> > drivers/iio/adc/cpcap-adc.c | 10 +---------
> > drivers/iio/adc/da9150-gpadc.c | 6 +-----
> > drivers/iio/adc/dln2-adc.c | 5 +----
> > drivers/iio/adc/ep93xx_adc.c | 5 +----
> > drivers/iio/adc/exynos_adc.c | 11 +----------
> > drivers/iio/adc/fsl-imx25-gcq.c | 5 +----
> > drivers/iio/adc/hi8435.c | 6 +-----
> > drivers/iio/adc/hx711.c | 6 +-----
> > drivers/iio/adc/imx7d_adc.c | 6 +-----
> > drivers/iio/adc/ina2xx-adc.c | 7 +------
> > drivers/iio/adc/lp8788_adc.c | 5 +----
> > drivers/iio/adc/lpc18xx_adc.c | 5 +----
> > drivers/iio/adc/lpc32xx_adc.c | 15 +--------------
> > drivers/iio/adc/ltc2471.c | 3 +--
> > drivers/iio/adc/ltc2485.c | 5 +----
> > drivers/iio/adc/ltc2497.c | 3 +--
> > drivers/iio/adc/max1027.c | 5 +----
> > drivers/iio/adc/max1118.c | 5 +----
> > drivers/iio/adc/max1363.c | 5 +----
> > drivers/iio/adc/mcp320x.c | 5 +----
> > drivers/iio/adc/mcp3422.c | 6 +-----
> > drivers/iio/adc/men_z188_adc.c | 5 +----
> > drivers/iio/adc/meson_saradc.c | 5 +----
> > drivers/iio/adc/mt6577_auxadc.c | 10 +---------
> > drivers/iio/adc/mxs-lradc-adc.c | 11 +----------
> > drivers/iio/adc/nau7802.c | 3 +--
> > drivers/iio/adc/palmas_gpadc.c | 5 +----
> > drivers/iio/adc/qcom-pm8xxx-xoadc.c | 1 +
> > drivers/iio/adc/qcom-spmi-iadc.c | 10 +---------
> > drivers/iio/adc/qcom-spmi-vadc.c | 10 +---------
> > drivers/iio/adc/rockchip_saradc.c | 11 +----------
> > drivers/iio/adc/spear_adc.c | 3 +--
> > drivers/iio/adc/stx104.c | 10 +---------
> > drivers/iio/adc/sun4i-gpadc-iio.c | 5 +----
> > drivers/iio/adc/ti-adc081c.c | 5 +----
> > drivers/iio/adc/ti-adc0832.c | 5 +----
> > drivers/iio/adc/ti-adc084s021.c | 5 +----
> > drivers/iio/adc/ti-adc108s102.c | 10 +---------
> > drivers/iio/adc/ti-adc12138.c | 5 +----
> > drivers/iio/adc/ti-ads1015.c | 5 +----
> > drivers/iio/adc/ti-ads8688.c | 5 +----
> > drivers/iio/adc/ti-tlc4541.c | 5 +----
> > drivers/iio/adc/ti_am335x_adc.c | 10 +---------
> > drivers/iio/adc/twl4030-madc.c | 16 +---------------
> > drivers/iio/adc/twl6030-gpadc.c | 16 +---------------
> > drivers/iio/adc/vf610_adc.c | 15 +--------------
> > drivers/iio/adc/viperboard_adc.c | 7 +------
> > drivers/iio/adc/xilinx-xadc-core.c | 3 +--
> > drivers/iio/adc/xilinx-xadc-events.c | 3 +--
> > drivers/iio/adc/xilinx-xadc.h | 3 +--
> > 68 files changed, 68 insertions(+), 361 deletions(-)
> >
>
> (...)
>
> > diff --git a/drivers/iio/adc/ep93xx_adc.c b/drivers/iio/adc/ep93xx_adc.c
> > index 5036c392cb20..8f614c84b643 100644
> > --- a/drivers/iio/adc/ep93xx_adc.c
> > +++ b/drivers/iio/adc/ep93xx_adc.c
> > @@ -1,12 +1,9 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > /*
> > * Driver for ADC module on the Cirrus Logic EP93xx series of SoCs
> > *
> > * Copyright (C) 2015 Alexander Sverdlin
> > *
> > - * 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.
> > - *
> > * The driver uses polling to get the conversion status. According to EP93xx
> > * datasheets, reading ADCResult register starts the conversion, but user is also
> > * responsible for ensuring that delay between adjacent conversion triggers is
> > diff --git a/drivers/iio/adc/exynos_adc.c b/drivers/iio/adc/exynos_adc.c
> > index f10443f92e4c..9f9c186e2135 100644
> > --- a/drivers/iio/adc/exynos_adc.c
> > +++ b/drivers/iio/adc/exynos_adc.c
> > @@ -1,3 +1,4 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > /*
> > * exynos_adc.c - Support for ADC in EXYNOS SoCs
> > *
> > @@ -5,16 +6,6 @@
> > *
> > * Copyright (C) 2013 Naveen Krishna Chatradhi <ch.naveen@xxxxxxxxxxx>
> > *
> > - * 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 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., 675 Mass Ave, Cambridge, MA 02139, USA.
>
> The paragraph about writing to FSF should also go away, especially it
> is discouraged at first place (see checkpatch).
>

Nice catch! Thanks,
Matheus

> Best regards,
> Krzysztof