lkml.org 
[lkml]   [2012]   [May]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/2] ARM: dt: tegra: cardhu: register core regulator tps65911
    On 05/22/2012 07:05 AM, Laxman Dewangan wrote:
    > Add device info for the PMIC device tps65911 in tegra-cardhu
    > dts file. This device supports the multiple regulator rails,
    > gpio, interrupts.

    FYI, patch 1 in this series looks fine. Some comments below though:

    > diff --git a/arch/arm/boot/dts/tegra-cardhu.dts b/arch/arm/boot/dts/tegra-cardhu.dts

    > + tps65911: tps65911@2d {
    > + compatible = "ti,tps65911";
    > + reg = <0x2d>;
    > +
    > + #gpio-cells = <2>;
    > + gpio-controller;
    > +
    > + regulators {

    Please add the following properties here:

    #address-cells = <1>;
    #size-cells = <0>;

    > + vdd1_reg: vdd1 {

    This node name should be "regulator", since nodes are generally named
    after the class of object they represent. Since all the nodes will then
    have the same name, you'll need to add a unit address ("@nnnn") to the
    node name.

    Nitpicky, but the labels might be more logical as reg_vdd1 rather than
    vdd1_reg, but not a big deal.

    So, please replace the line above with:

    reg_vdd1: regulator@0 {
    reg = <0>;

    > + regulator-name = "vdd1";
    > + regulator-min-microvolt = < 600000>;
    > + regulator-max-microvolt = <1500000>;
    > + regulator-always-on;
    > + regulator-boot-on;
    > + };
    > +
    > + vdd2_reg: vdd2 {

    And similarly, that would become:

    reg_vdd2: regulator@1 {
    reg = <1>;

    etc.


    \
     
     \ /
      Last update: 2012-05-22 19:21    [W:0.032 / U:124.240 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site