mkimage: Fix 'Unknown OMAP image type - 5'
authorDirk Behme <dirk.behme@googlemail.com>
Tue, 9 Aug 2011 07:52:57 +0000 (07:52 +0000)
committerAlbert ARIBAUD <albert.u.boot@aribaud.net>
Sun, 4 Sep 2011 09:34:00 +0000 (11:34 +0200)
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.behme@googlemail.com>
CC: John Rigby <john.rigby@linaro.org>
CC: Aneesh V <aneesh@ti.com>
CC: Sandeep Paulraj <s-paulraj@ti.com>
tools/omapimage.c

index 29d91cc69c1d1d4cc6c936a12094676211fc0237..5e739ac64758e0f13465f0b29f91d13822c4f6b5 100644 (file)
@@ -49,7 +49,6 @@ 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;
        }
 }