Hi Simon, On Tue, May 5, 2015 at 4:37 PM, Simon Glass <s...@chromium.org> wrote: > Hi Joe, > > On 5 May 2015 at 11:23, Joe Hershberger <joe.hershber...@ni.com> wrote: >> >> Introduced in change d20a40de9db07de1f1f06a79a4da1cdda5379b75 >> "Roll crc32 into hash infrastructure" >> >> Use a consistent define to enable the verify feature in crc32 command. >> >> Signed-off-by: Joe Hershberger <joe.hershber...@ni.com> >> --- >> >> common/cmd_mem.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/common/cmd_mem.c b/common/cmd_mem.c >> index 5d8c9e6..b91adb2 100644 >> --- a/common/cmd_mem.c >> +++ b/common/cmd_mem.c >> @@ -1303,7 +1303,7 @@ U_BOOT_CMD( >> >> #ifdef CONFIG_CMD_CRC32 >> >> -#ifndef CONFIG_CRC32_VERIFY >> +#ifndef CONFIG_HASH_VERIFY > > Do you need to change the users of CONFIG_CRC32_VERIFY?
Shouldn't need to just like you didn't on the other similar change to this file because in include/hash.h: #if defined(CONFIG_SHA1SUM_VERIFY) || defined(CONFIG_CRC32_VERIFY) #define CONFIG_HASH_VERIFY #endif >> >> U_BOOT_CMD( >> crc32, 4, 1, do_mem_crc, >> @@ -1311,7 +1311,7 @@ U_BOOT_CMD( >> "address count [addr]\n - compute CRC32 checksum [save at addr]" >> ); >> >> -#else /* CONFIG_CRC32_VERIFY */ >> +#else /* CONFIG_HASH_VERIFY */ >> >> U_BOOT_CMD( >> crc32, 5, 1, do_mem_crc, >> @@ -1320,7 +1320,7 @@ U_BOOT_CMD( >> "-v address count crc\n - verify crc of memory area" >> ); >> >> -#endif /* CONFIG_CRC32_VERIFY */ >> +#endif /* CONFIG_HASH_VERIFY */ >> >> #endif >> >> -- >> 1.7.11.5 Cheers, -Joe _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
