On Sun, 2018-02-18 at 17:01 +0530, Himanshu Jha wrote: > Hi Shreeya, > Hi Himanshu, > On Sat, Feb 17, 2018 at 09:34:56PM +0530, Shreeya Patel wrote: > > > > Use SPDX identifier format instead of GPLv2. Also rearrange the > > headers in alphabetical order. > > > > Signed-off-by: Shreeya Patel <shreeya.patel23498@xxxxxxxxx> > > --- > > drivers/staging/iio/accel/adis16209.c | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/staging/iio/accel/adis16209.c > > b/drivers/staging/iio/accel/adis16209.c > > index 7fcef9a..e3d9f80 100644 > > --- a/drivers/staging/iio/accel/adis16209.c > > +++ b/drivers/staging/iio/accel/adis16209.c > > @@ -1,19 +1,18 @@ > > +// SPDX-License-Identifier: GPL-2.0+ > > /* > > * ADIS16209 Dual-Axis Digital Inclinometer and Accelerometer > > * > > * Copyright 2010 Analog Devices Inc. > > - * > > - * Licensed under the GPL-2 or later. > I see that you too are doing similar cleanup which I did a while ago > https://lkml.org/lkml/2018/2/12/255 Yes, Jonathan suggested me to work on adis16209. Your patches were quite useful for me :) > where I got some update suggestions for the patch series. It would be > great if you could update this patch series consistent with the > reviewers. > > For eg: in this patch you changed > > +// SPDX-License-Identifier: GPL-2.0+ > > and therefore > > MODULE_LICENSE("GPL v2"); > > should also be changed to > > MODULE_LICENSE("GPL"); > > as explained by Philippe Ombredanne to me in my patch series. Great. I'll make the necessary changes. > > I am not sure if you're subscribed to the IIO mailing list but you > can find > all the update suggestions from the above link. :) Thanks for your suggestions > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
