From: Dirk Behme <dirk.be...@googlemail.com>

Using mkimage with e.g.

tools/mkimage -A arm -T firmware -O u-boot -d u-boot.bin foo.img

gives a warning

"Unknown OMAP image type - 5"

while it seems that the image itself is created successfully.

This does come from the patch "mkimage: Add OMAP boot image support".

The method check_image_type in image_type_params is supposed to just
return success or failure.  However, for omap it also calls fprintf:

static int omapimage_check_image_types(uint8_t type)
{
        if (type == IH_TYPE_OMAPIMAGE)
                return EXIT_SUCCESS;
        else {
                fprintf(stderr, "Unknown OMAP image type - %x", type);
                return EXIT_FAILURE;
        }
}

All the other image checkers and no others have this, so the fix is to
simply remove the fprintf.

Signed-off-by: Dirk Behme <dirk.be...@googlemail.com>
CC: John Rigby <john.ri...@linaro.org>
CC: Aneesh V <ane...@ti.com>
CC: Sandeep Paulraj <s-paul...@ti.com>

---

Changes in v2: Instead of reordering the init sequence, remove
the fprintf().

This is reproducable with the recent mainline mkimage where the
patch "mkimage: Add OMAP boot image support" is applied:

http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=3decb14abe76d244ba98fd158ef95f89e7e37d70

 tools/omapimage.c |    1 -
 1 file changed, 1 deletion(-)

Index: u-boot.git/tools/omapimage.c
===================================================================
--- u-boot.git.orig/tools/omapimage.c
+++ u-boot.git/tools/omapimage.c
@@ -49,7 +49,6 @@ static int omapimage_check_image_types(u
        if (type == IH_TYPE_OMAPIMAGE)
                return EXIT_SUCCESS;
        else {
-               fprintf(stderr, "Unknown OMAP image type - %x", type);
                return EXIT_FAILURE;
        }
 }
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to