On 21/07/2023 22:37, Duje Mihanović wrote: > Add dt bindings and documentation for Marvell PXA1908's clock controller > blocks. > > Signed-off-by: Duje Mihanović <duje.mihanovic@xxxxxxxx> > --- > .../bindings/clock/marvell,pxa1908.yaml | 53 +++++++++++++++++++ > 1 file changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/clock/marvell,pxa1908.yaml > > diff --git a/Documentation/devicetree/bindings/clock/marvell,pxa1908.yaml b/Documentation/devicetree/bindings/clock/marvell,pxa1908.yaml > new file mode 100644 > index 000000000000..3fa6f00a26f7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/marvell,pxa1908.yaml > @@ -0,0 +1,53 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/clock/marvell,pxa1908.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Marvell PXA1908 Clock Controllers > + > +maintainers: > + - Duje Mihanović <duje.mihanovic@xxxxxxxx> > + > +description: | > + The PXA1908 clock subsystem generates and supplies clock to various > + controllers within the PXA1908 SoC. The PXA1908 contains numerous clock > + controller blocks, with the ones currently supported being APBC, APBCP, MPMU > + and APMU roughly corresponding to internal buses. > + > + Each clock is assigned an identifier and client nodes use this identifier to > + specify the clock which they consume. Drop this sentence, it's obvious. > + > + All these identifiers could be found in <include/dt-bindings/marvell,pxa1908.h>. "Clock identifiers ..." > + > +properties: > + compatible: > + enum: > + - marvell,pxa1908-apbc > + - marvell,pxa1908-apbcp > + - marvell,pxa1908-mpmu > + - marvell,pxa1908-apmu Blank line > + reg: > + maxItems: 1 > + description: | > + Physical address of the clock controller and length of the memory mapped > + region. Drop description, it is obvious. > + > + '#clock-cells': > + const: 1 Best regards, Krzysztof