From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Sat, 25 Jul 2009 22:49:14 -0400 Subject: [U-Boot] Add md5sum and sha1 commands... In-Reply-To: <200907251607.49314.rgetz@blackfin.uclinux.org> References: <200907251607.49314.rgetz@blackfin.uclinux.org> Message-ID: <200907252249.15951.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > +#ifdef CONFIG_CMD_MD5SUM > +#include > +#endif > + > +#ifdef CONFIG_CMD_SHA1 > +#include > +#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