[coreboot-gerrit] Patch merged into coreboot/master: 05a8472 xcompile: improve mips toolchain handling

gerrit at coreboot.org gerrit at coreboot.org
Tue Apr 28 03:22:05 CEST 2015


the following patch was just integrated into master:
commit 05a8472900ab2a57218b5f3affdaad1df10ed44e
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Sat Apr 25 13:05:10 2015 -0700

    xcompile: improve mips toolchain handling
    
    The mips toolchain used by coreboot so far comes from Chrome OS chroot
    and is built explicitly for little endian code generation.
    
    Other flavors of MIPS toolchain usually generate big endian code by
    default and require command line options to switch to little endian
    mode.
    
    This patch adds another variable to the set of compiler flags examined
    to determine compiler compatibility. This results in adding another
    nested for loop in test_architecture(). To avoid the need to break
    from different levels of nesting, processing of the successful case is
    taken out from test_architecture().
    
    With this change the Mentor Graphics provided mips GCC toolchain is
    accepted by xcompile, resulting in the following output:
    
     ARCH_SUPPORTED+=mips
     SUBARCH_SUPPORTED+=mips mipsel
     CC_mips:=mips-linux-gnu-gcc
     CFLAGS_mips:= -Wno-unused-but-set-variable  -fno-stack-protector -Wl,--build-id=none -mno-abicalls -fno-pic -EL
     CPP_mips:=mips-linux-gnu-cpp
     AS_mips:=mips-linux-gnu-as
     LD_mips:=mips-linux-gnu-ld
     NM_mips:=mips-linux-gnu-nm
     OBJCOPY_mips:=mips-linux-gnu-objcopy
     OBJDUMP_mips:=mips-linux-gnu-objdump
     READELF_mips:=mips-linux-gnu-readelf
     STRIP_mips:=mips-linux-gnu-strip
     AR_mips:=mips-linux-gnu-ar
    
    Change-Id: I4da384b366880929693c59dc0e1c522b35c41bea
    Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Reviewed-on: http://review.coreboot.org/9997
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>


See http://review.coreboot.org/9997 for details.

-gerrit



More information about the coreboot-gerrit mailing list