From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laxman Dewangan Subject: Re: [PATCH V2] regulator: core: add support for configuring turn-on time through constraints Date: Wed, 11 Sep 2013 23:39:18 +0530 Message-ID: <5230B1CE.3090709@nvidia.com> References: <1378904331-5665-1-git-send-email-ldewangan@nvidia.com> <5230A593.9010401@wwwdotorg.org> <5230AC5E.2020504@nvidia.com> <5230AC61.30904@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5230AC61.30904@wwwdotorg.org> Sender: linux-kernel-owner@vger.kernel.org To: Stephen Warren Cc: "broonie@kernel.org" , "rob.herring@calxeda.com" , "mark.rutland@arm.com" , "rob@landley.net" , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "lgirdwood@gmail.com" List-Id: devicetree@vger.kernel.org On Wednesday 11 September 2013 11:16 PM, Stephen Warren wrote: > On 09/11/2013 11:46 AM, Laxman Dewangan wrote: >> On Wednesday 11 September 2013 10:47 PM, Stephen Warren wrote: >>> - regulator-enable-ramp-delay: The time taken, in uSec, for the supply >>> rail to reach the target voltage, plus/minus whatever tolerance the >>> board design requires, once the regulator output itself has ramped up. >>> This value is in addition to whatever built-in ramp time is inherent in >>> the regulator's own internal design or configuration. This property >>> describes the additional ramp time required due to board design issues >>> such as trace capacitance and load on the supply. >>> >>> That's text repeats "additional" a bit, but I think describes the >>> situation correctly? >> I wanted to provide the absolute delay rather than additional delay on >> top of inherit delay from device. > I suppose that either is fine from a DT perspective. But, the regulator > drivers already know their internal delay, so presumably driver code > will have to take the value from DT, and subtract out whatever delay the > driver already embodies, in order to calculate the extra delay required? > Or, if this property is set, does the driver-specified delay just get > ignored? Yes, if property is available then driver-specified delay get ignored. Delay will be used from dt provided value. Thanks, Laxman