On 31 October 2017 at 05:41, Gao, Liming <[email protected]> wrote:
> Ard:
>   I have no other comments.
>
> Reviewed-by: Liming Gao <[email protected]>
>

Thanks. I already pushed this as 8512fc5731df



>>-----Original Message-----
>>From: Ard Biesheuvel [mailto:[email protected]]
>>Sent: Thursday, October 26, 2017 6:12 PM
>>To: [email protected]
>>Cc: [email protected]; Gao, Liming <[email protected]>; Zhu,
>>Yonghong <[email protected]>; Ard Biesheuvel
>><[email protected]>
>>Subject: [PATCH] BaseTools/tools_def: suppress GCC predefined macros in
>>DTB compilation
>>
>>The standard GCC preprocessor we use to preprocess device tree sources
>>files has a whole bunch of macros predefined, among which
>>
>>This causes a property like 'linux,code' to be converted into '1,code'
>>which is obviously wrong. So let's get rid of all the predefined macros
>>by passing -undef to the preprocessor command line.
>>
>>Contributed-under: TianoCore Contribution Agreement 1.1
>>Signed-off-by: Ard Biesheuvel <[email protected]>
>>---
>> BaseTools/Conf/tools_def.template | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>>diff --git a/BaseTools/Conf/tools_def.template
>>b/BaseTools/Conf/tools_def.template
>>index df7c109438fd..98df0ffc9294 100755
>>--- a/BaseTools/Conf/tools_def.template
>>+++ b/BaseTools/Conf/tools_def.template
>>@@ -4369,7 +4369,7 @@ DEFINE GCC_VFRPP_FLAGS             = -x c -E -P -
>>DVFRCOMPILE --include $(DEST_DI
>> DEFINE GCC_ASLPP_FLAGS             = -x c -E -include AutoGen.h
>> DEFINE GCC_ASLCC_FLAGS             = -x c
>> DEFINE GCC_WINDRES_FLAGS           = -J rc -O coff
>>-DEFINE GCC_DTCPP_FLAGS             = -E -x assembler-with-cpp -imacros
>>$(DEST_DIR_DEBUG)/AutoGen.h -nostdinc
>>+DEFINE GCC_DTCPP_FLAGS             = -E -x assembler-with-cpp -imacros
>>$(DEST_DIR_DEBUG)/AutoGen.h -nostdinc -undef
>> DEFINE GCC_IA32_RC_FLAGS           = -I binary -O elf32-i386          -B 
>> i386    --
>>rename-section .data=.hii
>> DEFINE GCC_X64_RC_FLAGS            = -I binary -O elf64-x86-64        -B 
>> i386    --
>>rename-section .data=.hii
>> DEFINE GCC_IPF_RC_FLAGS            = -I binary -O elf64-ia64-little   -B 
>> ia64    --
>>rename-section .data=.hii
>>--
>>2.11.0
>
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to