On Thursday 21 May 2015 16:32:32 fu.wei@xxxxxxxxxx wrote: > + watchdog0: watchdog@2a450000 { > + compatible = "arm,sbsa-gwdt"; > + reg = <0x0 0x2a450000 0 0x10000>, > + <0x0 0x2a440000 0 0x10000>; > + reg-names = "refresh", > + "control"; > + interrupts = <0 27 4>; > + interrupt-names = "ws0"; > + timeout-sec = <10 5>; > + }; > }; > just one tiny comment: you can drop the 'watchdog0:' label here, there is normally no reason to assign a label to a watchdog device, especially if it's in a .dts file rather than a .dtsi. The same is true for the example in the binding documentation. Arnd -- 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