diff options
author | Guilherme Maciel Ferreira <guilherme.maciel.ferreira@gmail.com> | 2015-01-15 02:54:43 -0200 |
---|---|---|
committer | Tom Rini <trini@ti.com> | 2015-01-29 13:38:41 -0500 |
commit | 2662179998e4f7c2c67f00527c3af5123816bd49 (patch) | |
tree | 75bd759ba0e312efc687a0d8381a5650fba9c39f /tools/socfpgaimage.c | |
parent | 39931f966adaeadd66dc7a905f7dddb93f66bac3 (diff) | |
download | talos-obmc-uboot-2662179998e4f7c2c67f00527c3af5123816bd49.tar.gz talos-obmc-uboot-2662179998e4f7c2c67f00527c3af5123816bd49.zip |
tools: do not print error messages in verify_header() functions
default_image.c and socfpgaimage.c are the only image modules that print error
messages during header verification. The verify_header() is used to query if a
given image file is processed by the image format. Thus, if the image format
can't handle the file, it must simply return an error. Otherwise we pollute the
screen with errors messages until we find the image format that handle a given
image file.
Signed-off-by: Guilherme Maciel Ferreira <guilherme.maciel.ferreira@gmail.com>
Diffstat (limited to 'tools/socfpgaimage.c')
-rw-r--r-- | tools/socfpgaimage.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/socfpgaimage.c b/tools/socfpgaimage.c index 652ae45f81..8fe91fe80e 100644 --- a/tools/socfpgaimage.c +++ b/tools/socfpgaimage.c @@ -33,6 +33,8 @@ #include "pbl_crc32.h" #include "imagetool.h" +#include "mkimage.h" + #include <image.h> #define HEADER_OFFSET 0x40 @@ -133,12 +135,12 @@ static int verify_buffer(const uint8_t *buf) len = verify_header(buf + HEADER_OFFSET); if (len < 0) { - fprintf(stderr, "Invalid header\n"); + debug("Invalid header\n"); return -1; } if (len < HEADER_OFFSET || len > PADDED_SIZE) { - fprintf(stderr, "Invalid header length (%i)\n", len); + debug("Invalid header length (%i)\n", len); return -1; } |