[U-Boot] Add md5sum and sha1 commands...

[Mike Frysinger <vapier@gentoo.org>]

On Saturday 25 July 2009 16:07:49 Robin Getz wrote:
> --- a/common/cmd_mem.c
> +++ b/common/cmd_mem.c
> @@ -34,6 +34,14 @@
>  #endif
>  #include <watchdog.h>
>
> +#ifdef CONFIG_CMD_MD5SUM
> +#include <u-boot/md5.h>
> +#endif
> +
> +#ifdef CONFIG_CMD_SHA1
> +#include <sha1.h>
> +#endif

i dont think there would be a problem just including these all the time.  
would make it easier to notice problems down the line if people moved files 
and compile tested with boards that didnt enable these commands for example.

> +	for (i = 0; i < 16 ; i++)

no space before that semicolon

> +	for (i = 0; i < 20 ; i++)

same here

> +#ifdef CONFIG_CMD_MD5SUM
> +U_BOOT_CMD(
> +	md5sum,	3,	1,	do_md5sum,
> +	"compute MD5 message digest",
> +	"address count"
> +);
> +#endif
> +
> +#ifdef CONFIG_CMD_SHA1
> +U_BOOT_CMD(
> +	sha1,	3,	1,	do_sha1,
> +	"compute SHA1 message digest",
> +	"address count"
> +);
> +#endif /* CONFIG_CMD_SHA1 */

there's no need for these to be at the bottom of the file.  move the 
U_BOOT_CMD() into the releated #ifdef block.

also, they should both have a "sum" suffix or neither.  i'd lean towards the 
former ...
-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/20090725/3a69b9b6/attachment.pgp