On Sat, 17 Feb 2018 21:34:56 +0530 Shreeya Patel <shreeya.patel23...@gmail.com> wrote:
> Use SPDX identifier format instead of GPLv2. Also rearrange the > headers in alphabetical order. As Dan pointed out for the Himanshu Jha's patches, the moment we see the word 'also' in a patch it pretty much makes it obvious that the author knows it is doing two things. Two patches please. This is all about making the changes trivial to review. Sometimes it may seem silly where we have two small changes like here, but when you are reviewing hundreds of patches it really does help. Jonathan > > Signed-off-by: Shreeya Patel <shreeya.patel23...@gmail.com> > --- > 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. > */ > > #include <linux/delay.h> > #include <linux/device.h> > #include <linux/kernel.h> > +#include <linux/list.h> > +#include <linux/module.h> > #include <linux/spi/spi.h> > #include <linux/slab.h> > #include <linux/sysfs.h> > -#include <linux/list.h> > -#include <linux/module.h> > > #include <linux/iio/iio.h> > #include <linux/iio/sysfs.h> _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
