On Thu, Feb 06, 2025 at 09:23:44PM -0600, Rob Herring (Arm) wrote: > [EXTERNAL MAIL] > > On Fri, 07 Feb 2025 10:19:22 +0800, Ben Zong-You Xie wrote: > > Document devicetree bindings for Andes I2C controller. > > > > Signed-off-by: Ben Zong-You Xie <ben717@xxxxxxxxxxxxx> > > --- > > .../bindings/i2c/andestech,i2c-atciic100.yaml | 40 +++++++++++++++++++ > > MAINTAINERS | 5 +++ > > 2 files changed, 45 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/i2c/andestech,i2c-atciic100.yaml > > > > My bot found errors running 'make dt_binding_check' on your patch: > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/i2c/andestech,i2c-atciic100.yaml: $id: Cannot determine base path from $id, relative path/filename doesn't match actual path or filename > $id: http://devicetree.org/schemas/pwm/andestech,atciic100.yaml > file: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/i2c/andestech,i2c-atciic100.yaml > > doc reference errors (make refcheckdocs): > Warning: MAINTAINERS references a file that doesn't exist: Documentation/devicetree/bindings/i2c/andestech,atciic100.yaml > MAINTAINERS: Documentation/devicetree/bindings/i2c/andestech,atciic100.yaml > > See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250207021923.2912373-2-ben717@xxxxxxxxxxxxx > > 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, Thanks for your reminder. I found there are some errors in the YAML file and MAINTAINERS file for my patch. I will fix them and check with 'make dt_binding_check' before sending new patches. Best regards, Ben