Quoting Damien Le Moal (2020-12-01 19:24:46) > diff --git a/Documentation/devicetree/bindings/clock/canaan,k210-clk.yaml b/Documentation/devicetree/bindings/clock/canaan,k210-clk.yaml > new file mode 100644 > index 000000000000..3547916a2421 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/canaan,k210-clk.yaml > @@ -0,0 +1,55 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/clock/canaan,k210-clk.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Canaan Kendryte K210 Clock Device Tree Bindings > + > +maintainers: > + - Damien Le Moal <damien.lemoal@xxxxxxx> > + > +description: | > + Canaan Kendryte K210 SoC clocks driver bindings. The clock > + controller node must be defined as a child node of the K210 > + system controller node. > + > + See also: > + - dt-bindings/clock/k210-clk.h > + > +properties: > + compatible: > + const: canaan,k210-clk > + > + clocks: > + maxItems: 1 > + description: > + System fixed rate oscillator clock. Is it optional? I suspect not and it can always be specified, so drop maxItems. > + > + '#clock-cells': > + const: 1 > + > +required: > + - compatible > + - '#clock-cells' > + - clocks Maybe also add clock-names as an optional property, but doesn't really matter if there's only one clk.
