On Wed, Nov 12, 2014 at 7:18 AM, <Naidu.Tellapati@xxxxxxxxxx> wrote: > From: Naidu Tellapati <Naidu.Tellapati@xxxxxxxxxx> > > Add the devicetree binding document for ImgTec PDC Watchdog Timer. > > Signed-off-by: Jude Abraham <Jude.Abraham@xxxxxxxxxx> > Signed-off-by: Naidu Tellapati <Naidu.Tellapati@xxxxxxxxxx> > diff --git a/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt > new file mode 100644 > index 0000000..2f13896 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/imgpdc-wdt.txt > @@ -0,0 +1,18 @@ > +*ImgTec PowerDown Controller (PDC) Watchdog Timer (WDT) > + > +Required properties: > +- compatible : Should be "img,pistachio-pdc-wdt" I don't see anything Pistachio-specific about this driver or binding and it looks like IP revision is probable via the WD_CORE_REV register, so I think you can drop the "pistachio". > +- reg : Should contain WDT registers location and length > +- clock-names: Should contain "wdt" I believe there are two clocks: the 32kHz watchdog timer operating clock and the system interface gate clock. Perhaps call them "wdt" and "sys"? > +- clocks: phandles to input clocks > +- interrupts : Should contain WDT interrupt Not sure the interrupt will really be necessary. > +Examples: > + > +wdt@18102100 { > + compatible = "img,pistachio-pdc-wdt"; > + reg = <0x18102100 0x20>; The TRM I have shows the watchdog registers occupying a full 256 bytes. > + clocks = <&pdc_wdt_clk>; > + clock-names = "wdt"; > + interrupts = <0 52 IRQ_TYPE_LEVEL_HIGH>; > +}; > -- > 1.7.0.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html