Re: [PATCH V6 2/6] dt-bindings: mfd: pm8008: Add regulators node

[Satya Priya Kakitapalli (Temp)]

On 2/16/2022 10:01 AM, Subbaraman Narayanamurthy wrote:
> On 2/15/22 12:53 AM, Satya Priya wrote:
> > @@ -102,7 +121,7 @@ examples:
> >       qupv3_se13_i2c {
> >         #address-cells = <1>;
> >         #size-cells = <0>;
> > -      pm8008i@8 {
> > +      pm8008_infra: pm8008@8 {
> >           compatible = "qcom,pm8008";
> >           reg = <0x8>;
> >           #address-cells = <1>;
> > @@ -123,6 +142,26 @@ examples:
> >             #interrupt-cells = <2>;
> >           };
> >         };
> > -    };
> >   
> > +      pm8008_regulators: pm8008@9 {
> > +        compatible = "qcom,pm8008";
> Should this be "qcom,pm8008-regulators" ?


Yes, I'll correct it.


Thanks.


> > +        reg = <0x9>;
> > +        #address-cells = <1>;
> > +        #size-cells = <0>;
> > +
> > +        vdd_l1_l2-supply = <&vreg_s8b_1p2>;
> > +        vdd_l3_l4-supply = <&vreg_s1b_1p8>;
> > +        vdd_l5-supply = <&vreg_bob>;
> > +        vdd_l6-supply = <&vreg_bob>;
> > +        vdd_l7-supply = <&vreg_bob>;
> > +
> > +        regulators {
> > +          pm8008_l1: ldo1 {
> > +            regulator-name = "pm8008_l1";
> > +            regulator-min-microvolt = <950000>;
> > +            regulator-max-microvolt = <1300000>;
> > +          };
> > +        };
> > +      };
> > +    };
> >   ...
> -Subbaraman