Re: [PATCH] regulator: anatop: add is_enabled interface

From: Mark Brown
Date: Tue Dec 17 2013 - 08:53:14 EST


On Tue, Dec 17, 2013 at 01:10:12PM +0000, Anson.Huang@xxxxxxxxxxxxx wrote:

> > In any case I'd be much happier with this patch if it implemented the
> > enable and disable operations as well.

> understand now. then maybe I should remove the PU check in cpufreq,
> although setting PU LDO if it is off before would bring unnessary
> power leak. I will add all these changes together with the dynamic PU
> LDO management feature, thanks for your time!

If you've implemented the enable support correctly then setting the
voltage shouldn't enable the regulator.

Attachment: signature.asc
Description: Digital signature