In message <[EMAIL PROTECTED]> I wrote:
>
> >             /* 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?

Maybe I should have been more explicit - note that above code with
correct indentation looks like this:

                /* 3.2.2. strobe watchdog once */
                WATCHDOG_RESET();
                base = post_time_ms (0);
                while (post_time_ms (base) < 2)
                        WATCHDOG_RESET();

I. e. the comment "once" is wrong - you trigger the watchdog many more
times that you probably think.

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: [EMAIL PROTECTED]
Witch!  Witch!  They'll burn ya!
        -- Hag, "Tomorrow is Yesterday", stardate unknown

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to