On 29.08.2020 19:25, Krzysztof Kozlowski wrote: > The fixed clocks are kept under dedicated node fixed-rate-clocks, thus a > fake "reg" was added. This is not correct with dtschema as fixed-clock > binding does not have a "reg" property: > > arch/arm/boot/dts/exynos3250-artik5-eval.dt.yaml: clock@0: 'reg' does not match any of the regexes: 'pinctrl-[0-9]+' > > Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > --- > arch/arm/boot/dts/exynos3250.dtsi | 42 +++++++++++++------------------ > 1 file changed, 17 insertions(+), 25 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos3250.dtsi b/arch/arm/boot/dts/exynos3250.dtsi > index d3fb45a56527..c67c70e46794 100644 > --- a/arch/arm/boot/dts/exynos3250.dtsi > +++ b/arch/arm/boot/dts/exynos3250.dtsi > @@ -97,33 +97,25 @@ > }; > }; > > - fixed-rate-clocks { > - #address-cells = <1>; > - #size-cells = <0>; > - > - xusbxti: clock@0 { > - compatible = "fixed-clock"; > - reg = <0>; > - clock-frequency = <0>; > - #clock-cells = <0>; > - clock-output-names = "xusbxti"; > - }; > + xusbxti: clock-0 { > + compatible = "fixed-clock"; > + clock-frequency = <0>; > + #clock-cells = <0>; > + clock-output-names = "xusbxti"; > + }; > > - xxti: clock@1 { > - compatible = "fixed-clock"; > - reg = <1>; > - clock-frequency = <0>; > - #clock-cells = <0>; > - clock-output-names = "xxti"; > - }; > + xxti: clock-1 { > + compatible = "fixed-clock"; > + clock-frequency = <0>; > + #clock-cells = <0>; > + clock-output-names = "xxti"; > + }; > > - xtcxo: clock@2 { > - compatible = "fixed-clock"; > - reg = <2>; > - clock-frequency = <0>; > - #clock-cells = <0>; > - clock-output-names = "xtcxo"; > - }; > + xtcxo: clock-2 { > + compatible = "fixed-clock"; > + clock-frequency = <0>; > + #clock-cells = <0>; > + clock-output-names = "xtcxo"; > }; > > pmu { Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland