Re: [PATCH 1/2] regulator: act8945a: rely on hardware for operating mode

[Raag Jadav <raagjadav@gmail.com>]

On Mon, Jun 05, 2023 at 05:30:36PM +0100, Mark Brown wrote:
> On Mon, Jun 05, 2023 at 09:46:14PM +0530, Raag Jadav wrote:
> > On Mon, Jun 05, 2023 at 03:44:37PM +0100, Mark Brown wrote:
> 
> > > It's also removing a cache so we need to talk to the hardware more often
> > > which doesn't seem like a win.  If there's a bootstapping problem then
> > > shouldn't we just read once at startup?  If there's no problem what's
> > > the advantage?
> 
> > Well, there could be cases of access which are done
> > outside of driver context or hardware failure,
> > which are not really ideal but I've faced such problems
> > a while back, so just decided to share it.
> 
> If we have those then we've got a bigger problem, for example we might
> switch to a lower power mode which can't support the load we're trying
> to get the consumer to run.

Yes, but in such cases it will atleast report whatever mode currently
set in the hardware and not the incorrect one stored locally in the memory,
and this way I think we have a better chance of finding out that
there indeed is a problem.

Cheers,
Raag