[U-Boot] [PATCH v1] sandbox: Build with -fPIC

[U-Boot] [PATCH v1] sandbox: Build with -fPIC

[Andy Shevchenko]

Sandbox is not a real bootloader and it does require
a position independent code to be supported.

Thus, build it with -fPIC explicitly.

Fixes: 16940f720f9b ("Makefile: Don't generate position independent code")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reported-by: Simon Glass <sjg@chromium.org>
---
 arch/sandbox/config.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/sandbox/config.mk b/arch/sandbox/config.mk
index 5e7077bfe7..4599839791 100644
--- a/arch/sandbox/config.mk
+++ b/arch/sandbox/config.mk
@@ -3,6 +3,7 @@
 
 PLATFORM_CPPFLAGS += -D__SANDBOX__ -U_FORTIFY_SOURCE
 PLATFORM_CPPFLAGS += -DCONFIG_ARCH_MAP_SYSMEM
+PLATFORM_CPPFLAGS += -fPIC
 PLATFORM_LIBS += -lrt
 
 LDFLAGS_FINAL += --gc-sections
-- 
2.18.0

[U-Boot] [PATCH v1] sandbox: Build with -fPIC

[Bin Meng]

Hi Andy,

On Mon, Sep 3, 2018 at 4:33 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> Sandbox is not a real bootloader and it does require
> a position independent code to be supported.
>
> Thus, build it with -fPIC explicitly.
>
> Fixes: 16940f720f9b ("Makefile: Don't generate position independent code")
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Reported-by: Simon Glass <sjg@chromium.org>
> ---
>  arch/sandbox/config.mk | 1 +
>  1 file changed, 1 insertion(+)

This patch does not apply on top of u-boot/master. Can you please rebase?

Manually applied the changes, and tested Sandbox on my Ubuntu 16.04
machine, although it does not expose the issue that Simon was seeing
on 18.04.

FWIW

Tested-by: Bin Meng <bmeng.cn@gmail.com>

Regards,
Bin