Hi Suravee, On 11 February 2016 at 04:56, Suravee Suthikulpanit <Suravee.Suthikulpanit@xxxxxxx> wrote: > Hi Fu Wei, > > On 2/10/16 00:00, fu.wei@xxxxxxxxxx wrote: >> >> From: Fu Wei <fu.wei@xxxxxxxxxx> >> >> 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@xxxxxxxx> >> Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@xxxxxxx> >> Signed-off-by: Fu Wei <fu.wei@xxxxxxxxxx> >> --- >> 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@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 > > >> + }; >> + >> spi0: ssp@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 -- 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