On Thu, Sep 11, 2014 at 10:56:07AM +0200, Boris BREZILLON wrote: > Signed-off-by: Boris BREZILLON <boris.brezillon@xxxxxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/rtc/atmel,at91sam9-rtc.txt | 24 ++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.txt > > diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.txt b/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.txt > new file mode 100644 > index 0000000..258a036 > --- /dev/null > +++ b/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.txt > @@ -0,0 +1,24 @@ > +Atmel AT91SAM9260 Real Time Timer > + > +Required properties: > +- compatible: should be: "atmel,at91sam9260-rtt" > +- reg: should encode the memory region of the RTT controller > +- interrupts: rtc alarm/event interrupt > +- clocks: should contain one clock pointing the the slow clk > +- atmel,time-reg: should encode the GPBR register used to store the RTC > + time. The first cell should point to the GPBR node and > + the second one encode the offset within the GPBR block > + (or in other terms, the GPBR register used to store the > + current time value). > + > + > + > +Example: > + > +rtc@fffffe00 { This should be rtt@ > + compatible = "atmel,at91sam9260-rtt"; > + reg = <0xfffffd20 0x10>; > + interrupts = <1 4 7>; > + clocks = <&clk32k>; > + atmel,time-reg = <&gpbr 0x0>; I think the property name could be more descriptive as it is specific to when the RTT is used as an RTC. Perhaps, atmel,rtt-rtc-time-reg? > +}; Johan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html