On Aug 8, 2013, at 4:41 AM, John Crispin wrote: > Describe ralink-wdt binding. > > Signed-off-by: John Crispin <blogic@xxxxxxxxxxx> > Cc: linux-watchdog@xxxxxxxxxxxxxxx > Cc: linux-mips@xxxxxxxxxxxxxx > Cc: devicetree@xxxxxxxxxxxxxxx > --- > V1 used the old devicetree list as Cc. > > .../devicetree/bindings/watchdog/ralink-wdt.txt | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/ralink-wdt.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/ralink-wdt.txt b/Documentation/devicetree/bindings/watchdog/ralink-wdt.txt > new file mode 100644 > index 0000000..a70f0e8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/ralink-wdt.txt Seems like this should be ralink-rt2880-wdt.txt (I'm guessing there are other ralink watchdog timers). > @@ -0,0 +1,19 @@ > +Ralink Watchdog Timers Should probably be Ralink RT2880 Watchdog Timer > + > +Required properties : > +- compatible: must be "ralink,rt2880-wdt" > +- reg: physical base address of the controller and length of the register range > + > +Optional properties : > +- interrupt-parent: phandle to the INTC device node > +- interrupts : Specify the INTC interrupt number Since you'll be updating this to drop 'ralink,mt7620a-wdt' how about cleaning up the whitespace around ' :' > + > +Example: > + > + watchdog@120 { > + compatible = "ralink,mt7620a-wdt", "ralink,rt2880-wdt"; > + reg = <0x120 0x10>; > + > + interrupt-parent = <&intc>; > + interrupts = <1>; > + }; > -- > 1.7.10.4 > > -- > 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 - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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