On 17/08/2022 14:34, Jiucheng Xu wrote: > Add binding documentation for the Amlogic G12 series DDR > performance monitor unit. > > Signed-off-by: Jiucheng Xu <jiucheng.xu@xxxxxxxxxxx> (...) > + > + interrupts: > + items: > + - description: The IRQ of the inside timer timeout. > + > +required: > + - compatible > + - reg > + - interrupts > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + pmu { > + #address-cells=<2>; > + #size-cells=<2>; > + > + pmu@ff638000 { > + No need for blank line. > + compatible = "amlogic,g12a-ddr-pmu"; > + reg = <0x0 0xff638000 0x0 0x100>, > + <0x0 0xff638c00 0x0 0x100>; > + interrupts = <GIC_SPI 52 IRQ_TYPE_EDGE_RISING>; > + }; > + }; With above fixed: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof