On 4 May 2015 at 17:42, Mark Brown <broonie@xxxxxxxxxx> wrote: > On Thu, Apr 30, 2015 at 05:37:59PM +0530, Viresh Kumar wrote: > >> +- opp-microamp: current in micro Amperes. It can contain entries for multiple >> + regulators. >> + >> + A single regulator's current is specified with an array of size one or three. >> + Single entry is for target current and three entries are for <target min max> >> + currents. > > What is this for - are you trying to define OPPs for current regulators? > If you are that's worrying, I can't think of a sensible use case. If > that's not what's happening then the binding needs to be more specific > about what this is supposed to mean. Its another property of the same voltage regulator we are using in opp-microvolt. I hope that makes sense ? And that's why I wrote this as well: + Entries for multiple regulators must be present in the same order as + regulators are specified in device's DT node. If few regulators don't provide + capability to configure current, then values for then should be marked as + zero. i.e. this is optional but the voltage regulator isn't.. But, perhaps I need to write it with more clarity? Or this field doesn't make sense ? FWIW, its an outcome of this request from Stephen: https://www.marc.info/?l=linux-arm-kernel&m=142370250522589&w=3 -- 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
