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

Mike Frysinger vapier at gentoo.org
Sun Jul 26 04:49:14 CEST 2009


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 


More information about the U-Boot mailing list