On 11/13/19 1:52 AM, AKASHI Takahiro wrote:
This commit allows us to use common/image-sig.c even if CONFIG_FIT
is disabled but CONFIG_EFI_LOADER is enabled.
Signed-off-by: AKASHI Takahiro <takahiro.aka...@linaro.org>
---
include/image.h | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/include/image.h b/include/image.h
index bff87f51f01b..b79424a39c22 100644
--- a/include/image.h
+++ b/include/image.h
@@ -53,7 +53,7 @@ struct fdt_region;
#endif /* USE_HOSTCC */
-#if IMAGE_ENABLE_FIT
+#if IMAGE_ENABLE_FIT || defined(CONFIG_EFI_SECURE_BOOT)
Is this #if needed at all?
Did you run Travis to check that defining CONFIG_EFI_SECURE_BOOT does
not lead to build failures if IMAGE_ENABLE_FIT is not defined?
#include <hash.h>
#include <linux/libfdt.h>
#include <fdt_support.h>
@@ -86,13 +86,14 @@ struct fdt_region;
#endif
#if defined(CONFIG_FIT_ENABLE_SHA256_SUPPORT) || \
- defined(CONFIG_SPL_SHA256_SUPPORT)
+ defined(CONFIG_SPL_SHA256_SUPPORT) || \
+ defined(CONFIG_EFI_SECURE_BOOT)
IMAGE_ENABLE_SHA256 is only used in common/image-fit.c. So why would you
change anything here?
#define IMAGE_ENABLE_SHA256 1
#else
#define IMAGE_ENABLE_SHA256 0
#endif
-#endif /* IMAGE_ENABLE_FIT */
+#endif /* IMAGE_ENABLE_FIT || defined(CONFIG_EFI_SECURE_BOOT) */
#ifdef CONFIG_SYS_BOOT_GET_CMDLINE
# define IMAGE_BOOT_GET_CMDLINE 1
@@ -1261,7 +1262,6 @@ struct crypto_algo *image_get_crypto_algo(const char
*full_name);
struct padding_algo *image_get_padding_algo(const char *name);
#if IMAGE_ENABLE_FIT
-
This change is unrelated. It should be in a separate patch. Or at least
mention in the commit message that you cleaned up other parts of the code.
/**
* fit_image_verify_required_sigs() - Verify signatures marked as 'required'
*
@@ -1337,7 +1337,7 @@ static inline int fit_image_check_target_arch(const void
*fdt, int node)
#define fit_unsupported(msg)
#define fit_unsupported_reset(msg)
#endif /* CONFIG_FIT_VERBOSE */
-#endif /* CONFIG_FIT */
+#endif /* IMAGE_ENABLE_FIT */
Same here.
Best regards
Heinrich
#if defined(CONFIG_ANDROID_BOOT_IMAGE)
struct andr_img_hdr;
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot