Re: [PATCH 13/14] iio: pressure: mprls0025pa: cleanup includes and forward declarations
From: Marcelo Schmitt
Date: Fri Dec 19 2025 - 23:54:11 EST
On 12/18, Petre Rodan wrote:
> Remove unused headers and add required headers as needed.
Bring the patches that fix something to the beginning of the series and
bring this one right after the fixes.
>
> Signed-off-by: Petre Rodan <petre.rodan@xxxxxxxxxxxxxxx>
> ---
...
>
> diff --git a/drivers/iio/pressure/mprls0025pa.h b/drivers/iio/pressure/mprls0025pa.h
> index 2bdffe1e0eb1..ccd252f64351 100644
> --- a/drivers/iio/pressure/mprls0025pa.h
> +++ b/drivers/iio/pressure/mprls0025pa.h
> @@ -12,9 +12,6 @@
> #define _MPRLS0025PA_H
>
> #include <linux/completion.h>
> -#include <linux/delay.h>
> -#include <linux/device.h>
> -#include <linux/stddef.h>
> #include <linux/types.h>
>
> #include <linux/iio/iio.h>
> @@ -23,9 +20,6 @@
>
> struct device;
>
> -struct iio_chan_spec;
> -struct iio_dev;
> -
Also, I'd suggest to not mess with other stuff if this patch is supposedly only
organizing/cleaning up includes.
> struct mpr_data;
> struct mpr_ops;