[coreboot] Patch merged into coreboot/master: dd9e9f1 Add a capability for mainboard-specific posting.

gerrit at coreboot.org gerrit at coreboot.org
Sat Aug 4 19:31:22 CEST 2012


the following patch was just integrated into master:
commit dd9e9f1786dbdb314ce0c4244dee85357e2ae58a
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Fri Aug 3 13:20:57 2012 -0500

    Add a capability for mainboard-specific posting.
    
    Some mainboards have really nice capabilities for posting, beyond
    simple POST cards. Further, some can not use a POST card. This
    change defines a weak symbol (mainboard_post) that can be overridden
    by a real mainboard_post function.
    
    If, for example, you'd like to do something fancy before the payload starts,
    you can add this to mainboard.c:
    
    void mainboard_post(u8 value)
    {
    	switch(value){
    		case POST_TIME_TO_PARTY: some_fancy_lights();
    		break;
    	}
    }
    
    Maybe the post function should be an entry in the device. We're beginning to over-use
    weak symbols.
    
    BUG=None
    
    TEST=Build and boot a google chromebook. Observe that it still works. Use it to drive
    some pretty lights.
    
    Change-Id: I3512d2ec34a66c747287191851c3f68b6a7cc1b2
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>

Build-Tested: build bot (Jenkins) at Fri Aug  3 20:56:15 2012, giving +1
Reviewed-By: Alexandru Gagniuc <mr.nuke.me at gmail.com> at Sat Aug  4 19:31:19 2012, giving +2
See http://review.coreboot.org/1397 for details.

-gerrit




More information about the coreboot mailing list