[coreboot-gerrit] Patch merged into coreboot/master: fbff908 coreboot classes: Add dynamic classes to coreboot

gerrit at coreboot.org gerrit at coreboot.org
Tue Mar 17 16:38:16 CET 2015


the following patch was just integrated into master:
commit fbff908a7404a38d937d773e24824a52c4df389c
Author: Furquan Shaikh <furquan at google.com>
Date:   Tue Jul 22 15:59:16 2014 -0700

    coreboot classes: Add dynamic classes to coreboot
    
    Provide functionality to create dynamic classes based on program name and the
    architecture for which the program needs to be compiled/linked. define_class
    takes program_name and arch as its arguments and adds the program_name to
    classes-y to create dynamic class and compiler toolset is created for the
    specified arch. All the files for this program can then be added to
    program_name-y += .. Ensure that define_class is called before any files are
    added to the class. Check subdirs-y for order of directory inclusion.
    
    One such example of dynamic class is rmodules. Multiple rmodules can be used
    which need to be compiled for different architectures. With dynamic classes,
    this is possible.
    
    BUG=chrome-os-partner:30784
    BRANCH=None
    TEST=Compiles successfully for nyan, rush and link.
    
    Original-Change-Id: I3e3aadbe723d432b9b3500c44bcff578c98f5643
    Original-Signed-off-by: Furquan Shaikh <furquan at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/209379
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Tested-by: Furquan Shaikh <furquan at chromium.org>
    Original-Commit-Queue: Aaron Durbin <adurbin at chromium.org>
    (cherry picked from commit 242bb90d7476c2ee47d60c50ee18785edeb1a295)
    
    Some of this cherry-pick had already been committed here:
    commit 133096b6dc31163f59f658e15f2eb342a0de2ac6
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I9f5868d704c4b3251ca6f54afa634588108a788c
    Reviewed-on: http://review.coreboot.org/8672
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list