diff mbox

[U-Boot,39/48] x86: Add Makefile settings for EFI build

Message ID 1437580180-6405-40-git-send-email-sjg@chromium.org
State Superseded
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass July 22, 2015, 3:49 p.m. UTC
This is not in a good place. I'm hoping that Masahiro can help figure this
one out.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 scripts/Makefile.build | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Bin Meng July 24, 2015, 7:42 a.m. UTC | #1
Hi Simon,

On Wed, Jul 22, 2015 at 11:49 PM, Simon Glass <sjg@chromium.org> wrote:
> This is not in a good place. I'm hoping that Masahiro can help figure this
> one out.

Please check my approach without the need to change this file.

http://patchwork.ozlabs.org/patch/499458/

>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
>  scripts/Makefile.build | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index ac0554e..9a7d28d 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -294,6 +294,14 @@ $(obj)/%.lst: $(src)/%.c FORCE
>  # Compile assembler sources (.S)
>  # ---------------------------------------------------------------------------
>
> +# TODO(sjg@chromium.org): Move this to a sensible place. It does not seem to
> +# work if placed in arch/x86/config.mk, etc. It is placed here so that
> +# we use the correct flags when assembling u-boot-dtb.bin.S.
> +ifdef CONFIG_X86
> +AFLAGS_REMOVE_u-boot-dtb.bin.o += -mregparm=3 -march=i386 -m32
> +AFLAGS_u-boot-dtb.bin.o += -fpic -fshort-wchar
> +endif
> +
>  modkern_aflags := $(KBUILD_AFLAGS_KERNEL) $(AFLAGS_KERNEL)
>
>  $(real-objs-m)      : modkern_aflags := $(KBUILD_AFLAGS_MODULE) $(AFLAGS_MODULE)
> --

Regards,
Bin
diff mbox

Patch

diff --git a/scripts/Makefile.build b/scripts/Makefile.build
index ac0554e..9a7d28d 100644
--- a/scripts/Makefile.build
+++ b/scripts/Makefile.build
@@ -294,6 +294,14 @@  $(obj)/%.lst: $(src)/%.c FORCE
 # Compile assembler sources (.S)
 # ---------------------------------------------------------------------------
 
+# TODO(sjg@chromium.org): Move this to a sensible place. It does not seem to
+# work if placed in arch/x86/config.mk, etc. It is placed here so that
+# we use the correct flags when assembling u-boot-dtb.bin.S.
+ifdef CONFIG_X86
+AFLAGS_REMOVE_u-boot-dtb.bin.o += -mregparm=3 -march=i386 -m32
+AFLAGS_u-boot-dtb.bin.o += -fpic -fshort-wchar
+endif
+
 modkern_aflags := $(KBUILD_AFLAGS_KERNEL) $(AFLAGS_KERNEL)
 
 $(real-objs-m)      : modkern_aflags := $(KBUILD_AFLAGS_MODULE) $(AFLAGS_MODULE)