[PATCH v2 2/5] iio: adc: bcm_iproc_adc: fix include dependencies
Andy Shevchenko
andriy.shevchenko at intel.com
Mon Aug 31 23:57:00 PDT 2026
On Mon, Aug 31, 2026 at 11:27:37PM -0300, Marcelo Schmitt wrote:
> On 08/30, mdshahid03 at gmail.com wrote:
> > Add the headers required directly by the driver for completion and
> > mutex APIs.
> >
> > Remove the unused `linux/io.h` header now that register access is
> > handled through regmap APIs.
...
> > #include <linux/clk.h>
> > +#include <linux/completion.h>
> > #include <linux/delay.h>
> > #include <linux/interrupt.h>
> > -#include <linux/io.h>
> > #include <linux/mfd/syscon.h>
> > #include <linux/module.h>
> > +#include <linux/mutex.h>
> > #include <linux/platform_device.h>
> > #include <linux/regmap.h>
>
> I think that's okay. Though, if you go for a v3, you may also add
> #include <linux/bits.h>
> #include <linux/types.h>
> #include <linux/array_size.h>
> #include <linux/device-id/of.h>
This one is covered by platform_device.h.
> #include <linux/iio/types.h>
This one is covered by iio/iio.h.
> which mostly completes the list of includes iwyu tool reports to be missing.
The rest is indeed needs to be added. Thanks for spotting that!
--
With Best Regards,
Andy Shevchenko
More information about the linux-arm-kernel
mailing list