[coreboot-gerrit] Patch set updated for coreboot: cf2d0c1 libpayload: Fix xcompile

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Wed Nov 20 01:11:33 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 cf2d0c1ac906365ad105869a64ddb2bd08abe2ed
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