On 23-09-16, 14:59, Rob Herring wrote:

On Wed, Sep 21, 2016 at 03:02:50PM +0530, Viresh Kumar wrote:

quoted

Multiple regulators per device aren't supported yet by the kernel code
and the bindings provided in documentation aren't sufficient to handle
that case (as there is no way for kernel code to link multiple
voltage/current values to a power supply).

What do you mean? Because the supplies are in the cpu node?

Not just that. For example if two supplies are present in CPU node like:
ABC-supply and XYZ-supply. And the values present in OPP node are like:

               opp00 {
                       opp-hz = /bits/ 64 <1700000000>;
                        /*                     ABC supply                XYZ        supply */
                       opp-microvolt = <1300000 1300000 1300000>, <1400000 1400000 1400000>;
                       opp-microamp = <70000>;
                       clock-latency-ns = <30>;
                       opp-suspend;
               };


Then the code can get regulators with those names (ABC and XYZ), but it can't
figure out which triplet in the "opp-microvolt" property belongs to which
supply.

I am working on some solution to that, but wanted to keep the entire series self
sufficient and so removing the leftovers first in this patch.

-- 
viresh