[coreboot] Patch set updated for coreboot: 5605707 update CFLAGS for armv7

Ronald G. Minnich (rminnich@gmail.com) gerrit at coreboot.org
Thu Jan 3 22:37:35 CET 2013


Ronald G. Minnich (rminnich at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2075

-gerrit

commit 5605707c208935ecefd07c4430ff7a053315ddeb
Author: David Hendricks <dhendrix at chromium.org>
Date:   Thu Dec 27 13:30:55 2012 -0800

    update CFLAGS for armv7
    
    This updates $CFLAGS used for armv7. Most of them were just added
    to be consistent with what u-boot does. The important ones here
    are -march=armv7-a and -mthumb (to allow 16-bit Thumb instructions).
    
    I removed the hard float support because it got errors and
    coreboot should never use floats anyway. We're still having trouble
    with enums but I want to see how far it gets with this patch.
    
    Also, put the flags in a form that makes diffs easier to read. It's
    almost impossible otherwise.
    
    Depends-On: I6f730d017391f9ec4401cdfd34931c869df10a9e
    Change-Id: Ia8a1ae22959933e06f7b996d1832cea40819f1ff
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
---
 util/xcompile/xcompile | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 4239176..d800ba0 100644
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -106,14 +106,21 @@ detect_special_flags() {
 		CFLAGS="$CFLAGS -Wl,--build-id=none"
 
 	case "$architecture" in
-		arm )
-			# testcc "$CC" "$CFLAGS -mcpu=cortex-a9" &&
-			#	CFLAGS="$CFLAGS -mcpu=cortex-a9"
-			testcc "$CC" "\
-$CFLAGS -ffixed-r8 -msoft-float -marm -mabi=aapcs-linux \
--mno-thumb-interwork -march=armv7 -mno-thumb-interwork" && CFLAGS="\
-$CFLAGS -ffixed-r8 -msoft-float -marm -mabi=aapcs-linux \
--mno-thumb-interwork -march=armv7 -mno-thumb-interwork"
+		armv7 )
+			ARMFLAGS="\
+			-fno-builtin\
+			-fno-common-ffixed-r8\
+			-mabi=aapcs-linux\
+			-march=armv7-a\
+			-marm\
+			-mno-unaligned-access-Werror\
+			-mthumb-mthumb-interwork\
+			-pipe\
+			-Wall\
+			-Wno-format-nonliteral\
+			-Wno-format-security"
+
+			testcc "$CC" "$CFLAGS $ARMFLAGS" && CFLAGS="$CFLAGS $ARMFLAGS"
 			;;
 	esac
 }



More information about the coreboot mailing list