[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge/raminit: Add ECC debug code

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/40946 )

Change subject: nb/intel/sandybridge/raminit: Add ECC debug code
......................................................................

nb/intel/sandybridge/raminit: Add ECC debug code

* Add ECC test code when DEBUG_RAM_SETUP is enabled
* Move ECC scrubbing after set_scrambling_seed() to be able to observe
  what has been cleared in the test routine.
* ECC scrubbing must happen after dram_dimm_set_mapping()
* Move method out of try_init_dram_ddr3()

Change-Id: I76174ec962c9b0bb72852897586eb95d896d301e
Signed-off-by: Patrick Rudolph 
---
M src/northbridge/intel/sandybridge/raminit.c
M src/northbridge/intel/sandybridge/raminit_native.c
2 files changed, 25 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/40946/1

diff --git a/src/northbridge/intel/sandybridge/raminit.c b/src/northbridge/intel/sandybridge/raminit.c
index 6c8145d..ff563e7 100644
--- a/src/northbridge/intel/sandybridge/raminit.c
+++ b/src/northbridge/intel/sandybridge/raminit.c
@@ -366,10 +366,35 @@

set_scrambling_seed(&ctrl);

+	if (!s3resume && ctrl.ecc_enabled)
+		channel_scrub(&ctrl);
+
    set_normal_operation(&ctrl);

final_registers(&ctrl);

+	/* can't do this earlier because it needs to be done in normal operation */
+	if (CONFIG(DEBUG_RAM_SETUP) && !s3resume && ctrl.ecc_enabled) {
+		uint32_t i, tseg = pci_read_config32(HOST_BRIDGE, TSEGMB);
+
+		printk(BIOS_INFO, "RAMINIT: ECC scrub test on first channel up to 0x%x\n",
+		       tseg);
+
+		/* Skip first MB to avoid special case for A-seg and test up to TSEG */
+		for (i = 1; i < tseg >> 20; i++) {
+			for (int j = 0; j < 1 * MiB; j += 4096) {
+				uintptr_t addr = i * MiB + j;
+				if (read32((u32 *)addr) == 0)
+					continue;
+
+				printk(BIOS_ERR, "RAMINIT: ECC scrub: DRAM not cleared at"
+				       " addr 0x%lx\n", addr);
+				break;
+			}
+		}
+		printk(BIOS_INFO, "RAMINIT: ECC scrub test done.\n");
+	}
+
    /* Zone config */
    dram_zones(&ctrl, 0);

diff --git a/src/northbridge/intel/sandybridge/raminit_native.c b/src/northbridge/intel/sandybridge/raminit_native.c
index 832391f..34299a3 100644
--- a/src/northbridge/intel/sandybridge/raminit_native.c
+++ b/src/northbridge/intel/sandybridge/raminit_native.c
@@ -685,9 +685,6 @@
    	err = channel_test(ctrl);
    	if (err)
    		return err;
-
-		if (ctrl->ecc_enabled)
-			channel_scrub(ctrl);
    }

/* Set MAD-DIMM registers */