[PATCH 2/3] iio: adc: add Axiado SARADC driver
Petar Stepanovic
pstepanovic at axiado.com
Tue Jun 2 03:27:05 PDT 2026
Hi Joshua, Jonathan,
Thanks for the review. I will address the comments in v2.
[inline replies below]
On 5/28/2026 11:44 AM, Jonathan Cameron wrote:
> On Thu, 28 May 2026 11:02:05 +0200
> Joshua Crofts <joshua.crofts1 at gmail.com> wrote:
>
>> On Thu, 28 May 2026 at 10:11, Petar Stepanovic <pstepanovic at axiado.com> wrote:
>>> + return 0;
>>> +}
>>> +
>> I don't see any kind of cleanup procedure, like a remove() function
>> or devm_add_action_or_reset callback, is this intentional?
Thanks for pointing this out.
Yes, this was intentional because the driver only enables the ADC during probe,
and all resources are devm-managed.
However, I agree it would be better to explicitly leave the hardware in a safe
state on driver unbind. I will add a devm_add_action_or_reset() callback in v2
to disable the ADC / put it back into power-down state.
Best regards,
Petar
More information about the linux-arm-kernel
mailing list