| Message ID | 1631875538-22473-2-git-send-email-skakit@codeaurora.org |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Add Qualcomm Technologies, Inc. PM8008 regulator driver | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dt-meta-schema | fail | build log |
On Fri, 17 Sep 2021 16:15:35 +0530, Satya Priya wrote: > Add pm8008-regulator node and example. > > Signed-off-by: Satya Priya <skakit@codeaurora.org> > --- > .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: Unknown file referenced: [Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/dtschema/schemas/regulator/qcom,pm8008-regulator.yaml' xargs: dt-doc-validate: exited with status 255; aborting make[1]: *** Deleting file 'Documentation/devicetree/bindings/mfd/qcom,pm8008.example.dt.yaml' Unknown file referenced: [Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/dtschema/schemas/regulator/qcom,pm8008-regulator.yaml' make[1]: *** [scripts/Makefile.lib:385: Documentation/devicetree/bindings/mfd/qcom,pm8008.example.dt.yaml] Error 255 make[1]: *** Waiting for unfinished jobs.... make: *** [Makefile:1441: dt_binding_check] Error 2 doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/patch/1529335 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit.
Quoting Satya Priya (2021-09-17 03:45:35) > Add pm8008-regulator node and example. > > Signed-off-by: Satya Priya <skakit@codeaurora.org> > --- > .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > index ec3138c..de182f8 100644 > --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > @@ -45,6 +45,10 @@ properties: > const: 0 > > patternProperties: > + "^pm8008[a-z]?-regulator$": Shouldn't it be ^pm8008-regulator$ without the a-z optional letter? > + type: object > + $ref: "../regulator/qcom,pm8008-regulator.yaml#" > + > "^gpio@[0-9a-f]+$": > type: object >
On Fri, Sep 17, 2021 at 04:15:35PM +0530, Satya Priya wrote: > Add pm8008-regulator node and example. > > Signed-off-by: Satya Priya <skakit@codeaurora.org> > --- > .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > index ec3138c..de182f8 100644 > --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml > @@ -45,6 +45,10 @@ properties: > const: 0 > > patternProperties: > + "^pm8008[a-z]?-regulator$": Is more than 1 node possible for a given PMIC? If not use 'regulators' for the node name. > + type: object > + $ref: "../regulator/qcom,pm8008-regulator.yaml#" > + > "^gpio@[0-9a-f]+$": > type: object > > @@ -122,6 +126,26 @@ examples: > interrupt-controller; > #interrupt-cells = <2>; > }; > + > + pm8008-regulator { > + compatible = "qcom,pm8008-regulator"; > + #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>; > + > + pm8008_l1: regulator@4000 { > + reg = <0x4000>; > + regulator-name = "pm8008_l1"; > + regulator-min-microvolt = <950000>; > + regulator-max-microvolt = <1300000>; > + qcom,min-dropout-voltage = <96000>; > + }; > + }; > }; > }; > > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member > of Code Aurora Forum, hosted by The Linux Foundation > >
On 2021-09-21 01:02, Stephen Boyd wrote: > Quoting Satya Priya (2021-09-17 03:45:35) >> Add pm8008-regulator node and example. >> >> Signed-off-by: Satya Priya <skakit@codeaurora.org> >> --- >> .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 >> ++++++++++++++++++++++ >> 1 file changed, 24 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> index ec3138c..de182f8 100644 >> --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> @@ -45,6 +45,10 @@ properties: >> const: 0 >> >> patternProperties: >> + "^pm8008[a-z]?-regulator$": > > Shouldn't it be ^pm8008-regulator$ without the a-z optional letter? > Some platforms use 2 PM8008 PMICS, in that case we need suffixing like pm8008i pm8008j etc. So, I mentioned this way. >> + type: object >> + $ref: "../regulator/qcom,pm8008-regulator.yaml#" >> + >> "^gpio@[0-9a-f]+$": >> type: object >>
On 2021-09-21 02:10, Rob Herring wrote: > On Fri, Sep 17, 2021 at 04:15:35PM +0530, Satya Priya wrote: >> Add pm8008-regulator node and example. >> >> Signed-off-by: Satya Priya <skakit@codeaurora.org> >> --- >> .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 >> ++++++++++++++++++++++ >> 1 file changed, 24 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> index ec3138c..de182f8 100644 >> --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml >> @@ -45,6 +45,10 @@ properties: >> const: 0 >> >> patternProperties: >> + "^pm8008[a-z]?-regulator$": > > Is more than 1 node possible for a given PMIC? If not use 'regulators' > for the node name. > Not possible, will change the node name. >> + type: object >> + $ref: "../regulator/qcom,pm8008-regulator.yaml#" >> + >> "^gpio@[0-9a-f]+$": >> type: object >> >> @@ -122,6 +126,26 @@ examples: >> interrupt-controller; >> #interrupt-cells = <2>; >> }; >> + >> + pm8008-regulator { >> + compatible = "qcom,pm8008-regulator"; >> + #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>; >> + >> + pm8008_l1: regulator@4000 { >> + reg = <0x4000>; >> + regulator-name = "pm8008_l1"; >> + regulator-min-microvolt = <950000>; >> + regulator-max-microvolt = <1300000>; >> + qcom,min-dropout-voltage = <96000>; >> + }; >> + }; >> }; >> }; >> >> -- >> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a >> member >> of Code Aurora Forum, hosted by The Linux Foundation >> >>
diff --git a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml index ec3138c..de182f8 100644 --- a/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml +++ b/Documentation/devicetree/bindings/mfd/qcom,pm8008.yaml @@ -45,6 +45,10 @@ properties: const: 0 patternProperties: + "^pm8008[a-z]?-regulator$": + type: object + $ref: "../regulator/qcom,pm8008-regulator.yaml#" + "^gpio@[0-9a-f]+$": type: object @@ -122,6 +126,26 @@ examples: interrupt-controller; #interrupt-cells = <2>; }; + + pm8008-regulator { + compatible = "qcom,pm8008-regulator"; + #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>; + + pm8008_l1: regulator@4000 { + reg = <0x4000>; + regulator-name = "pm8008_l1"; + regulator-min-microvolt = <950000>; + regulator-max-microvolt = <1300000>; + qcom,min-dropout-voltage = <96000>; + }; + }; }; };
Add pm8008-regulator node and example. Signed-off-by: Satya Priya <skakit@codeaurora.org> --- .../devicetree/bindings/mfd/qcom,pm8008.yaml | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+)