All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] iio: adis: Cleanout unused headers
@ 2021-06-03 19:36 Jonathan Cameron
  2021-06-04  7:12 ` Alexandru Ardelean
  0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Cameron @ 2021-06-03 19:36 UTC (permalink / raw)
  To: linux-iio; +Cc: Nuno Sá, Jonathan Cameron

From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

0-day recently added the include-what-you-use header checker and
it gave a warning on an adis patch.  As such I decided to
run it on all the adis drivers and see if it made sensible suggestions.

Note this doesn't represent a complete list of what it suggested changing
as I filtered out a few on the basis they are standard headers used to
effectively include a bunch of other headers.

Could split this into a patch per driver if people prefer.

Note to anyone else trying this tool is that it is somewhat
of a loose cannon so you will be wanting to carefully check any
suggestions before proposing patches!

I thought about also reorganising the headers whilst here, but
that would make this patch harder to read, or lead to another rather
noisy patch across most of the files.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
---
 drivers/iio/accel/adis16201.c         | 3 ---
 drivers/iio/accel/adis16209.c         | 3 ---
 drivers/iio/gyro/adis16136.c          | 5 -----
 drivers/iio/gyro/adis16260.c          | 5 -----
 drivers/iio/imu/adis.c                | 4 ----
 drivers/iio/imu/adis16400.c           | 7 -------
 drivers/iio/imu/adis16460.c           | 1 -
 drivers/iio/imu/adis16475.c           | 1 -
 drivers/iio/imu/adis16480.c           | 6 ------
 drivers/staging/iio/accel/adis16203.c | 6 ------
 drivers/staging/iio/accel/adis16240.c | 6 ------
 11 files changed, 47 deletions(-)

diff --git a/drivers/iio/accel/adis16201.c b/drivers/iio/accel/adis16201.c
index fe225990de24..7a434e2884d4 100644
--- a/drivers/iio/accel/adis16201.c
+++ b/drivers/iio/accel/adis16201.c
@@ -8,10 +8,7 @@
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
-#include <linux/mutex.h>
-#include <linux/slab.h>
 #include <linux/spi/spi.h>
-#include <linux/sysfs.h>
 
 #include <linux/iio/iio.h>
 #include <linux/iio/imu/adis.h>
diff --git a/drivers/iio/accel/adis16209.c b/drivers/iio/accel/adis16209.c
index 6c2d4a967de7..ac08e866d612 100644
--- a/drivers/iio/accel/adis16209.c
+++ b/drivers/iio/accel/adis16209.c
@@ -7,11 +7,8 @@
 
 #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/iio/iio.h>
 #include <linux/iio/imu/adis.h>
diff --git a/drivers/iio/gyro/adis16136.c b/drivers/iio/gyro/adis16136.c
index d248c8a4a924..36879f01e28c 100644
--- a/drivers/iio/gyro/adis16136.c
+++ b/drivers/iio/gyro/adis16136.c
@@ -6,19 +6,14 @@
  *   Author: Lars-Peter Clausen <lars@metafoo.de>
  */
 
-#include <linux/interrupt.h>
-#include <linux/delay.h>
-#include <linux/mutex.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/spi/spi.h>
-#include <linux/slab.h>
 #include <linux/sysfs.h>
 #include <linux/module.h>
 
 #include <linux/iio/iio.h>
 #include <linux/iio/sysfs.h>
-#include <linux/iio/buffer.h>
 #include <linux/iio/imu/adis.h>
 
 #include <linux/debugfs.h>
diff --git a/drivers/iio/gyro/adis16260.c b/drivers/iio/gyro/adis16260.c
index 56f930bbd341..66b6b7bd5e1b 100644
--- a/drivers/iio/gyro/adis16260.c
+++ b/drivers/iio/gyro/adis16260.c
@@ -5,17 +5,12 @@
  * Copyright 2010 Analog Devices Inc.
  */
 
-#include <linux/interrupt.h>
-#include <linux/mutex.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/spi/spi.h>
-#include <linux/sysfs.h>
 #include <linux/module.h>
 
 #include <linux/iio/iio.h>
-#include <linux/iio/sysfs.h>
-#include <linux/iio/buffer.h>
 #include <linux/iio/imu/adis.h>
 
 #define ADIS16260_STARTUP_DELAY	220 /* ms */
diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
index 319b64b2fd88..a5b421f42287 100644
--- a/drivers/iio/imu/adis.c
+++ b/drivers/iio/imu/adis.c
@@ -12,14 +12,10 @@
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/spi/spi.h>
-#include <linux/slab.h>
-#include <linux/sysfs.h>
 #include <linux/module.h>
 #include <asm/unaligned.h>
 
 #include <linux/iio/iio.h>
-#include <linux/iio/sysfs.h>
-#include <linux/iio/buffer.h>
 #include <linux/iio/imu/adis.h>
 
 #define ADIS_MSC_CTRL_DATA_RDY_EN	BIT(2)
diff --git a/drivers/iio/imu/adis16400.c b/drivers/iio/imu/adis16400.c
index cb8d3ffab6fc..641b4f7d066d 100644
--- a/drivers/iio/imu/adis16400.c
+++ b/drivers/iio/imu/adis16400.c
@@ -10,22 +10,15 @@
  * Copyright (c) 2011 Analog Devices Inc.
  */
 
-#include <linux/interrupt.h>
 #include <linux/irq.h>
-#include <linux/delay.h>
-#include <linux/mutex.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/spi/spi.h>
-#include <linux/slab.h>
-#include <linux/sysfs.h>
-#include <linux/list.h>
 #include <linux/module.h>
 #include <linux/debugfs.h>
 #include <linux/bitops.h>
 
 #include <linux/iio/iio.h>
-#include <linux/iio/sysfs.h>
 #include <linux/iio/buffer.h>
 #include <linux/iio/trigger_consumer.h>
 #include <linux/iio/imu/adis.h>
diff --git a/drivers/iio/imu/adis16460.c b/drivers/iio/imu/adis16460.c
index 73bf45e859b8..ba373d7aaa2b 100644
--- a/drivers/iio/imu/adis16460.c
+++ b/drivers/iio/imu/adis16460.c
@@ -5,7 +5,6 @@
  * Copyright 2019 Analog Devices Inc.
  */
 
-#include <linux/delay.h>
 #include <linux/module.h>
 #include <linux/spi/spi.h>
 
diff --git a/drivers/iio/imu/adis16475.c b/drivers/iio/imu/adis16475.c
index 5654c0c15426..d0e84e5dee6c 100644
--- a/drivers/iio/imu/adis16475.c
+++ b/drivers/iio/imu/adis16475.c
@@ -14,7 +14,6 @@
 #include <linux/iio/buffer.h>
 #include <linux/iio/iio.h>
 #include <linux/iio/imu/adis.h>
-#include <linux/iio/sysfs.h>
 #include <linux/iio/trigger_consumer.h>
 #include <linux/irq.h>
 #include <linux/lcm.h>
diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c
index a9c21a5ae05a..9ec0e61b484f 100644
--- a/drivers/iio/imu/adis16480.c
+++ b/drivers/iio/imu/adis16480.c
@@ -9,24 +9,18 @@
 #include <linux/bitfield.h>
 #include <linux/of_irq.h>
 #include <linux/interrupt.h>
-#include <linux/delay.h>
 #include <linux/math.h>
-#include <linux/mutex.h>
 #include <linux/device.h>
 #include <linux/kernel.h>
 #include <linux/spi/spi.h>
-#include <linux/slab.h>
-#include <linux/sysfs.h>
 #include <linux/module.h>
 #include <linux/lcm.h>
 #include <linux/swab.h>
 #include <linux/crc32.h>
 
 #include <linux/iio/iio.h>
-#include <linux/iio/sysfs.h>
 #include <linux/iio/buffer.h>
 #include <linux/iio/imu/adis.h>
-#include <linux/iio/triggered_buffer.h>
 #include <linux/iio/trigger_consumer.h>
 
 #include <linux/debugfs.h>
diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c
index b68304da288b..1d3026dae827 100644
--- a/drivers/staging/iio/accel/adis16203.c
+++ b/drivers/staging/iio/accel/adis16203.c
@@ -5,20 +5,14 @@
  * Copyright 2010 Analog Devices Inc.
  */
 
-#include <linux/delay.h>
 #include <linux/device.h>
 
-#include <linux/iio/buffer.h>
 #include <linux/iio/iio.h>
 #include <linux/iio/imu/adis.h>
-#include <linux/iio/sysfs.h>
 
 #include <linux/kernel.h>
 #include <linux/module.h>
-#include <linux/mutex.h>
-#include <linux/slab.h>
 #include <linux/spi/spi.h>
-#include <linux/sysfs.h>
 
 #define ADIS16203_STARTUP_DELAY 220 /* ms */
 
diff --git a/drivers/staging/iio/accel/adis16240.c b/drivers/staging/iio/accel/adis16240.c
index 8d3afc6dc755..2a8aa83b8d9e 100644
--- a/drivers/staging/iio/accel/adis16240.c
+++ b/drivers/staging/iio/accel/adis16240.c
@@ -5,20 +5,14 @@
  * Copyright 2010 Analog Devices Inc.
  */
 
-#include <linux/interrupt.h>
-#include <linux/irq.h>
-#include <linux/delay.h>
 #include <linux/device.h>
 #include <linux/kernel.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>
-#include <linux/iio/buffer.h>
 #include <linux/iio/imu/adis.h>
 
 #define ADIS16240_STARTUP_DELAY	220 /* ms */
-- 
2.31.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] iio: adis: Cleanout unused headers
  2021-06-03 19:36 [PATCH] iio: adis: Cleanout unused headers Jonathan Cameron
@ 2021-06-04  7:12 ` Alexandru Ardelean
  2021-06-04  8:30   ` Jonathan Cameron
  0 siblings, 1 reply; 3+ messages in thread
From: Alexandru Ardelean @ 2021-06-04  7:12 UTC (permalink / raw)
  To: Jonathan Cameron; +Cc: linux-iio, Nuno Sá, Jonathan Cameron

On Thu, Jun 3, 2021 at 10:36 PM Jonathan Cameron <jic23@kernel.org> wrote:
>
> From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>
> 0-day recently added the include-what-you-use header checker and
> it gave a warning on an adis patch.  As such I decided to
> run it on all the adis drivers and see if it made sensible suggestions.
>
> Note this doesn't represent a complete list of what it suggested changing
> as I filtered out a few on the basis they are standard headers used to
> effectively include a bunch of other headers.
>
> Could split this into a patch per driver if people prefer.
>
> Note to anyone else trying this tool is that it is somewhat
> of a loose cannon so you will be wanting to carefully check any
> suggestions before proposing patches!
>
> I thought about also reorganising the headers whilst here, but
> that would make this patch harder to read, or lead to another rather
> noisy patch across most of the files.
>

Reviewed-by: Alexandru Ardelean <ardeleanalex@gmail.com>

> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
>  drivers/iio/accel/adis16201.c         | 3 ---
>  drivers/iio/accel/adis16209.c         | 3 ---
>  drivers/iio/gyro/adis16136.c          | 5 -----
>  drivers/iio/gyro/adis16260.c          | 5 -----
>  drivers/iio/imu/adis.c                | 4 ----
>  drivers/iio/imu/adis16400.c           | 7 -------
>  drivers/iio/imu/adis16460.c           | 1 -
>  drivers/iio/imu/adis16475.c           | 1 -
>  drivers/iio/imu/adis16480.c           | 6 ------
>  drivers/staging/iio/accel/adis16203.c | 6 ------
>  drivers/staging/iio/accel/adis16240.c | 6 ------
>  11 files changed, 47 deletions(-)
>
> diff --git a/drivers/iio/accel/adis16201.c b/drivers/iio/accel/adis16201.c
> index fe225990de24..7a434e2884d4 100644
> --- a/drivers/iio/accel/adis16201.c
> +++ b/drivers/iio/accel/adis16201.c
> @@ -8,10 +8,7 @@
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/module.h>
> -#include <linux/mutex.h>
> -#include <linux/slab.h>
>  #include <linux/spi/spi.h>
> -#include <linux/sysfs.h>
>
>  #include <linux/iio/iio.h>
>  #include <linux/iio/imu/adis.h>
> diff --git a/drivers/iio/accel/adis16209.c b/drivers/iio/accel/adis16209.c
> index 6c2d4a967de7..ac08e866d612 100644
> --- a/drivers/iio/accel/adis16209.c
> +++ b/drivers/iio/accel/adis16209.c
> @@ -7,11 +7,8 @@
>
>  #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/iio/iio.h>
>  #include <linux/iio/imu/adis.h>
> diff --git a/drivers/iio/gyro/adis16136.c b/drivers/iio/gyro/adis16136.c
> index d248c8a4a924..36879f01e28c 100644
> --- a/drivers/iio/gyro/adis16136.c
> +++ b/drivers/iio/gyro/adis16136.c
> @@ -6,19 +6,14 @@
>   *   Author: Lars-Peter Clausen <lars@metafoo.de>
>   */
>
> -#include <linux/interrupt.h>
> -#include <linux/delay.h>
> -#include <linux/mutex.h>
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/spi/spi.h>
> -#include <linux/slab.h>
>  #include <linux/sysfs.h>
>  #include <linux/module.h>
>
>  #include <linux/iio/iio.h>
>  #include <linux/iio/sysfs.h>
> -#include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
>
>  #include <linux/debugfs.h>
> diff --git a/drivers/iio/gyro/adis16260.c b/drivers/iio/gyro/adis16260.c
> index 56f930bbd341..66b6b7bd5e1b 100644
> --- a/drivers/iio/gyro/adis16260.c
> +++ b/drivers/iio/gyro/adis16260.c
> @@ -5,17 +5,12 @@
>   * Copyright 2010 Analog Devices Inc.
>   */
>
> -#include <linux/interrupt.h>
> -#include <linux/mutex.h>
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/spi/spi.h>
> -#include <linux/sysfs.h>
>  #include <linux/module.h>
>
>  #include <linux/iio/iio.h>
> -#include <linux/iio/sysfs.h>
> -#include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
>
>  #define ADIS16260_STARTUP_DELAY        220 /* ms */
> diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
> index 319b64b2fd88..a5b421f42287 100644
> --- a/drivers/iio/imu/adis.c
> +++ b/drivers/iio/imu/adis.c
> @@ -12,14 +12,10 @@
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/spi/spi.h>
> -#include <linux/slab.h>
> -#include <linux/sysfs.h>
>  #include <linux/module.h>
>  #include <asm/unaligned.h>
>
>  #include <linux/iio/iio.h>
> -#include <linux/iio/sysfs.h>
> -#include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
>
>  #define ADIS_MSC_CTRL_DATA_RDY_EN      BIT(2)
> diff --git a/drivers/iio/imu/adis16400.c b/drivers/iio/imu/adis16400.c
> index cb8d3ffab6fc..641b4f7d066d 100644
> --- a/drivers/iio/imu/adis16400.c
> +++ b/drivers/iio/imu/adis16400.c
> @@ -10,22 +10,15 @@
>   * Copyright (c) 2011 Analog Devices Inc.
>   */
>
> -#include <linux/interrupt.h>
>  #include <linux/irq.h>
> -#include <linux/delay.h>
> -#include <linux/mutex.h>
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/spi/spi.h>
> -#include <linux/slab.h>
> -#include <linux/sysfs.h>
> -#include <linux/list.h>
>  #include <linux/module.h>
>  #include <linux/debugfs.h>
>  #include <linux/bitops.h>
>
>  #include <linux/iio/iio.h>
> -#include <linux/iio/sysfs.h>
>  #include <linux/iio/buffer.h>
>  #include <linux/iio/trigger_consumer.h>
>  #include <linux/iio/imu/adis.h>
> diff --git a/drivers/iio/imu/adis16460.c b/drivers/iio/imu/adis16460.c
> index 73bf45e859b8..ba373d7aaa2b 100644
> --- a/drivers/iio/imu/adis16460.c
> +++ b/drivers/iio/imu/adis16460.c
> @@ -5,7 +5,6 @@
>   * Copyright 2019 Analog Devices Inc.
>   */
>
> -#include <linux/delay.h>
>  #include <linux/module.h>
>  #include <linux/spi/spi.h>
>
> diff --git a/drivers/iio/imu/adis16475.c b/drivers/iio/imu/adis16475.c
> index 5654c0c15426..d0e84e5dee6c 100644
> --- a/drivers/iio/imu/adis16475.c
> +++ b/drivers/iio/imu/adis16475.c
> @@ -14,7 +14,6 @@
>  #include <linux/iio/buffer.h>
>  #include <linux/iio/iio.h>
>  #include <linux/iio/imu/adis.h>
> -#include <linux/iio/sysfs.h>
>  #include <linux/iio/trigger_consumer.h>
>  #include <linux/irq.h>
>  #include <linux/lcm.h>
> diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c
> index a9c21a5ae05a..9ec0e61b484f 100644
> --- a/drivers/iio/imu/adis16480.c
> +++ b/drivers/iio/imu/adis16480.c
> @@ -9,24 +9,18 @@
>  #include <linux/bitfield.h>
>  #include <linux/of_irq.h>
>  #include <linux/interrupt.h>
> -#include <linux/delay.h>
>  #include <linux/math.h>
> -#include <linux/mutex.h>
>  #include <linux/device.h>
>  #include <linux/kernel.h>
>  #include <linux/spi/spi.h>
> -#include <linux/slab.h>
> -#include <linux/sysfs.h>
>  #include <linux/module.h>
>  #include <linux/lcm.h>
>  #include <linux/swab.h>
>  #include <linux/crc32.h>
>
>  #include <linux/iio/iio.h>
> -#include <linux/iio/sysfs.h>
>  #include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
> -#include <linux/iio/triggered_buffer.h>
>  #include <linux/iio/trigger_consumer.h>
>
>  #include <linux/debugfs.h>
> diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c
> index b68304da288b..1d3026dae827 100644
> --- a/drivers/staging/iio/accel/adis16203.c
> +++ b/drivers/staging/iio/accel/adis16203.c
> @@ -5,20 +5,14 @@
>   * Copyright 2010 Analog Devices Inc.
>   */
>
> -#include <linux/delay.h>
>  #include <linux/device.h>
>
> -#include <linux/iio/buffer.h>
>  #include <linux/iio/iio.h>
>  #include <linux/iio/imu/adis.h>
> -#include <linux/iio/sysfs.h>
>
>  #include <linux/kernel.h>
>  #include <linux/module.h>
> -#include <linux/mutex.h>
> -#include <linux/slab.h>
>  #include <linux/spi/spi.h>
> -#include <linux/sysfs.h>
>
>  #define ADIS16203_STARTUP_DELAY 220 /* ms */
>
> diff --git a/drivers/staging/iio/accel/adis16240.c b/drivers/staging/iio/accel/adis16240.c
> index 8d3afc6dc755..2a8aa83b8d9e 100644
> --- a/drivers/staging/iio/accel/adis16240.c
> +++ b/drivers/staging/iio/accel/adis16240.c
> @@ -5,20 +5,14 @@
>   * Copyright 2010 Analog Devices Inc.
>   */
>
> -#include <linux/interrupt.h>
> -#include <linux/irq.h>
> -#include <linux/delay.h>
>  #include <linux/device.h>
>  #include <linux/kernel.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>
> -#include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
>
>  #define ADIS16240_STARTUP_DELAY        220 /* ms */
> --
> 2.31.1
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] iio: adis: Cleanout unused headers
  2021-06-04  7:12 ` Alexandru Ardelean
