RE: [PATCH v2] regulator: pwm: Fix regulator ramp delay for continuous mode

From	Aleksandr Frid <afrid@nvidia.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH v2] regulator: pwm: Fix regulator ramp delay for continuous mode
Date	2016-07-07 20:50 +0200
Message-ID	<rSmx5-2ew-55@gated-at.bofh.it> (permalink)
References	<rS03v-4kw-13@gated-at.bofh.it> <rSdk7-4BD-33@gated-at.bofh.it> <rSkvg-ZX-17@gated-at.bofh.it> <rSmdH-275-1@gated-at.bofh.it> <rSmno-2ax-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

>>
In the case that you don't need multiple steps then don't specify a regulator-ramp-delay, just a pwm-regulator-settle-us.
>>
Agreed

-----Original Message-----
From: dianders@google.com [mailto:dianders@google.com] On Behalf Of Doug Anderson
Sent: Thursday, July 07, 2016 11:32 AM
To: Aleksandr Frid
Cc: Laxman Dewangan; Mark Brown; Boris Brezillon; Lee Jones; Brian Norris; open list:ARM/Rockchip SoC...; Heiko Stuebner; Thierry Reding; Liam Girdwood; linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] regulator: pwm: Fix regulator ramp delay for continuous mode

Hi,

On Thu, Jul 7, 2016 at 11:23 AM, Aleksandr Frid <afrid@nvidia.com> wrote:
> Hi,
>
>>>
> In that case we should probably add a new PWM regulator property and not abuse the existing one.  Maybe you use "pwm-regulator-settle-us"
> or something?
>>>
> Looks reasonable to me.
>
>>>
> actually the right thing is probably to implement 
> 'regulator-ramp-delay' as doing several small steps in that case
>>>
> Ramp delay uV/us  is not a "real" metric for some PWM regulators with exponential transition -- as opposite to fixed slew-rate linear transition on other regulators. So splitting transition into multiple steps to implement artificial  (in this case) metric seems questionable.

In the case that you don't need multiple steps then don't specify a regulator-ramp-delay, just a pwm-regulator-settle-us.

...the suggestion for multiple steps is because (so I'm told) it helps avoid overshoot or undershoot problems.  In general the whole point of ramping a regulator slowly is to avoid overshoot or undershoot problems.  The "regulator-ramp-delay" property in Linux is a little odd because it sort of "describes" the ramp delay and sort of "sets"
the ramp delay.  Many PMICs allow you to set how fast the regulator will ramp and this property is used to specify how the register in the PMIC should be set.  However, it is also used as the actual delay in Linux.

-Doug

Thread

[PATCH v2] regulator: pwm: Fix regulator ramp delay for continuous mode Douglas Anderson <dianders@chromium.org> - 2016-07-06 20:50 +0200
  Re: [PATCH v2] regulator: pwm: Fix regulator ramp delay for continuous  mode Laxman Dewangan <ldewangan@nvidia.com> - 2016-07-07 11:00 +0200
    Re: [PATCH v2] regulator: pwm: Fix regulator ramp delay for  continuous mode Doug Anderson <dianders@chromium.org> - 2016-07-07 18:40 +0200
      RE: [PATCH v2] regulator: pwm: Fix regulator ramp delay for  continuous mode Aleksandr Frid <afrid@nvidia.com> - 2016-07-07 20:30 +0200
        Re: [PATCH v2] regulator: pwm: Fix regulator ramp delay for  continuous mode Doug Anderson <dianders@chromium.org> - 2016-07-07 20:40 +0200
          RE: [PATCH v2] regulator: pwm: Fix regulator ramp delay for  continuous mode Aleksandr Frid <afrid@nvidia.com> - 2016-07-07 20:50 +0200
            Re: [PATCH v2] regulator: pwm: Fix regulator ramp delay for  continuous mode Mark Brown <broonie@kernel.org> - 2016-07-08 11:00 +0200
  Applied "regulator: pwm: Fix regulator ramp delay for continuous mode" to the regulator tree Mark Brown <broonie@kernel.org> - 2016-07-07 12:10 +0200

csiph-web