Re: [PATCH v3 3/3] iio: adc: ad9467: drop kernel.h in favor of array_size.h

From: Andy Shevchenko
Date: Mon Dec 08 2025 - 11:23:17 EST


On Mon, Dec 08, 2025 at 06:21:25PM +0200, Tomas Melin wrote:
> On 08/12/2025 17:58, Andy Shevchenko wrote:
> > On Mon, Dec 08, 2025 at 05:41:20PM +0200, Tomas Melin wrote:
> >> On 08/12/2025 15:20, Andy Shevchenko wrote:
> >>> On Mon, Dec 08, 2025 at 12:30:59PM +0000, Tomas Melin wrote:

...

> > If you don't feel going that deep, than it's a (small) problem.
> >
> > As the author of a driver feature one should understand slightly more about
> > this (yeah, currently mess) header stuff. So, your first patch should add
> > missed headers, if any, that's required to your changes, this one can be
> > omitted after all.
> Well, I think reworking the headers is a sane idea, but it is not the
> topic of this series.
>
> > But on some day you will still need to understand a bit more about headers...
> >
> > TL;DR: make sure you have all needed headers for your changes in the previous
> > patch and drop this one.
>
> Hope we mean the same thing, I was thinking more like
> 1. sort current headers
> 2. add feature which adds the new header in correct location

Sure, it's even better!

--
With Best Regards,
Andy Shevchenko