Re: [PATCH 2/2] regulator: core: add support for configuring turn-ontime through constraints

From: Stephen Warren
Date: Tue Sep 10 2013 - 13:34:23 EST


On 09/10/2013 10:51 AM, Mark Brown wrote:
> On Tue, Sep 10, 2013 at 09:11:13AM -0600, Stephen Warren wrote:
>
>>> +- regulator-enable-time: Turn ON time for regulator(in uS)
>
>> Bike-shedding slightly: This isn't really the time it takes to
>> enable a regulator, but the time the voltage takes to become
>> stable, or settle. Perhaps name the property
>> regulator-settle-time/regulator-settle-delay?
>
> The normal term would be ramp delay. It's not usually the time
> taken to completely settle, it's usually quoted as the time taken
> to reach within some proportion of the target voltage.

I notice there's a regulator-ramp-delay property, already documented
right above this new property. Is this a conflicting usage of the same
term, or should that existing property just be used in this case too?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/