[coreboot-gerrit] Change in coreboot[master]: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/20971


Change subject: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop
......................................................................

nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop

No need to further search for non matching SPD when first one is
found.

Change-Id: Ie669d1bc38411e40ab95c28b17ee8818874dc741
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
1 file changed, 5 insertions(+), 2 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/20971/1

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
index da803ff..e4612f1 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -2715,9 +2715,12 @@
 			struct DCTStatStruc *pDCTstat;
 			pDCTstat = pDCTstatA + Node;
 
-			if (pDCTstat->NodePresent)
-				if (!pDCTstat->spd_data.nvram_spd_match)
+			if (pDCTstat->NodePresent) {
+				if (!pDCTstat->spd_data.nvram_spd_match) {
 					allow_config_restore = 0;
+					break;
+				}
+			}
 		}
 
 		/* FIXME

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie669d1bc38411e40ab95c28b17ee8818874dc741
Gerrit-Change-Number: 20971
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170812/2a00858e/attachment-0001.html>