On 16/03/2023 14:53, Rob Herring wrote: > > On Thu, 16 Mar 2023 04:40:55 +0530, Vijaya Anand wrote: >> Convert the binding document for adi,adau17x1 from txt to yaml >> so one could validate dt-entries correctly and any future additions >> can go into yaml format. Add address and size cells to example to >> prevent errors regarding reg format. >> >> Signed-off-by: Vijaya Anand <sunrockers8@xxxxxxxxx> >> Cc: Daniel Baluta <daniel.baluta@xxxxxxx> >> --- >> Changes since v1: >> - added 'Codec' in title to clarify type of devices >> - put compatible devices in lexographic order >> - changed description of clocks >> - changed 'unevaluatedProperties' to 'additionalProperties' >> - changed node names to be generic >> >> --- >> .../bindings/sound/adi,adau17x1.txt | 32 ------------ >> .../bindings/sound/adi,adau17x1.yaml | 52 +++++++++++++++++++ >> 2 files changed, 52 insertions(+), 32 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/sound/adi,adau17x1.txt >> create mode 100644 Documentation/devicetree/bindings/sound/adi,adau17x1.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: > ./Documentation/devicetree/bindings/sound/adi,adau17x1.yaml:8:2: [warning] wrong indentation: expected 2 but found 1 (indentation) > > dtschema/dtc warnings/errors: > Error: Documentation/devicetree/bindings/sound/adi,adau17x1.example.dts:29.3-30.1 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [scripts/Makefile.lib:419: Documentation/devicetree/bindings/sound/adi,adau17x1.example.dtb] Error 1 Test your patches before sending... this never worked. Best regards, Krzysztof