[coreboot-gerrit] New patch to review for coreboot: 4ce0b78 libpayload: Fix xcompile

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Nov 19 00:58:17 CET 2013


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4179

-gerrit

commit 4ce0b7871b289f032cfeaf7e69b741e1a3e19aab
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Tue May 21 12:15:50 2013 -0700

    libpayload: Fix xcompile
    
    The architecture name for our ARM port is armv7, not arm.
    Hence, none of those flags were ever actually used.
    
    Fix the architecture name and remove the flags, they should
    not be set in xcompile, but in the Makefile, like in coreboot.
    
    Change-Id: Id9c5db7ebceafddb58a1ce1988417f09c074ba6c
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/56084
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 payloads/libpayload/util/xcompile/xcompile | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/payloads/libpayload/util/xcompile/xcompile b/payloads/libpayload/util/xcompile/xcompile
index 58f086f..09c1879 100644
--- a/payloads/libpayload/util/xcompile/xcompile
+++ b/payloads/libpayload/util/xcompile/xcompile
@@ -106,12 +106,9 @@ detect_special_flags() {
 		CFLAGS="$CFLAGS -Wl,--build-id=none"
 
 	case "$architecture" in
-		arm )
+		armv7 )
 			# testcc "$CC" "$CFLAGS -mcpu=cortex-a9" &&
 			#	CFLAGS="$CFLAGS -mcpu=cortex-a9"
-			testcc "$CC" \
-"$CFLAGS -ffixed-r8 -msoft-float -marm -mabi=aapcs-linux -march=armv7" &&
-CFLAGS="$CFLAGS -ffixed-r8 -msoft-float -marm -mabi=aapcs-linux -march=armv7"
 			;;
 	esac
 }



More information about the coreboot-gerrit mailing list