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

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