[coreboot] Patch merged into coreboot/master: 3d63b0a BiosCallOuts: Replace REQUIRED_CALLOUTS define with flexible variable

gerrit at coreboot.org gerrit at coreboot.org
Fri Dec 28 21:19:43 CET 2012


the following patch was just integrated into master:
commit 3d63b0a9656d5982dd0e9eef652c6cc1e8f8eaa3
Author: Aladyshev Konstantin <aladyshev at nicevt.ru>
Date:   Wed Dec 19 00:58:35 2012 +0400

    BiosCallOuts: Replace REQUIRED_CALLOUTS define with flexible variable
    
    Size of BiosCallouts[] struct can be calculated as:
    
            CallOutCount = sizeof (BiosCallouts) / sizeof (BiosCallouts [0]);
    
    There is no longer need for REQUIRED_CALLOUTS define.
    
    Originally that change was done for AMD Persimmon in
    
            commit d7a696d0f229abccc95ff411f28d91b9b796ab74
            Author: efdesign98 <efdesign98 at gmail.com>
            Date:   Thu Sep 15 15:24:26 2011 -0600
    
                Persimmon updates for AMD F14 rev C0
    
    without deleting the define. This was ported to some of the other
    boards and for some the define was not removed.
    
    The AMD Inagua, Parmer and Thatcher boards were already adapted but
    the define was left in. So just remove it for those.
    
    Tested on Supermicro H8QGI.
    
    Change-Id: Ia09795579a1170fa20ab94a30feb1af6821153d2
    Signed-off-by: Aladyshev Konstantin <aladyshev at nicevt.ru>
    Reviewed-on: http://review.coreboot.org/2049
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Tested-by: build bot (Jenkins)
    Reviewed-by: Martin Roth <martin at se-eng.com>
    Reviewed-by: Marc Jones <marcj303 at gmail.com>

Build-Tested: build bot (Jenkins) at Wed Dec 19 19:46:28 2012, giving +1
Reviewed-By: Marc Jones <marcj303 at gmail.com> at Fri Dec 28 21:19:42 2012, giving +2
See http://review.coreboot.org/2049 for details.

-gerrit



More information about the coreboot mailing list