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

[Jonathan Cameron <jic23@kernel.org>]

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.

Jonathan

>