Re: CHKDT error by f95cad74acdb ("dt-bindings: clocks: Convert Allwinner legacy clocks to schemas")

From: Rob Herring
Date: Sat Mar 28 2020 - 17:49:02 EST


On Sat, Mar 28, 2020 at 12:21 PM H. Nikolaus Schaller <hns@xxxxxxxxxxxxx> wrote:
>
>
> > Am 28.03.2020 um 18:20 schrieb Andreas Kemnade <andreas@xxxxxxxxxxxx>:
> >
> > Hi Nikolaus,
> >
> > On Sat, 28 Mar 2020 13:26:24 +0100
> > "H. Nikolaus Schaller" <hns@xxxxxxxxxxxxx> wrote:
> >
> >> Hi Rob,
> >> I am trying to check my new bindings with v5.6-rc7 but get this before
> >> the process tries mine:
> >>
> >> make dt_binding_check dtbs_check
> >> ...
> >>
> >> CHKDT Documentation/devicetree/bindings/bus/renesas,bsc.yaml - due to target missing
> >> CHKDT Documentation/devicetree/bindings/bus/simple-pm-bus.yaml - due to target missing
> >> CHKDT Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ahb-clk.yaml - due to target missing
> >> /Volumes/CaseSensitive/master/Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ahb-clk.yaml: Additional properties are not allowed ('deprecated' was unexpected)
> >> make[2]: *** [Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ahb-clk.example.dts] Error 1
> >> make[1]: *** [dt_binding_check] Error 2
> >> make: *** [sub-make] Error 2
> >>
> >> What am I doing wrong?
> >> Is there an option to skip such errors and continue?
> >> Is there an option to just test my bindings and yaml file?
> >>
> > maybe your tools are outdated, so rerun
> >
> > pip3 install git+https://github.com/devicetree-org/dt-schema.git@master
>
> Yes, that one solves the problem!
>
> A better error message of CHKDT would have been helpful. Or an auto-update.

If you know of a "python way" to do auto/self updating I'd like to
know. I looked a while back and didn't find anything. I'm working on a
version check so I can enforce/warn on a minimum version which should
help some.

> Now I just get a lot of messages like
>
> CHKDT Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml - due to: Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml
> Documentation/devicetree/bindings/arm/sunxi/allwinner,sun4i-a10-mbus.yaml:: $id: relative path/filename doesn't match actual path or filename
> expected: http://devicetree.org/schemas/arm/sunxi/allwinner,sun4i-a10-mbus.yaml:#
> CHKDT Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ahb-clk.yaml - due to target missing
> Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ahb-clk.yaml:: $id: relative path/filename doesn't match actual path or filename
> expected: http://devicetree.org/schemas/clock/allwinner,sun4i-a10-ahb-clk.yaml:#

Should be fixed if you are on a current rc.

Rob