H Rob Yes, I see the same issue. As you wrote probably dt-schema was not up to data. I have fresh system, and I had problem with running 'make dt_binding_check' The script complain for: dtc needs libyaml for DT schema validation support. Install the necessary libyaml development package but in reality it was lacking pkg-config. It could be good to add some information about this package in documentation or some checking in script. I leave it to your decision. > > >On Fri, 26 Jun 2020 06:54:46 +0200, Pawel Laszczak wrote: >> This patch aim at documenting USB related dt-bindings for the >> Cadence CDNSP-DRD controller. >> >> Signed-off-by: Pawel Laszczak <pawell@xxxxxxxxxxx> >> --- >> .../devicetree/bindings/usb/cdns-cdnsp.yaml | 104 ++++++++++++++++++ >> 1 file changed, 104 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/usb/cdns-cdnsp.yaml >> > > >My bot found errors running 'make dt_binding_check' on your patch: > >/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/usb/cdns-cdnsp.example.dt.yaml: example-0: >usb@f3000000:reg:0: [0, 29437952, 0, 1024] is too long >/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/usb/cdns-cdnsp.example.dt.yaml: example-0: >usb@f3000000:reg:1: [0, 4076929024, 0, 65536] is too long >/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/usb/cdns-cdnsp.example.dt.yaml: example-0: >usb@f3000000:reg:2: [0, 4076994560, 0, 65536] is too long > > >See https://urldefense.com/v3/__https://patchwork.ozlabs.org/patch/1317380__;!!EHscmS1ygiU1lA!THRFzEi4Bx7XfHahLfM_Oo53V- >hW49AW-xHAcJ7J53DIaLVRbpAOTe92Jrsu6A$ > >If you already ran 'make dt_binding_check' and didn't see the above >error(s), then make sure dt-schema is up to date: > >pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade > >Please check and re-submit. thanks, pawell
