On Wed, May 13, 2015 at 6:54 AM, Mark Brown <broonie@xxxxxxxxxx> wrote: > On Tue, May 12, 2015 at 02:57:29PM -0500, Nishanth Menon wrote: >> On 05/12/2015 02:41 PM, Mark Brown wrote: >> > On Tue, May 12, 2015 at 02:14:19PM -0500, Nishanth Menon wrote: > >> > specifying the currents involved. For plausible applications these are >> > likely to be ballpark figures rather than anything too accurate - if >> > nothing else the instantaneous current draw normally varies very >> > substantially so realistically you're talking about a maximum here. If >> > the corners vary that dramatically then I'd expect you'd see different >> > OPP tables being used anyway. > >> OK - If we state "worst case", then it is quantifiable (if SoC vendors >> would like to expose such information - I doubt mine ever will ;) ). >> - We might be able to quantify it better by stating worst case(under >> maximum load) steady state current (to avoid including transient >> spikes which are never representative) at ambient temperature(25C). > > Qualcomm do. It kind of depends on the system how worst case it needs > to be - it'll depend on the expected performance of both the system and > potentially the regulator. For some systems it may be important to have > some accounting for transients. Equally systems would only need to be > as accurate as the underlying hardware was able to make use of. True, some parts of the system like choice in regulator sizing has to account for transients. How about rephrasing as following: - opp-microamp: The maximum current drawn by the device in microamperes considering system specific parameters (such as transients, process, aging, etc.) as necessary, at an ambient temperature of 25C. This may be used to set the most efficient regulator operating mode. Valid only if opp-microvolt is set for the OPP. Entries for multiple regulators must be present in the same order as regulators are specified in device's DT node. If this property isn't required for few regulators, then this should be marked as zero for them. If it isn't required for any regulator, then this property need not be present. --- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
