Re: [PATCH v2 6/6] ARM: dts: hix5hd2: add wdg node

On Thu, Aug 28, 2014 at 10:21:43AM +0100, Dinh Nguyen wrote:

On 8/28/14, 12:34 AM, Zhangfei Gao wrote:

quoted

Signed-off-by: Jiancheng Xue <redacted>
Signed-off-by: Zhangfei Gao <redacted>
---
 arch/arm/boot/dts/hisi-x5hd2.dtsi |    9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/hisi-x5hd2.dtsi b/arch/arm/boot/dts/hisi-x5hd2.dtsi
index 152f3ad..7b1cb53 100644
--- a/arch/arm/boot/dts/hisi-x5hd2.dtsi
+++ b/arch/arm/boot/dts/hisi-x5hd2.dtsi

@@ -365,6 +365,15 @@
 				#interrupt-cells = <2>;
 				status = "disabled";
 			};
+
+			wdg0: wdg@a2c000 {

A nit-pick here, but everybody else is using 'wdt' for a watchdog timer
node.

quoted

+				compatible = "arm,sp805", "arm,primecell";

I didn't see an entry for "arm,sp805" in Documentations, should there be
one?

There probably should, yes. It looks like only the sp804 is documented.

Pawel, it looks like you were the first sp805,sp810 dt user. Care to put
together a couple of documents? ;)

Mark.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html