[coreboot-gerrit] New patch to review for coreboot: nb/amd/amdfam10: Write MCT variables to Flash after PCI configuration

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sun Apr 10 04:48:57 CEST 2016


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14305

-gerrit

commit 6a39cbf871a9896749486499ce5c400a8820900d
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Sat Apr 9 21:47:18 2016 -0500

    nb/amd/amdfam10: Write MCT variables to Flash after PCI configuration
    
    The SPI controller needs to be set up on devices such as the SP1500
    before it can be accessed to write MCT backup data.  Move the backup
    data write after PCI configuration has been completed.
    
    Change-Id: Ibcf31755242ac058407a422ce8aa33d6b0b293c7
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdfam10/northbridge.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/northbridge/amd/amdfam10/northbridge.c b/src/northbridge/amd/amdfam10/northbridge.c
index 06079dd..da93175 100644
--- a/src/northbridge/amd/amdfam10/northbridge.c
+++ b/src/northbridge/amd/amdfam10/northbridge.c
@@ -1975,10 +1975,6 @@ static void root_complex_enable_dev(struct device *dev)
 	/* Do not delay UMA setup, as a device on the PCI bus may evaluate
 	   the global uma_memory variables already in its enable function. */
 	if (!done) {
-#if IS_ENABLED(CONFIG_HAVE_ACPI_RESUME) && IS_ENABLED(CONFIG_DIMM_DDR3)
-		save_mct_information_to_nvram();
-#endif
-
 		setup_bsp_ramtop();
 		setup_uma_memory();
 		done = 1;
@@ -1992,7 +1988,14 @@ static void root_complex_enable_dev(struct device *dev)
 	}
 }
 
+static void root_complex_finalize(void *chip_info) {
+#if IS_ENABLED(CONFIG_HAVE_ACPI_RESUME) && IS_ENABLED(CONFIG_DIMM_DDR3)
+	save_mct_information_to_nvram();
+#endif
+}
+
 struct chip_operations northbridge_amd_amdfam10_root_complex_ops = {
 	CHIP_NAME("AMD Family 10h/15h Root Complex")
 	.enable_dev = root_complex_enable_dev,
+	.final = root_complex_finalize,
 };



More information about the coreboot-gerrit mailing list