spl: Rework image header parse to allow abort on raw image and os boot
authorPaul Kocialkowski <contact@paulk.fr>
Wed, 24 Aug 2016 18:04:42 +0000 (20:04 +0200)
committerTom Rini <trini@konsulko.com>
Tue, 6 Sep 2016 17:41:38 +0000 (13:41 -0400)
This reworks spl_set_header_raw_uboot to allow having both os boot
(which comes with a valid header) and aborting when no valid header is
found (thus excluding raw u-boot.bin images).

Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
Reviewed-by: Tom Rini <trini@konsulko.com>
common/spl/spl.c

index b7ec333c8a5fcf8081446803b01f0af6d2b2abf0..e14ec8052e9a075b9dfde8642f9c5c04c53e03d4 100644 (file)
@@ -129,10 +129,8 @@ int spl_parse_image_header(const struct image_header *header)
                 * is bad, and thus should be skipped silently.
                 */
                panic("** no mkimage signature but raw image not supported");
-#elif defined(CONFIG_SPL_ABORT_ON_RAW_IMAGE)
-               /* Signature not found, proceed to other boot methods. */
-               return -EINVAL;
-#else
+#endif
+
 #ifdef CONFIG_SPL_OS_BOOT
                ulong start, end;
 
@@ -147,6 +145,11 @@ int spl_parse_image_header(const struct image_header *header)
                        return 0;
                }
 #endif
+
+#ifdef CONFIG_SPL_ABORT_ON_RAW_IMAGE
+               /* Signature not found, proceed to other boot methods. */
+               return -EINVAL;
+#else
                /* Signature not found - assume u-boot.bin */
                debug("mkimage signature not found - ih_magic = %x\n",
                        header->ih_magic);