From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heinrich Schuchardt <xypron.glpk@gmx.de>
Date: Wed, 18 Apr 2018 17:48:45 +0200
Subject: [U-Boot] [PATCH 3/8] efi_loader: selftest: Do not build
 relocation tests for risc-v
In-Reply-To: <20180418134030.55127-4-agraf@suse.de>
References: <20180418134030.55127-1-agraf@suse.de>
 <20180418134030.55127-4-agraf@suse.de>
Message-ID: <944e65e7-16c1-83d3-0975-0164e820df5a@gmx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 04/18/2018 03:40 PM, Alexander Graf wrote:
> The relocation selftest doesn't compile for me on RISC-V. Disable for now.

Can this problem be reproduced with one of the qemu targets?
Does loading a binary work?

> 
> Signed-off-by: Alexander Graf <agraf@suse.de>
> ---
>  lib/efi_selftest/Makefile | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/lib/efi_selftest/Makefile b/lib/efi_selftest/Makefile
> index 31b444fc8b..ede7831449 100644
> --- a/lib/efi_selftest/Makefile
> +++ b/lib/efi_selftest/Makefile
> @@ -35,9 +35,10 @@ ifeq ($(CONFIG_BLK)$(CONFIG_PARTITIONS),yy)
>  obj-$(CONFIG_CMD_BOOTEFI_SELFTEST) += efi_selftest_block_device.o
>  endif
>  
> -# TODO: As of v2018.01 the relocation code for the EFI application cannot
> -# be built on x86_64.
> +# TODO: As of v2018.05 the relocation code for the EFI application cannot
> +# be built on x86_64 / RISC-V.
>  ifeq ($(CONFIG_X86_64),)
> +ifeq ($(CONFIG_RISCV),)

Elsewhere we abbreviate (!A && !B) as:

ifeq ($(CONFIG_X86_64)$(CONFIG_RISCV),)

Regards

Heinrich

>  
>  ifneq ($(CONFIG_CMD_BOOTEFI_SELFTEST),)
>  
> @@ -63,6 +64,8 @@ $(obj)/efi_selftest_startimage_exit.o: $(obj)/efi_miniapp_file_image_exit.h
>  
>  $(obj)/efi_selftest_startimage_return.o: $(obj)/efi_miniapp_file_image_return.h
>  
> -endif
> +endif # CONFIG_CMD_BOOTEFI_SELFTEST
>  
> -endif
> +endif # !CONFIG_RISCV
> +
> +endif # !CONFIG_X86_64
>