Hi, On 19/01/2019 15:43:31+0100, Martin Blumenstingl wrote: > The 32-bit Amlogic Meson SoCs (Meson6, Meson8, Meson8b and Meson8m2) > have a built-in RTC block. > It has the following inputs: > - an 32.768kHz crystal oscillator > - an interrupt line > - a reset line > - 0.9V voltage input > > Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> > [resurrected patches from Ben after 2 years] > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> > --- > .../devicetree/bindings/rtc/rtc-meson.txt | 32 +++++++++++++++++++ > 1 file changed, 32 insertions(+) > create mode 100644 Documentation/devicetree/bindings/rtc/rtc-meson.txt > > diff --git a/Documentation/devicetree/bindings/rtc/rtc-meson.txt b/Documentation/devicetree/bindings/rtc/rtc-meson.txt > new file mode 100644 > index 000000000000..23df3cf2dd73 > --- /dev/null > +++ b/Documentation/devicetree/bindings/rtc/rtc-meson.txt > @@ -0,0 +1,32 @@ > +* Amlogic Meson6, Meson8, Meson8b and Meson8m2 RTC > + > +Required properties: > +- compatible: should be one of the following describing the hardware: > + * "amlogic,meson6-rtc" > + * "amlogic,meson8-rtc" > + * "amlogic,meson8b-rtc" > + * "amlogic,meson8m2-rtc" > + > +- reg: physical register space for the controller's memory mapped registers. > +- interrupts: the interrupt line of the RTC block. > +- clocks: reference to the external 32.768kHz crystal oscillator. > +- vdd-supply: reference to the power supply of the RTC block. > +- resets: reset controller reference to allow reset of the controller > + > +Optional properties: > +- #address-cells: should be 1 to address the internal non-volatile memory > +- #address-cells: should be 1 to reference the internal non-volatile memory Shouldn't that one be #size-cells ? -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com