[coreboot-gerrit] Change in coreboot[master]: nb/amd/amdk8: Don't bail out on dimms on channel 1

Arthur Heymans (Code Review) gerrit at coreboot.org
Wed May 10 17:05:48 CEST 2017


Arthur Heymans has uploaded a new change for review. ( https://review.coreboot.org/19655 )

Change subject: nb/amd/amdk8: Don't bail out on dimms on channel 1
......................................................................

nb/amd/amdk8: Don't bail out on dimms on channel 1

UNTESTED.

Change-Id: Ieb4a7c7c0e154bdb683d22691f4323635b37cae3
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/amd/amdk8/raminit.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/19655/1

diff --git a/src/northbridge/amd/amdk8/raminit.c b/src/northbridge/amd/amdk8/raminit.c
index 43229ea..8c05c03 100644
--- a/src/northbridge/amd/amdk8/raminit.c
+++ b/src/northbridge/amd/amdk8/raminit.c
@@ -2185,7 +2185,7 @@
 	hw_enable_ecc(ctrl);
 	activate_spd_rom(ctrl);
 	dimm_mask = spd_detect_dimms(ctrl);
-	if (!(dimm_mask & ((1 << DIMM_SOCKETS) - 1))) {
+	if (!(dimm_mask & ((1 << DIMM_SOCKETS * 2) - 1))) {
 		printk(BIOS_DEBUG, "No memory for this cpu\n");
 		return;
 	}

-- 
To view, visit https://review.coreboot.org/19655
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieb4a7c7c0e154bdb683d22691f4323635b37cae3
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>



More information about the coreboot-gerrit mailing list