[coreboot-gerrit] Patch merged into coreboot/master: b3997ba arm: Remove some pointless CFLAGS

gerrit at coreboot.org gerrit at coreboot.org
Fri Sep 12 22:09:13 CEST 2014


the following patch was just integrated into master:
commit b3997ba6f2ba26e0dfa851caed98f030ac25ffd0
Author: Julius Werner <jwerner at chromium.org>
Date:   Tue Nov 12 14:46:07 2013 -0800

    arm: Remove some pointless CFLAGS
    
    This patch removes the -ffixed-r8 CFLAG from the coreboot and libpayload
    Makefiles. This seems to be a relic from U-Boot, which uses that
    register to keep it's global data structure pointer. There's no reason
    for us to throw away a perfectly fine register on this already pretty
    constrained architecture.
    
    Also removed a config.h inclusion from the Makefile because that should
    really be done inside the C files.
    
    Change-Id: Ia176c0f323c1be07cddf88fa5488788786a27cdf
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/177110
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 2a81112abde284ba09020db6afa363169911a7f6)
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6880
    Tested-by: build bot (Jenkins)
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>


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

-gerrit



More information about the coreboot-gerrit mailing list