Re: [PATCH v6 0/8] iio: add new backend framework

["Nuno Sá" <noname.nuno@gmail.com>]

On Sun, 2024-01-21 at 17:57 +0000, Jonathan Cameron wrote:
> On Fri, 19 Jan 2024 17:00:46 +0100
> Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@kernel.org> wrote:
> 
> 
> > Changes in v6:
> >  - Patch 2
> >   * Add '#io-backends-cells' property to the example dts.
> >  - Patch 6
> >   * Improve comment on struct iio_backend_buffer_pair;
> >   * Be consistent with dots in kernel-docs. So dots only exist now in
> >     description and return. Also be consistent on device managed APIs
> >     subject and @dev description;
> >   * Added devm_iio_backend_enable() and hence (and for now)
> >     iio_backend_disable is static;
> >   * Fixed __devm_iio_backend_get_from_fwnode_lookup() kernel-doc;
> >   * Improved english in devm_iio_backend_register() comment;
> >   * Don't make private data mandatory;
> >  - Patch 7
> >   * Use local ad9467_chip_info *info variable;
> >   * Improve error handling for optional devm_iio_backend_get();
> >   * Make use of devm_iio_backend_enable().
> >  - Patch 8
> >   * Also change string format in version error path.
> Hi Nuno,
> 
> Other than wanting a few acks (one for 2nd DT patch with the fix and one
> for the device link changes) this looks good to me.
> 
> Ideally of course I'd like to see the second user if that happens to be
> available in the first few weeks of this cycle, but I don't propose to hold
> this indefinitely to wait for additional users as the framework seems logical
> and it's all in kernel anyway so we can probably change whatever is needed
> later without too much trouble.
> 

Alright. I plan to start working on the next user already this week so let's see how
it goes.

- Nuno Sá

>