On 07/03/2022 04:24, Rob Herring wrote: > On Sun, 06 Mar 2022 12:11:16 +0100, Krzysztof Kozlowski wrote: >> Convert the Cadence Universal Flash Storage (UFS) Controlle to DT schema >> format. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx> >> Reviewed-by: Rob Herring <robh@xxxxxxxxxx> >> --- >> .../devicetree/bindings/ufs/cdns,ufshc.txt | 32 --------- >> .../devicetree/bindings/ufs/cdns,ufshc.yaml | 68 +++++++++++++++++++ >> .../devicetree/bindings/ufs/ti,j721e-ufs.yaml | 7 +- >> 3 files changed, 71 insertions(+), 36 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.txt >> create mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.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: > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/ufs/cdns,ufshc.example.dt.yaml: ufs@fd030000: freq-table-hz: 'anyOf' conditional failed, one must be fixed: > [[0, 0], [0, 0]] is too long > [0, 0] is too long > From schema: /usr/local/lib/python3.8/dist-packages/dtschema/schemas/property-units.yaml > This will be fixed with my dtschema patch: https://github.com/devicetree-org/dt-schema/pull/69 Best regards, Krzysztof
