Dear Prafulla,
In message 20090818122223.73095833DBD2@gemini.denx.de I wrote:
In message 1250380700-15636-1-git-send-email-prafulla@marvell.com you wrote:
uninitialized retval variable warning fixed crc32 APIs moved to crc.h (newly added) and build warnings fixed some indentation tabs fixed
Signed-off-by: Prafulla Wadaskar prafulla@marvell.com
Change log: v2: updated as per review feedback for v1
include/common.h | 4 +--- include/u-boot/crc.h | 33 +++++++++++++++++++++++++++++++++ tools/mkimage.c | 20 +++++++++++--------- 3 files changed, 45 insertions(+), 12 deletions(-) create mode 100644 include/u-boot/crc.h
Applied to "mkimage" branch.
I decided to omit the tab cleanup, and used "uchar" instead of "unsigned char" so no reformatting due to growing line length was needed. This way the changes are (IMHO) better visible.
...what a stupid idea. I didn't consider that "uchar" might not be defined in standard compile environments. Undone.
I applied your original patch now, just with minimal indentation changes.
Sorry for the confusion.
Best regards,
Wolfgang Denk