Re: [PATCH 02/10] serial: clps711x: Switch to SPDX identifier

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 04, 2019 at 11:46:55AM +0100, Roland Hieber wrote:
> Hi Alexander,
> 
> On Wed, Dec 19, 2018 at 11:03:55AM +0300, Alexander Shiyan wrote:
> > Adopt the SPDX license identifier headers to ease license compliance
> > management.
> > 
> > Signed-off-by: Alexander Shiyan <shc_work@xxxxxxx>
> > ---
> >  drivers/serial/serial_clps711x.c | 12 ++----------
> >  1 file changed, 2 insertions(+), 10 deletions(-)
> > 
> > diff --git a/drivers/serial/serial_clps711x.c b/drivers/serial/serial_clps711x.c
> > index 9a9ae24c0..c21ca6510 100644
> > --- a/drivers/serial/serial_clps711x.c
> > +++ b/drivers/serial/serial_clps711x.c
> > @@ -1,13 +1,5 @@
> > -/*
> > - * Simple CLPS711X serial driver
> > - *
> > - * (C) Copyright 2012-2014 Alexander Shiyan <shc_work@xxxxxxx>
> > - *
> > - * 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.
> > - */
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/* Author: Alexander Shiyan <shc_work@xxxxxxx> */
> 
> Your removed the "Copyright" here and replaced it with "Author", but in
> some jurisdictions copyright and authorship are actually different
> things, and most automated license compliance tools only look for the
> string "Copyright", so your change will probably mean that your name
> will not be picked up as copyright. Was this change made deliberately?
> 
> Same goes for "[PATCH 06/10] gpio: clps711x: Switch to SPDX identifier".

and "[PATCH 09/10] clocksource: clps711x: Switch to SPDX identifier".

 - Roland

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux