lkml.org 
[lkml]   [2014]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 4/4] ARM: dts: bcm590xx: add support for GPLDO and VBUS regulators
    On Wed, May 21, 2014 at 10:29:13AM +0100, Lee Jones wrote:
    > > Adds additional nodes to support GPLDO1-6 and VBUS regulators which
    > > are now supported in the bcm590xx regulator driver.
    > >
    > > Signed-off-by: Matt Porter <mporter@linaro.org>
    > > ---
    > > arch/arm/boot/dts/bcm59056.dtsi | 21 +++++++++++++++++++++
    > > 1 file changed, 21 insertions(+)
    >
    > I'm not going to apply this with the rest of the set. It will have to
    > go in via your normal arch/arm route.

    Ok

    > > diff --git a/arch/arm/boot/dts/bcm59056.dtsi b/arch/arm/boot/dts/bcm59056.dtsi
    > > index dfadaaa..066adfb 100644
    > > --- a/arch/arm/boot/dts/bcm59056.dtsi
    > > +++ b/arch/arm/boot/dts/bcm59056.dtsi
    > > @@ -70,5 +70,26 @@
    > >
    > > vsr_reg: vsr {
    > > };
    > > +
    > > + gpldo1_reg: gpldo1 {
    > > + };
    >
    > What do these empty nodes do in any case?

    They instantiate regulators. The bcm590xx binding specifies the
    allowable subnode names permitted here. This replaces the old method
    of specifying an index or the deprecated regulator-compatible property.
    The board-specific dts includes this dtsi and applies any board
    constraints to a subset of regulators that need them. Unused regulators
    are all disabled which is what we want.

    > Don't you at least need a status property in there or something?

    Not necessary for these subnodes.

    -Matt



    \
     
     \ /
      Last update: 2014-05-21 14:41    [W:2.532 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site