Re: [PATCH v4 4/4] regulator: Prevent falling too fast

[Mark Brown]

On Mon, Sep 19, 2016 at 11:39:24AM -0700, Doug Anderson wrote:
> On Fri, Sep 16, 2016 at 9:32 AM, Mark Brown <broonie@xxxxxxxxxx> wrote:

> > So the PWM is just configuring this external regulator chip (which
> > doesn't seem to be described in DT...) and that's just incredibly bad at
> > coping with voltage changes?  It does sound rather like we ought to be
> > representing this chip directly in case it needs other workarounds.

> I'm not 100% sure you can blame the regulator chip.  What we describe
> in the device tree as a "PWM Regulator" is actually:

> 1. Some discreet buck regulator whose output voltage is configured by
> adjusting an input voltage.  AKA: the buck regulator has "3" inputs:
> vin, vout, config.  You put a certain voltage on the "config" pin and
> that controls the value that comes out of "vout".

> 2. A network of resistors, capacitors, and inductors that take the
> output of a PWM and filter / smooth it enough that it can be a good
> config input to the discreet buck.

Ugh, right.  So you're using the PWM regulator output voltage as an
input to this other regulator.  TBH that sounds even more like this
other regulator should be represented in DT as the consumer of the PWM
regulator, the PWM regulator is not actually producing the voltages
claimed directly.

> The actual behavior of the PWM regulator depends as much (or more) on
> what values you have for the resistors, capacitors, and inductors than
> it does on the actual buck.  ...so two people using the same discreet
> buck might have very different behaviors in terms of rise time and how
> much they are impacted by the over voltage protection.

Right, these are properties of the PWM regulator.  But for some reason
the DCDC is still incapable of understanding it's own transitions and
flags out of spec too easily?  That isn't really a sign of high quality,
but then this does seem like the DCDC is really intended for a fixed
voltage application and is being abused in this system design to scale
dynamically so really it's a badly concieved hardware design I suppose.

Attachment: signature.asc
Description: PGP signature