Re: [PATCH 2/6] iio: health: afe4403: Use devm_regulator_get_enable() helper

[Andrew Davis <afd@ti.com>]

On 9/13/25 9:20 AM, Jonathan Cameron wrote:
> On Wed, 13 Aug 2025 18:12:26 -0500
> David Lechner <dlechner@baylibre.com> wrote:
> 
>> On 8/13/25 5:58 PM, Andrew Davis wrote:
>>> This takes care of both getting and enabling the regulator in one
>>> function, it also handles the devm action, so remove that. Also
>>> do not disable the regulator on suspend, this will be handled for
>>
>> I didn't know that this worked automatically. What is the mechanism
>> that makes it work? I've seen lots of drivers doing the disable/
>> enable in suspend/resume, so I just always assumed that was how one
>> is supposed to do it.
>>
> Hi Andrew
> 
> This question is still open, so I'll not pick the reset of the series
> up until it's resolved.  I'm going to mark these in patchwork as
> changes requested so will need a v2 now.
> 

Sounds good, I'm still working on tracking down the answer to the
question. I think it involves the regulator suspend path but
will sort it out fully for v2.

Thanks,
Andrew

> 
> Jonathan
> 
>>> us. We now do not need to track the regulator at all, so drop it
>>> from the device struct.
>>>
>>> Signed-off-by: Andrew Davis <afd@ti.com>
>