[U-Boot-Users] [PATCH] bugfix: the watchdog post for lwmon5

[Sascha Laue]

You are right! I've checked it again and find the mistake by myself. 

Subject: [PATCH] bugfix: lwmon5 watchdog post
If the hardware watchdog detects a voltage error, the watchdog set GPIO62 to low.
The watchdog POST have to detect this lowpegel.

Signed-off-by: Sascha Laue <leglas0 at legpc180.leg.liebherr.i>
---
 post/board/lwmon5/watchdog.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/post/board/lwmon5/watchdog.c b/post/board/lwmon5/watchdog.c
index 48ff687..899fbfb 100644
--- a/post/board/lwmon5/watchdog.c
+++ b/post/board/lwmon5/watchdog.c
@@ -55,7 +55,7 @@ static void watchdog_magic_write(uint value)
 
 int sysmon1_post_test(int flags)
 {
-	if (gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS)) {
+	if (gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS) == 0) {
 		/*
 		 * 3.1. GPIO62 is low
 		 * Assuming system voltage failure.
-- 
1.5.2.4

-------- Original-Nachricht --------
> Datum: Tue, 01 Apr 2008 10:33:20 +0200
> Von: Wolfgang Denk <wd at denx.de>
> An: "Sascha Laue" <Sascha.Laue at gmx.biz>
> CC: u-boot-users at lists.sourceforge.net
> Betreff: Re: [U-Boot-Users] [PATCH] bugfix: the watchdog post for lwmon5

> In message <20080401075941.145820 at gmx.net> you wrote:
> > bugfix: the watchdog post for lwmon5 doesn't work. This Patch fix it.
> > To trigger and activate the watchdog correct it is necessary to create 2
> pulses.
> > Lowpegel on GPIO62 point to a voltagefailure.
> 
> Please fix English ("correct", "Lowpegel", "voltagefailure").
> 
> Is it really 2 pulses - or just one, consisting of 
> 
> > Signed-off-by: Sascha Laue <sascha.laue at liebherr.com>
> ...
> > -	if (gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS)) {
> > +	if (0 == gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS)) {
> 
> Please write that as:
> 
> 	if (gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS) == 0) {
> or
> 	if (!gpio_read_in_bit(CFG_GPIO_SYSMON_STATUS)) {
> 
> >  		 * 3.1. GPIO62 is low
> >  		 * Assuming system voltage failure.
> > @@ -99,6 +99,9 @@ int lwmon5_watchdog_post_test(int flags)
> >  		ints = disable_interrupts ();
> >  		/* 3.2.2. strobe watchdog once */
> >  		WATCHDOG_RESET();
> > +		base = post_time_ms (0);
> > +		while (post_time_ms (base) < 2)
> > +		WATCHDOG_RESET();
> 
> Are you absolutely sure that is needed?
> 
> Best regards,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> Tactical? TACTICAL!?!? Hey, buddy, we went from kilotons to  megatons
> several  minutes  ago.  We don't need no stinkin' tactical nukes. (By
> the way, do you have change for 10 million people?)           - lwall