[U-Boot] [PATCH] CFI Driver: Reset watchdog timer after each flash operation
Mike Frysinger
vapier at gentoo.org
Fri Oct 2 20:31:22 CEST 2009
On Friday 02 October 2009 08:30:51 Wolfgang Denk wrote:
> Ingo van Lil wrote:
> > The CFI driver does not reset the device's watchdog, so long-running
> > flash operations will cause the watchdog timer to expire. A comment in
> > flash_status_check() suggests that udelay() is expected to reset the
> > watchdog, but I can't find any architecture where it does.
>
> Please have a closer look, then. On PowerPC, udelay()
> ["lib_ppc/time.c"] calls wait_ticks(), which in turn
> ["lib_ppc/ticks.S"] calls WATCHDOG_RESET
>
> If this is missing in other architectures, it should be fixed at the
> root cause, i. e. in udelay() or in the respective support routines.
Blackfin is missing it as well as i really had no idea it was supposed to be
there. certainly no doc states this requirement. perhaps it'd make sense to
break apart the common stuff to a common udelay() that does things like call
the watchdog and then call the implementation __udelay(). there should be at
least a doc/README.arch that includes these kind of details ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091002/35cc04ba/attachment.pgp
More information about the U-Boot
mailing list