Re: [PATCH v3 6/8] iio: adc: ti-ads1015: convert to fully managed resources
Andy Shevchenko <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel |
|---|---|
| Organization | Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo |
| Message-ID | <[email protected]> |
On Wed, Aug 12, 2026 at 10:14:31PM +0530, Archit Anant wrote:
> Refactor the driver to use devm_ allocations and power
> management, allowing for the complete removal of the
> ads1015_remove()
...
> - ret = iio_device_register(indio_dev);
> - if (ret < 0) {
> + ret = devm_pm_runtime_set_active_enabled(dev);
> + if (ret)
There are more places with unneeded ' < 0' comparisons, can you add an
intermediate patch to drop those? With that being done, this, in particular,
will have less churn.
--
With Best Regards,
Andy Shevchenko