@ 2021-06-04  8:30   ` Jonathan Cameron
  0 siblings, 0 replies; 3+ messages in thread
From: Jonathan Cameron @ 2021-06-04  8:30 UTC (permalink / raw)
  To: Alexandru Ardelean; +Cc: linux-iio, Nuno Sá, Jonathan Cameron

On Fri, 4 Jun 2021 10:12:53 +0300
Alexandru Ardelean <ardeleanalex@gmail.com> wrote:

> On Thu, Jun 3, 2021 at 10:36 PM Jonathan Cameron <jic23@kernel.org> wrote:
> >
> > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> >
> > 0-day recently added the include-what-you-use header checker and
> > it gave a warning on an adis patch.  As such I decided to
> > run it on all the adis drivers and see if it made sensible suggestions.
> >
> > Note this doesn't represent a complete list of what it suggested changing
> > as I filtered out a few on the basis they are standard headers used to
> > effectively include a bunch of other headers.
> >
> > Could split this into a patch per driver if people prefer.
> >
> > Note to anyone else trying this tool is that it is somewhat
> > of a loose cannon so you will be wanting to carefully check any
> > suggestions before proposing patches!
> >
> > I thought about also reorganising the headers whilst here, but
> > that would make this patch harder to read, or lead to another rather
> > noisy patch across most of the files.
> >  
> 
> Reviewed-by: Alexandru Ardelean <ardeleanalex@gmail.com>

Given it's trivial (hopefully) but I want some decent build coverage,
I've applied this to the togreg branch of iio.git and pushed out as testing
for the autobuilders to poke at it.

Thanks,

Jonathan

> 
> > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > ---
> >  drivers/iio/accel/adis16201.c         | 3 ---
> >  drivers/iio/accel/adis16209.c         | 3 ---
> >  drivers/iio/gyro/adis16136.c          | 5 -----
> >  drivers/iio/gyro/adis16260.c          | 5 -----
> >  drivers/iio/imu/adis.c                | 4 ----
> >  drivers/iio/imu/adis16400.c           | 7 -------
> >  drivers/iio/imu/adis16460.c           | 1 -
> >  drivers/iio/imu/adis16475.c           | 1 -
> >  drivers/iio/imu/adis16480.c           | 6 ------
> >  drivers/staging/iio/accel/adis16203.c | 6 ------
> >  drivers/staging/iio/accel/adis16240.c | 6 ------
> >  11 files changed, 47 deletions(-)
> >
> > diff --git a/drivers/iio/accel/adis16201.c b/drivers/iio/accel/adis16201.c
> > index fe225990de24..7a434e2884d4 100644
> > --- a/drivers/iio/accel/adis16201.c
> > +++ b/drivers/iio/accel/adis16201.c
> > @@ -8,10 +8,7 @@
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/module.h>
> > -#include <linux/mutex.h>
> > -#include <linux/slab.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/sysfs.h>
> >
> >  #include <linux/iio/iio.h>
> >  #include <linux/iio/imu/adis.h>
> > diff --git a/drivers/iio/accel/adis16209.c b/drivers/iio/accel/adis16209.c
> > index 6c2d4a967de7..ac08e866d612 100644
> > --- a/drivers/iio/accel/adis16209.c
> > +++ b/drivers/iio/accel/adis16209.c
> > @@ -7,11 +7,8 @@
> >
> >  #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/iio/iio.h>
> >  #include <linux/iio/imu/adis.h>
> > diff --git a/drivers/iio/gyro/adis16136.c b/drivers/iio/gyro/adis16136.c
> > index d248c8a4a924..36879f01e28c 100644
> > --- a/drivers/iio/gyro/adis16136.c
> > +++ b/drivers/iio/gyro/adis16136.c
> > @@ -6,19 +6,14 @@
> >   *   Author: Lars-Peter Clausen <lars@metafoo.de>
> >   */
> >
> > -#include <linux/interrupt.h>
> > -#include <linux/delay.h>
> > -#include <linux/mutex.h>
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/slab.h>
> >  #include <linux/sysfs.h>
> >  #include <linux/module.h>
> >
> >  #include <linux/iio/iio.h>
> >  #include <linux/iio/sysfs.h>
> > -#include <linux/iio/buffer.h>
> >  #include <linux/iio/imu/adis.h>
> >
> >  #include <linux/debugfs.h>
> > diff --git a/drivers/iio/gyro/adis16260.c b/drivers/iio/gyro/adis16260.c
> > index 56f930bbd341..66b6b7bd5e1b 100644
> > --- a/drivers/iio/gyro/adis16260.c
> > +++ b/drivers/iio/gyro/adis16260.c
> > @@ -5,17 +5,12 @@
> >   * Copyright 2010 Analog Devices Inc.
> >   */
> >
> > -#include <linux/interrupt.h>
> > -#include <linux/mutex.h>
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/sysfs.h>
> >  #include <linux/module.h>
> >
> >  #include <linux/iio/iio.h>
> > -#include <linux/iio/sysfs.h>
> > -#include <linux/iio/buffer.h>
> >  #include <linux/iio/imu/adis.h>
> >
> >  #define ADIS16260_STARTUP_DELAY        220 /* ms */
> > diff --git a/drivers/iio/imu/adis.c b/drivers/iio/imu/adis.c
> > index 319b64b2fd88..a5b421f42287 100644
> > --- a/drivers/iio/imu/adis.c
> > +++ b/drivers/iio/imu/adis.c
> > @@ -12,14 +12,10 @@
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/slab.h>
> > -#include <linux/sysfs.h>
> >  #include <linux/module.h>
> >  #include <asm/unaligned.h>
> >
> >  #include <linux/iio/iio.h>
> > -#include <linux/iio/sysfs.h>
> > -#include <linux/iio/buffer.h>
> >  #include <linux/iio/imu/adis.h>
> >
> >  #define ADIS_MSC_CTRL_DATA_RDY_EN      BIT(2)
> > diff --git a/drivers/iio/imu/adis16400.c b/drivers/iio/imu/adis16400.c
> > index cb8d3ffab6fc..641b4f7d066d 100644
> > --- a/drivers/iio/imu/adis16400.c
> > +++ b/drivers/iio/imu/adis16400.c
> > @@ -10,22 +10,15 @@
> >   * Copyright (c) 2011 Analog Devices Inc.
> >   */
> >
> > -#include <linux/interrupt.h>
> >  #include <linux/irq.h>
> > -#include <linux/delay.h>
> > -#include <linux/mutex.h>
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/slab.h>
> > -#include <linux/sysfs.h>
> > -#include <linux/list.h>
> >  #include <linux/module.h>
> >  #include <linux/debugfs.h>
> >  #include <linux/bitops.h>
> >
> >  #include <linux/iio/iio.h>
> > -#include <linux/iio/sysfs.h>
> >  #include <linux/iio/buffer.h>
> >  #include <linux/iio/trigger_consumer.h>
> >  #include <linux/iio/imu/adis.h>
> > diff --git a/drivers/iio/imu/adis16460.c b/drivers/iio/imu/adis16460.c
> > index 73bf45e859b8..ba373d7aaa2b 100644
> > --- a/drivers/iio/imu/adis16460.c
> > +++ b/drivers/iio/imu/adis16460.c
> > @@ -5,7 +5,6 @@
> >   * Copyright 2019 Analog Devices Inc.
> >   */
> >
> > -#include <linux/delay.h>
> >  #include <linux/module.h>
> >  #include <linux/spi/spi.h>
> >
> > diff --git a/drivers/iio/imu/adis16475.c b/drivers/iio/imu/adis16475.c
> > index 5654c0c15426..d0e84e5dee6c 100644
> > --- a/drivers/iio/imu/adis16475.c
> > +++ b/drivers/iio/imu/adis16475.c
> > @@ -14,7 +14,6 @@
> >  #include <linux/iio/buffer.h>
> >  #include <linux/iio/iio.h>
> >  #include <linux/iio/imu/adis.h>
> > -#include <linux/iio/sysfs.h>
> >  #include <linux/iio/trigger_consumer.h>
> >  #include <linux/irq.h>
> >  #include <linux/lcm.h>
> > diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c
> > index a9c21a5ae05a..9ec0e61b484f 100644
> > --- a/drivers/iio/imu/adis16480.c
> > +++ b/drivers/iio/imu/adis16480.c
> > @@ -9,24 +9,18 @@
> >  #include <linux/bitfield.h>
> >  #include <linux/of_irq.h>
> >  #include <linux/interrupt.h>
> > -#include <linux/delay.h>
> >  #include <linux/math.h>
> > -#include <linux/mutex.h>
> >  #include <linux/device.h>
> >  #include <linux/kernel.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/slab.h>
> > -#include <linux/sysfs.h>
> >  #include <linux/module.h>
> >  #include <linux/lcm.h>
> >  #include <linux/swab.h>
> >  #include <linux/crc32.h>
> >
> >  #include <linux/iio/iio.h>
> > -#include <linux/iio/sysfs.h>
> >  #include <linux/iio/buffer.h>
> >  #include <linux/iio/imu/adis.h>
> > -#include <linux/iio/triggered_buffer.h>
> >  #include <linux/iio/trigger_consumer.h>
> >
> >  #include <linux/debugfs.h>
> > diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c
> > index b68304da288b..1d3026dae827 100644
> > --- a/drivers/staging/iio/accel/adis16203.c
> > +++ b/drivers/staging/iio/accel/adis16203.c
> > @@ -5,20 +5,14 @@
> >   * Copyright 2010 Analog Devices Inc.
> >   */
> >
> > -#include <linux/delay.h>
> >  #include <linux/device.h>
> >
> > -#include <linux/iio/buffer.h>
> >  #include <linux/iio/iio.h>
> >  #include <linux/iio/imu/adis.h>
> > -#include <linux/iio/sysfs.h>
> >
> >  #include <linux/kernel.h>
> >  #include <linux/module.h>
> > -#include <linux/mutex.h>
> > -#include <linux/slab.h>
> >  #include <linux/spi/spi.h>
> > -#include <linux/sysfs.h>
> >
> >  #define ADIS16203_STARTUP_DELAY 220 /* ms */
> >
> > diff --git a/drivers/staging/iio/accel/adis16240.c b/drivers/staging/iio/accel/adis16240.c
> > index 8d3afc6dc755..2a8aa83b8d9e 100644
> > --- a/drivers/staging/iio/accel/adis16240.c
> > +++ b/drivers/staging/iio/accel/adis16240.c
> > @@ -5,20 +5,14 @@
> >   * Copyright 2010 Analog Devices Inc.
> >   */
> >
> > -#include <linux/interrupt.h>
> > -#include <linux/irq.h>
> > -#include <linux/delay.h>
> >  #include <linux/device.h>
> >  #include <linux/kernel.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>
> > -#include <linux/iio/buffer.h>
> >  #include <linux/iio/imu/adis.h>
> >
> >  #define ADIS16240_STARTUP_DELAY        220 /* ms */
> > --
> > 2.31.1
> >  


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-06-04  8:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-03 19:36 [PATCH] iio: adis: Cleanout unused headers Jonathan Cameron
2021-06-04  7:12 ` Alexandru Ardelean
2021-06-04  8:30   ` Jonathan Cameron

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.