On Tue, Mar 28, 2017 at 03:39:41PM +0300, Leonard Crestez wrote: > On Fri, 2017-03-24 at 12:52 +0000, Mark Brown wrote: Please fix your mail client to word wrap within paragraphs at something substantially less than 80 columns. Doing this makes your messages much easier to read and reply to. > > to the supply. Usually bypass is used for low power retention modes > > with different settings to those used in normal operation that wouldn't > > be desired in normal operation, if we were going to have constraints for > > this I'd expect a separate set used during bypass. > In this particular case it's not possible to set constraints on the parent > regulator so that both ldo-enable and ldo-bypass modes work. The maximum allowed > voltage for ldo-bypass is lower than the minimum required to support the chip at > max frequency wit ldo-enable. If things are really so sensitive that you can't bypass without lowering the voltage then it's hard to see how you can safely transition into and out of bypass mode. > I'm not sure I understand why you are against applying constraints to the parent > when in bypass mode, it seems like the obvious thing to do if you want to > support flexible configuration. The check I introduced is probably not enough to > cover all cases, for example it would still be possible to explicitly change > parent voltage afterwards. To repeat what I said previously the whole point of bypassing is to not do regulation and generally the constraints in the unregulated idle case are substantially more relaxed. This would break use cases relying on the existing behaviour which wouldn't expect to affect the parent voltage at all, either stopping things working or making them less efficient by needlessly regulating the voltage down which defeats the main point of bypassing.
Attachment:
signature.asc
Description: PGP signature
