diff mbox

[v3,9/9] ARM: dts: add the watchdog nodes for at91sam9g25ek boards

Message ID 1357353529-20901-10-git-send-email-wenyou.yang@atmel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Wenyou Yang Jan. 5, 2013, 2:38 a.m. UTC
Tested on the at91sam9g25ek boards

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Cc: linux@arm.linux.org.uk
Cc: linux-kernel@vger.kernel.org
---
 arch/arm/boot/dts/at91sam9x5ek.dtsi |    4 ++++
 1 file changed, 4 insertions(+)

Comments

Jean-Christophe PLAGNIOL-VILLARD Jan. 8, 2013, 8:01 p.m. UTC | #1
On 10:38 Sat 05 Jan     , Wenyou Yang wrote:
> Tested on the at91sam9g25ek boards
> 
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> Cc: linux@arm.linux.org.uk
> Cc: linux-kernel@vger.kernel.org
> ---
>  arch/arm/boot/dts/at91sam9x5ek.dtsi |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/at91sam9x5ek.dtsi b/arch/arm/boot/dts/at91sam9x5ek.dtsi
> index 8a7cf1d..afddf75 100644
> --- a/arch/arm/boot/dts/at91sam9x5ek.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5ek.dtsi
> @@ -69,6 +69,10 @@
>  				status = "okay";
>  			};
>  
> +			watchdog@fffffe40 {
> +				status = "okay";
> +			};
> +

use less today as the watchdog is shutdown at bootstrap level

and on at91 once shutdown we can not enable it again

Best Regards,
J.
>  			pinctrl@fffff400 {
>  				mmc0 {
>  					pinctrl_board_mmc0: mmc0-board {
> -- 
> 1.7.9.5
>
Wenyou Yang Jan. 9, 2013, 1:04 a.m. UTC | #2
> -----Original Message-----

> From: Jean-Christophe PLAGNIOL-VILLARD [mailto:plagnioj@jcrosoft.com]

> Sent: 2013?1?9? 4:02

> To: Yang, Wenyou

> Cc: linux-arm-kernel@lists.infradead.org; Ferre, Nicolas; Lin, JM;

> linux@arm.linux.org.uk; linux-kernel@vger.kernel.org

> Subject: Re: [v3 PATCH 9/9] ARM: dts: add the watchdog nodes for

> at91sam9g25ek boards

> 

> On 10:38 Sat 05 Jan     , Wenyou Yang wrote:

> > Tested on the at91sam9g25ek boards

> >

> > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>

> > Cc: linux@arm.linux.org.uk

> > Cc: linux-kernel@vger.kernel.org

> > ---

> >  arch/arm/boot/dts/at91sam9x5ek.dtsi |    4 ++++

> >  1 file changed, 4 insertions(+)

> >

> > diff --git a/arch/arm/boot/dts/at91sam9x5ek.dtsi

> b/arch/arm/boot/dts/at91sam9x5ek.dtsi

> > index 8a7cf1d..afddf75 100644

> > --- a/arch/arm/boot/dts/at91sam9x5ek.dtsi

> > +++ b/arch/arm/boot/dts/at91sam9x5ek.dtsi

> > @@ -69,6 +69,10 @@

> >  				status = "okay";

> >  			};

> >

> > +			watchdog@fffffe40 {

> > +				status = "okay";

> > +			};

> > +

> 

> use less today as the watchdog is shutdown at bootstrap level

> 

> and on at91 once shutdown we can not enable it again


Yes, you are right.
But, If you want to use the watchdog, you should not disable the watchdog.
Also in the new version at91bootstrap, it provide this config option.

If the watchdog is disabled in the bootstrap, "the watchdog has been disabled" will display in the dmesg.

> 

> Best Regards,

> J.

> >  			pinctrl@fffff400 {

> >  				mmc0 {

> >  					pinctrl_board_mmc0: mmc0-board {

> > --

> > 1.7.9.5

> >


Best Regards,
Wenyou Yang
diff mbox

Patch

diff --git a/arch/arm/boot/dts/at91sam9x5ek.dtsi b/arch/arm/boot/dts/at91sam9x5ek.dtsi
index 8a7cf1d..afddf75 100644
--- a/arch/arm/boot/dts/at91sam9x5ek.dtsi
+++ b/arch/arm/boot/dts/at91sam9x5ek.dtsi
@@ -69,6 +69,10 @@ 
 				status = "okay";
 			};
 
+			watchdog@fffffe40 {
+				status = "okay";
+			};
+
 			pinctrl@fffff400 {
 				mmc0 {
 					pinctrl_board_mmc0: mmc0-board {