[U-Boot] [PATCH v2 1/2]: common: Add a watchdog CLI command

Mike Frysinger vapier at gentoo.org
Fri Nov 6 12:18:27 CET 2009


On Friday 06 November 2009 04:28:23 Simon Kagstrom wrote:
> (Ping!)
> 
> On Thu, 29 Oct 2009 09:09:23 +0100
> 
> Simon Kagstrom <simon.kagstrom at netinsight.net> wrote:
> > A watchdog command to enable the watchdog with a timeout from the CLI
> > can sometimes be useful. Add that. This also adds a common API for
> > enabling watchdogs. The API is simple:
> >
> >         int watchdog_enable(unsigned int timeout);
> >
> > the timeout range vary depending on hardware, and the driver should
> > return a negative value if the call failed.
> 
> Wolfgang: Do you have any additional comments on this patch?

here's one: dont quote the entire e-mail when you dont need to.  you force 
people to scan the entire thing looking for possible inline comments.
-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/20091106/85b90f82/attachment.pgp 


More information about the U-Boot mailing list