[PATCH v2 09/39] efi: Drop the OF_EMBED warning for EFI

Heinrich Schuchardt xypron.glpk at gmx.de
Sat Oct 23 13:37:14 CEST 2021



On 9/25/21 02:30, Simon Glass wrote:
> For the EFI app, we must embed the devicetree in the ELF file since that
> is the only thing that is run by UEFI. Drop the warning to avoid
> confusion.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Add new patch to drop the OF_EMBED warning for EFI
>
>   Makefile | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index a09f48f84b2..784efd4ef6e 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1087,7 +1087,7 @@ endif
>   ifeq ($(CONFIG_DEPRECATED),y)
>   	$(warning "You have deprecated configuration options enabled in your .config! Please check your configuration.")
>   endif
> -ifeq ($(CONFIG_OF_EMBED),y)
> +ifeq ($(CONFIG_OF_EMBED)$(CONFIG_EFI_APP),y)
>   	@echo >&2 "===================== WARNING ======================"
>   	@echo >&2 "CONFIG_OF_EMBED is enabled. This option should only"
>   	@echo >&2 "be used for debugging purposes. Please use"

There are 47 defconfigs and 5 include/configs/*.h using CONFIG_OF_EMBED.
doc/board/intel/slimbootloader.rst and doc/board/coreboot/coreboot.rst
require to use it for building U-Boot.

Is there a migration requirement and plan?

Best regards

Heinrich


More information about the U-Boot mailing list