Re: [PATCH v4 1/9] regulator: core: Add "enable and wait" functions

Mark Brown <[email protected]> Wed, 12 Aug 2026 19:02:25 +0100

Newsgroups	org.kernel.vger.linux-i2c,dev.linux.lists.chrome-platform,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-mediatek,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-input,org.kernel.vger.linux-kernel
Message-ID	<[email protected]>

On Wed, Aug 12, 2026 at 12:39:42PM +0800, Chen-Yu Tsai wrote:
> On Tue, Aug 11, 2026 at 8:43 PM Mark Brown <[email protected]> wrote:

> > Just take a timestamp during registration if the regulator is enabled,
> > we check if it's enabled anyway so we may as well take that as the last
> > time the enable happened.

> Yeah, I just haven't figured out where exactly in regulator_register() to
> put it. I think it should be after the supplies are resolved, so maybe
> I could do this in the "final setup" part in regulator_resolve_supply()?

Makes sense I think.  The other place would be while doing the
constraints but they're optional so might not happen.  Perhaps make the
one at the end of registration skip if there's a timestamp already?

signature.asc (application/pgp-signature, 488 B)