On 07/14/2014 02:16 PM, Harini Katakam wrote: > Add cadence-wdt bindings documentation. > > Signed-off-by: Harini Katakam <harinik@xxxxxxxxxx> > --- > > v3 changes: > - Change reset property type and improve description. > - Improve description of clocks and interrupts. > - Use watchdog@ in the example. > - Use only cdns compatible string for now. > > v2: > No changes > > --- > .../devicetree/bindings/watchdog/cadence-wdt.txt | 27 ++++++++++++++++++++ > 1 file changed, 27 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/cadence-wdt.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt b/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt > new file mode 100644 > index 0000000..ab23e38 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/cadence-wdt.txt > @@ -0,0 +1,27 @@ > +Zynq Watchdog Device Tree Bindings > +------------------------------------------- > + > +Required properties: > +- compatible : Should be "cdns,wdt-r1p2". > +- clocks : Input clock specifier. This should be the ref clk. > +- reg : Physical base address and size of WDT registers map. > +- interrupts : Property with a value describing the interrupt > + number. This interrupt is used for indication > + when the watchdog times out. > +- interrupt-parent : Must be core interrupt controller. > + > +Optional properties > +- reset : If this property exists, then a reset is done > + when watchdog times out. > +- timeout-sec : Watchdog timeout value (in seconds). > + > +Example: > + watchdog@f8005000 { > + compatible = "xlnx,zynq-wdt-r1p2"; According to description above - you should use cdns compatible string here too. Thanks, Michal -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html