| Message ID | 20231202192536.266885-1-biju.das.jz@bp.renesas.com |
|---|---|
| Headers | show |
| Series | Convert DA906{1,2} bindings to json-schema | expand |
On Sat, 02 Dec 2023 19:25:28 +0000, Biju Das wrote: > Convert the da906{1,2} thermal device tree binding documentation to > json-schema. > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > .../bindings/thermal/da9062-thermal.txt | 36 ------------ > .../bindings/thermal/dlg,da9062-thermal.yaml | 58 +++++++++++++++++++ > 2 files changed, 58 insertions(+), 36 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt > create mode 100644 Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > 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: Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.example.dtb: /example-0/i2c/pmic@58: failed to match any schema with compatible: ['dlg,da9062'] doc reference errors (make refcheckdocs): Warning: MAINTAINERS references a file that doesn't exist: Documentation/devicetree/bindings/thermal/da90??-thermal.txt MAINTAINERS: Documentation/devicetree/bindings/thermal/da90??-thermal.txt See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20231202192536.266885-5-biju.das.jz@bp.renesas.com The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. 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 after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
Hi Rob Herring, Thanks for the feedback, > Subject: Re: [PATCH v2 04/11] dt-bindings: thermal: Convert da906{1,2} > thermal to json-schema > > > On Sat, 02 Dec 2023 19:25:28 +0000, Biju Das wrote: > > Convert the da906{1,2} thermal device tree binding documentation to > > json-schema. > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > .../bindings/thermal/da9062-thermal.txt | 36 ------------ > > .../bindings/thermal/dlg,da9062-thermal.yaml | 58 > > +++++++++++++++++++ > > 2 files changed, 58 insertions(+), 36 deletions(-) delete mode > > 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > create mode 100644 > > Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > > > 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: > Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.example.dtb: > /example-0/i2c/pmic@58: failed to match any schema with compatible: > ['dlg,da9062'] Looks validation is not checking compatible defines in the mfd/da9062.txt file. So will merge this patch with patch #11 to fix this error. > > doc reference errors (make refcheckdocs): > Warning: MAINTAINERS references a file that doesn't exist: > Documentation/devicetree/bindings/thermal/da90??-thermal.txt > MAINTAINERS: Documentation/devicetree/bindings/thermal/da90??-thermal.txt Also, will fix the MAINTAINERS entry. Cheers, Biju
On Sun, Dec 03, 2023 at 10:07:19AM +0000, Biju Das wrote: > Hi Rob Herring, > > Thanks for the feedback, > > > Subject: Re: [PATCH v2 04/11] dt-bindings: thermal: Convert da906{1,2} > > thermal to json-schema > > > > > > On Sat, 02 Dec 2023 19:25:28 +0000, Biju Das wrote: > > > Convert the da906{1,2} thermal device tree binding documentation to > > > json-schema. > > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > > --- > > > .../bindings/thermal/da9062-thermal.txt | 36 ------------ > > > .../bindings/thermal/dlg,da9062-thermal.yaml | 58 > > > +++++++++++++++++++ > > > 2 files changed, 58 insertions(+), 36 deletions(-) delete mode > > > 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > > create mode 100644 > > > Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > > > > > > 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: > > Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.example.dtb: > > /example-0/i2c/pmic@58: failed to match any schema with compatible: > > ['dlg,da9062'] > > Looks validation is not checking compatible defines in the mfd/da9062.txt file. > So will merge this patch with patch #11 to fix this error. No need to merge it. Just note below the '---' the warning is expected. Rob
On Sat, Dec 02, 2023 at 07:25:28PM +0000, Biju Das wrote: > Convert the da906{1,2} thermal device tree binding documentation to > json-schema. > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > .../bindings/thermal/da9062-thermal.txt | 36 ------------ > .../bindings/thermal/dlg,da9062-thermal.yaml | 58 +++++++++++++++++++ > 2 files changed, 58 insertions(+), 36 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt > create mode 100644 Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > diff --git a/Documentation/devicetree/bindings/thermal/da9062-thermal.txt b/Documentation/devicetree/bindings/thermal/da9062-thermal.txt > deleted file mode 100644 > index e241bb5a5584..000000000000 > --- a/Documentation/devicetree/bindings/thermal/da9062-thermal.txt > +++ /dev/null > @@ -1,36 +0,0 @@ > -* Dialog DA9062/61 TJUNC Thermal Module > - > -This module is part of the DA9061/DA9062. For more details about entire > -DA9062 and DA9061 chips see Documentation/devicetree/bindings/mfd/da9062.txt > - > -Junction temperature thermal module uses an interrupt signal to identify > -high THERMAL_TRIP_HOT temperatures for the PMIC device. > - > -Required properties: > - > -- compatible: should be one of the following valid compatible string lines: > - "dlg,da9061-thermal", "dlg,da9062-thermal" > - "dlg,da9062-thermal" > - > -Optional properties: > - > -- polling-delay-passive : Specify the polling period, measured in > - milliseconds, between thermal zone device update checks. > - > -Example: DA9062 > - > - pmic0: da9062@58 { > - thermal { > - compatible = "dlg,da9062-thermal"; > - polling-delay-passive = <3000>; > - }; > - }; > - > -Example: DA9061 using a fall-back compatible for the DA9062 onkey driver > - > - pmic0: da9061@58 { > - thermal { > - compatible = "dlg,da9061-thermal", "dlg,da9062-thermal"; > - polling-delay-passive = <3000>; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml b/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > new file mode 100644 > index 000000000000..0021ebdd83a0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/thermal/dlg,da9062-thermal.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Dialog DA9062/61 TJUNC Thermal Module > + > +description: > + This module is part of the DA9061/DA9062. For more details about entire > + DA9062 and DA9061 chips see Documentation/devicetree/bindings/mfd/da9062.txt > + > + Junction temperature thermal module uses an interrupt signal to identify > + high THERMAL_TRIP_HOT temperatures for the PMIC device. > + > +maintainers: > + - Biju Das <biju.das.jz@bp.renesas.com> > + > +properties: > + compatible: > + oneOf: > + - items: > + - enum: > + - dlg,da9062-thermal > + - items: > + - enum: > + - dlg,da9061-thermal > + - const: dlg,da9062-thermal # da9062-thermal fallback > + > + polling-delay-passive: > + description: > + Specify the polling period, measured in milliseconds, between > + thermal zone device update checks. > + > +required: > + - compatible > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + pmic@58 { > + compatible = "dlg,da9062"; > + reg = <0x58>; > + interrupt-parent = <&gpio6>; > + interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > + interrupt-controller; > + > + thermal { > + compatible = "dlg,da9062-thermal"; > + polling-delay-passive = <3000>; > + }; > + }; > + }; Just drop the example here. No need for 2 examples, just provide 1 complete example in the MFD schema. With that, Reviewed-by: Rob Herring <robh@kernel.org> Rob
Hi Rob Herring, Thanks for the feedback. > -----Original Message----- > From: Rob Herring <robh@kernel.org> > Sent: Monday, December 4, 2023 3:13 PM > Subject: Re: [PATCH v2 04/11] dt-bindings: thermal: Convert da906{1,2} > thermal to json-schema > > On Sat, Dec 02, 2023 at 07:25:28PM +0000, Biju Das wrote: > > Convert the da906{1,2} thermal device tree binding documentation to > > json-schema. > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > .../bindings/thermal/da9062-thermal.txt | 36 ------------ > > .../bindings/thermal/dlg,da9062-thermal.yaml | 58 > > +++++++++++++++++++ > > 2 files changed, 58 insertions(+), 36 deletions(-) delete mode > > 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > create mode 100644 > > Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > > > diff --git > > a/Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > b/Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > deleted file mode 100644 > > index e241bb5a5584..000000000000 > > --- a/Documentation/devicetree/bindings/thermal/da9062-thermal.txt > > +++ /dev/null > > @@ -1,36 +0,0 @@ > > -* Dialog DA9062/61 TJUNC Thermal Module > > - > > -This module is part of the DA9061/DA9062. For more details about > > entire > > -DA9062 and DA9061 chips see > > Documentation/devicetree/bindings/mfd/da9062.txt > > - > > -Junction temperature thermal module uses an interrupt signal to > > identify -high THERMAL_TRIP_HOT temperatures for the PMIC device. > > - > > -Required properties: > > - > > -- compatible: should be one of the following valid compatible string > lines: > > - "dlg,da9061-thermal", "dlg,da9062-thermal" > > - "dlg,da9062-thermal" > > - > > -Optional properties: > > - > > -- polling-delay-passive : Specify the polling period, measured in > > - milliseconds, between thermal zone device update checks. > > - > > -Example: DA9062 > > - > > - pmic0: da9062@58 { > > - thermal { > > - compatible = "dlg,da9062-thermal"; > > - polling-delay-passive = <3000>; > > - }; > > - }; > > - > > -Example: DA9061 using a fall-back compatible for the DA9062 onkey > > driver > > - > > - pmic0: da9061@58 { > > - thermal { > > - compatible = "dlg,da9061-thermal", "dlg,da9062-thermal"; > > - polling-delay-passive = <3000>; > > - }; > > - }; > > diff --git > > a/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > b/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml > > new file mode 100644 > > index 000000000000..0021ebdd83a0 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yam > > +++ l > > @@ -0,0 +1,58 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > +--- > > +$id: > > + > > +title: Dialog DA9062/61 TJUNC Thermal Module > > + > > +description: > > + This module is part of the DA9061/DA9062. For more details about > > +entire > > + DA9062 and DA9061 chips see > > +Documentation/devicetree/bindings/mfd/da9062.txt > > + > > + Junction temperature thermal module uses an interrupt signal to > > + identify high THERMAL_TRIP_HOT temperatures for the PMIC device. > > + > > +maintainers: > > + - Biju Das <biju.das.jz@bp.renesas.com> > > + > > +properties: > > + compatible: > > + oneOf: > > + - items: > > + - enum: > > + - dlg,da9062-thermal > > + - items: > > + - enum: > > + - dlg,da9061-thermal > > + - const: dlg,da9062-thermal # da9062-thermal fallback > > + > > + polling-delay-passive: > > + description: > > + Specify the polling period, measured in milliseconds, between > > + thermal zone device update checks. > > + > > +required: > > + - compatible > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/interrupt-controller/irq.h> > > + i2c { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + pmic@58 { > > + compatible = "dlg,da9062"; > > + reg = <0x58>; > > + interrupt-parent = <&gpio6>; > > + interrupts = <11 IRQ_TYPE_LEVEL_LOW>; > > + interrupt-controller; > > + > > + thermal { > > + compatible = "dlg,da9062-thermal"; > > + polling-delay-passive = <3000>; > > + }; > > + }; > > + }; > > Just drop the example here. No need for 2 examples, just provide 1 > complete example in the MFD schema. > > With that, OK, will drop example in the next version. Cheers, Biju > > Reviewed-by: Rob Herring <robh@kernel.org> > > Rob
Convert the below bindings to json-schema 1) DA906{1,2} mfd bindings 2) DA906{1,2,3} onkey bindings 3) DA906{1,2,3} thermal bindings Also add fallback for DA9061 watchdog device and document DA9063 watchdog device. v1->v2: Ref : https://lore.kernel.org/all/20231201110840.37408-5-biju.das.jz@bp.renesas.com/ * Made as a separate series * DA9062 and DA9061 merged with DA9063 * Sorted the child devices * mfd,onkey and thermal are pointing to child bindings Biju Das (11): dt-bindings: watchdog: dlg,da9062-watchdog: Add fallback for DA9061 watchdog dt-bindings: watchdog: dlg,da9062-watchdog: Document DA9063 watchdog dt-bindings: input: Convert da906{1,2,3} onkey to json-schema dt-bindings: thermal: Convert da906{1,2} thermal to json-schema dt-bindings: mfd: dlg,da9063: Update watchdog property dt-bindings: mfd: dlg,da9063: Update onkey property dt-bindings: mfd: dlg,da9063: Sort child devices dt-bindings: mfd: da9062: Update watchdog description dt-bindings: mfd: da9062: Update onkey description dt-bindings: mfd: da9062: Update thermal description dt-bindings: mfd: dlg,da9063: Convert da9062 to json-schema .../bindings/input/da9062-onkey.txt | 47 ---- .../bindings/input/dlg,da9062-onkey.yaml | 61 +++++ .../devicetree/bindings/mfd/da9062.txt | 124 ---------- .../devicetree/bindings/mfd/dlg,da9063.yaml | 220 +++++++++++++++--- .../bindings/thermal/da9062-thermal.txt | 36 --- .../bindings/thermal/dlg,da9062-thermal.yaml | 58 +++++ .../watchdog/dlg,da9062-watchdog.yaml | 12 +- 7 files changed, 313 insertions(+), 245 deletions(-) delete mode 100644 Documentation/devicetree/bindings/input/da9062-onkey.txt create mode 100644 Documentation/devicetree/bindings/input/dlg,da9062-onkey.yaml delete mode 100644 Documentation/devicetree/bindings/mfd/da9062.txt delete mode 100644 Documentation/devicetree/bindings/thermal/da9062-thermal.txt create mode 100644 Documentation/devicetree/bindings/thermal/dlg,da9062-thermal.yaml