On 09/02/2023 14:41, Neil Armstrong wrote: > Convert the Amlogic Clock Controller bindings to dt-schema. Subject prefix - clock, not soc. > > Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx> > --- > .../bindings/clock/amlogic,gxbb-clkc.txt | 53 ---------------------- > .../bindings/clock/amlogic,gxbb-clkc.yaml | 51 +++++++++++++++++++++ > 2 files changed, 51 insertions(+), 53 deletions(-) > + clocks: > + maxItems: 1 > + > + clock-names: > + const: xtal > + > + '#clock-cells': > + const: 1 > + > +required: > + - compatible > + - clocks > + - clock-names > + - '#clock-cells' > + > +unevaluatedProperties: false additionalProperties: false > + > +examples: > + - | > + sysctrl: system-controller@0 { > + compatible = "amlogic,meson-gx-hhi-sysctrl", "simple-mfd", "syscon"; > + reg = <0 0x400>; Drop the parent node, not needed here. You could even skip the example entirely as it is part of your syscon. If you keep example, then without parent pieces. > + > + clkc: clock-controller { > + compatible = "amlogic,gxbb-clkc"; > + #clock-cells = <1>; > + clocks = <&xtal>; > + clock-names = "xtal"; > + }; > + }; > Best regards, Krzysztof