Hi Suravee,

On 11 February 2016 at 04:56, Suravee Suthikulpanit
[off-list ref] wrote:

Hi Fu Wei,

On 2/10/16 00:00, fu.wei at linaro.org wrote:

quoted

From: Fu Wei <redacted>

This can be a example of adding SBSA Generic Watchdog device node
into some dts files for the Soc which contains SBSA Generic Watchdog.

Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com>
Signed-off-by: Fu Wei <redacted>
---
  arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi | 9 +++++++++
  1 file changed, 9 insertions(+)

diff --git a/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi

b/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
index 2874d92..67eb636 100644

--- a/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
+++ b/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi

@@ -84,6 +84,15 @@
                        clock-names = "uartclk", "apb_pclk";
                };

+               watchdog0: watchdog at e0bb0000 {
+                       compatible = "arm,sbsa-gwdt";
+                       reg = <0x0 0xe0bc0000 0 0x1000>,
+                               <0x0 0xe0bb0000 0 0x1000>;
+                       interrupts = <0 337 4>;
+                       timeout-sec = <15>;
+                       status = "disabled";

Could you please remove this status line? I do not think it is necessary for
this one here anymore.

OK, will do
:-)

Thanks,
Suravee

quoted

+               };
+
                spi0: ssp at e1020000 {
                        status = "disabled";
                        compatible = "arm,pl022", "arm,primecell";

-- 
Best regards,

Fu Wei
Software Engineer
Red Hat Software (Beijing) Co.,Ltd.Shanghai Branch
Ph: +86 21 61221326(direct)
Ph: +86 186 2020 4684 (mobile)
Room 1512, Regus One Corporate Avenue,Level 15,
One Corporate Avenue,222 Hubin Road,Huangpu District,
Shanghai,China 200021