Re: [PATCH v4 2/3] iio: adc: ad9467: sort header includes
From: Andy Shevchenko
Date: Tue Dec 09 2025 - 09:42:06 EST
On Tue, Dec 9, 2025 at 7:34 AM Tomas Melin <tomas.melin@xxxxxxxxxxx> wrote:
>
> Include headers in ascending order.
Thanks, but...
> #include <linux/iio/backend.h>
> #include <linux/iio/iio.h>
> #include <linux/iio/sysfs.h>
...this was specifically grouped to show the relation to the certain
subsystem. At the end we should have something like this
linux/*.h // generic ones
...blank line...
asm/*.h // generic ones (optionally, if there is a need in a such)
...blank line...
linux/iio/*.h
...blank line...
So, what I meant with fixing clk.h is just to move it to the main
group and sort that.
You can check (recent enough) drivers for the examples. Yes, some of
them (but minority IIRC) might have linux/iio/*.h among linux/*.h, but
usually they are grouped separately.
--
With Best Regards,
Andy Shevchenko