On Wed, Nov 29, 2023 at 6:29 AM Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > > On 29/11/2023 13:27, Rob Herring wrote: > > > > On Wed, 29 Nov 2023 12:10:41 +0100, Krzysztof Kozlowski wrote: > >> The dtschema treats now properties with '-pascal' suffix as standard one > >> and already defines $ref for them, thus the $ref should be dropped from > >> the bindings. > >> > >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > >> > >> --- > >> > >> dtschema change was merged: > >> https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 > >> but not yet released as new dtschema version. > >> > >> This change should be applied once new dtschema version is released or > >> Rob says otherwise. > >> --- > >> .../devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml | 2 -- > >> 1 file changed, 2 deletions(-) > >> > > > > 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/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmin-pascal: missing type definition > > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmax-pascal: missing type definition > > > > That's expected, depends on the dtschema change. Well, it wasn't because dtschema version used is git tree. However, the CI job was using master rather than main branch. master falls behind when I use the web interface... Now fixed the CI job to use main. Rob
