[coreboot-gerrit] Patch merged into coreboot/master: ee905a8 vendorcode/amd/agesa/fam15tn: Build as a static library

gerrit at coreboot.org gerrit at coreboot.org
Tue Apr 15 01:30:26 CEST 2014


the following patch was just integrated into master:
commit ee905a8161a2cde891c6ce4dffbf99a89806658d
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Sat Mar 29 13:15:18 2014 -0500

    vendorcode/amd/agesa/fam15tn: Build as a static library
    
    Up until now, we were building AGESA by specifying each AGESA source
    file and adding it to the list of romstage and ramstage source files.
    As a result, we were compiling each AGESA source twice, despite the
    fact that it does not depend on the stage we're in.
    
    Since AGESA is stage-independent, we can build it just once, and use
    the resulting static library in both rom and ram stages.
    
    We still keep the practice of specifying every single AGESA directory
    as an include dir and adding the AGESA CFLAGS to our global CFLAGS;
    this is needed due to the way AGESA builds.
    
    Change-Id: I9b23264129d1c08cb67cabc31d15a68d43ed7624
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>
    Reviewed-on: http://review.coreboot.org/5430
    Tested-by: build bot (Jenkins)
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-by: Idwer Vollering <vidwer at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list