[coreboot] Patch merged into coreboot/master: b37ec54 Tyan S8226: Use SPD read code from F15 wrapper

gerrit at coreboot.org gerrit at coreboot.org
Wed Mar 20 05:54:14 CET 2013


the following patch was just integrated into master:
commit b37ec540affaaeb3a8a230895c08778c54f1d076
Author: Kimarie Hoot <kimarie.hoot at se-eng.com>
Date:   Fri Mar 8 15:31:49 2013 -0700

    Tyan S8226: Use SPD read code from F15 wrapper
    
    Changes:
     - Get rid of the s8226 mainboard specific code and use the
       platform generic function wrapper that was added in change
       http://review.coreboot.org/#/c/2777/
       AMD Fam15: Add SPD read functions to wrapper code
    
     - Move DIMM addresses into devicetree.cb
    
    Notes:
     - The DIMM reads only happen in romstage, so the function is not
       available in ramstage.  Point the read-SPD callback to a generic
       function in ramstage.
     - select_socket() and restore_socket() started by duplicating
       sp5100_set_gpio() and sp5100_restore_gpio(), which were in
       dimmSpd.c.  In addition to renaming the functions to more
       specifically state their purpose, some cleanup and magic number
       reduction was done.
    
    Change-Id: I1eaf64986ef4fa3f89aed2b69d3f9c8c913f726f
    Signed-off-by: Kimarie Hoot <kimarie.hoot at se-eng.com>
    Reviewed-on: http://review.coreboot.org/2827
    Tested-by: build bot (Jenkins)
    Reviewed-by: Siyuan Wang <wangsiyuanbuaa at gmail.com>
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>

Build-Tested: build bot (Jenkins) at Tue Mar 19 18:07:15 2013, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer at coreboot.org> at Wed Mar 20 05:54:08 2013, giving +2
See http://review.coreboot.org/2827 for details.

-gerrit



More information about the coreboot mailing list