[coreboot-gerrit] New patch to review for coreboot: nb/intel/sandybridge/raminit: Always run quick_ram_check

Patrick Rudolph (siro@das-labor.org) gerrit at coreboot.org
Sat Nov 26 16:04:46 CET 2016


Patrick Rudolph (siro at das-labor.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17615

-gerrit

commit 55b4f413da0c87d065255c074e45c5db8c81cf6d
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Sat Nov 26 10:11:14 2016 +0100

    nb/intel/sandybridge/raminit: Always run quick_ram_check
    
    quick_ram_check doesn't change contents of memory.
    Run it in S3 resume, too.
    
    Tested on Lenovo T420.
    
    Change-Id: Icaf3650fadbb3bb87d8c780a9e79737c3cf7eb06
    Signed-off-by: Patrick Rudolph <siro at das-labor.org>
---
 src/northbridge/intel/sandybridge/raminit.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/intel/sandybridge/raminit.c b/src/northbridge/intel/sandybridge/raminit.c
index 956234b..7560b7e 100644
--- a/src/northbridge/intel/sandybridge/raminit.c
+++ b/src/northbridge/intel/sandybridge/raminit.c
@@ -425,8 +425,8 @@ static void init_dram_ddr3(int mobile, int min_tck, int s3resume)
 	/* Zone config */
 	dram_zones(&ctrl, 0);
 
-	if (!fast_boot)
-		quick_ram_check();
+	/* Non intrusive, fast ram check */
+	quick_ram_check();
 
 	intel_early_me_status();
 	intel_early_me_init_done(ME_INIT_STATUS_SUCCESS);



More information about the coreboot-gerrit mailing list