Re: [PATCH] regulator: twl: Enable regulators over the powerbus as well

From: Mark Brown
Date: Fri Mar 25 2016 - 13:23:37 EST


On Fri, Mar 25, 2016 at 05:47:50PM +0100, Sebastian Reichel wrote:

> I just tried to describe what's going on, so that you can see the
> whole picture. I don't agree with the patch and think that the mode
> should be switched to normal at probe time. I assumed, that you
> would suggest the correct solution if I describe the problem.

Well, I'm not 100% sure what's going on. If there is a constraint in
the system that requires setting the mode then something needs to
set that but it sounds like it might be more the case that the mode
support in this driver is just broken at the minute if it's not
orthogonal to the enable support. Perhaps the mode API is being used
for suspend states or something?

Attachment: signature.asc
Description: PGP signature