Message ID | 20240119-iio-backend-v6-0-189536c35a05@analog.com (mailing list archive) |
---|---|
Headers | show |
Series | iio: add new backend framework | expand |
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 >
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á >