On Fri, 15 Mar 2024 23:27:49 +0100, Wadim Mueller wrote: > Add DT binding schema documentation for the NXP S32 dwmac glue driver. This documentation is based on the patchset originally provided by Chester Lin [1]. This commit is a re-send of [2] and [3]. > > [1] https://patchwork.kernel.org/project/netdevbpf/patch/20221031101052.14956-6-clin@xxxxxxxx/#25068228 > [2] https://lore.kernel.org/lkml/20221031101052.14956-1-clin@xxxxxxxx/T/#me96c28bd0536de276dee941469ea084d51b42244 > [3] https://lore.kernel.org/lkml/20221031101052.14956-1-clin@xxxxxxxx/T/#m887a1b34e612f8dc0d5b718e4d6834c083f1e245 > > Signed-off-by: Wadim Mueller <wafgo01@xxxxxxxxx> > --- > .../bindings/net/nxp,s32-dwmac.yaml | 130 ++++++++++++++++++ > .../devicetree/bindings/net/snps,dwmac.yaml | 5 +- > 2 files changed, 133 insertions(+), 2 deletions(-) > create mode 100644 Documentation/devicetree/bindings/net/nxp,s32-dwmac.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/net/nxp,s32-dwmac.yaml:5:10: [error] string value is redundantly quoted with any quotes (quoted-strings) dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/net/nxp,s32-dwmac.yaml: 'maintainers' is a required property hint: Metaschema for devicetree binding documentation from schema $id: http://devicetree.org/meta-schemas/base.yaml# doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240315222754.22366-4-wafgo01@xxxxxxxxx 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.