Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/27100
Change subject: nb/intel/x4x: Issue a hard reset with empty MRC cache on warm reset
......................................................................
nb/intel/x4x: Issue a hard reset with empty MRC cache on warm reset
The dram controller cannot fully initialize the memory controller on
warm reset (receive enable calibration consistently fails).
This option is mostly useful when rebooting after having flashed a new
rom which overwrites the mrc cache region.
Change-Id: I405c0eca076fe081641ede9a670f734c98cbf8fc
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/northbridge/intel/x4x/raminit.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/27100/1
diff --git a/src/northbridge/intel/x4x/raminit.c b/src/northbridge/intel/x4x/raminit.c
index d41b74c..30c6477 100644
--- a/src/northbridge/intel/x4x/raminit.c
+++ b/src/northbridge/intel/x4x/raminit.c
@@ -653,6 +653,9 @@
/* Failed S3 resume, reset to come up cleanly */
outb(0x6, 0xcf9);
halt();
+ } else if (boot_path == BOOT_PATH_WARM_RESET) {
+ outb(0xe, 0xcf9);
+ halt();
}
ctrl_cached = NULL;
} else {
--
To view, visit https://review.coreboot.org/27100
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I405c0eca076fe081641ede9a670f734c98cbf8fc
Gerrit-Change-Number: 27100
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26565 )
Change subject: nb/intel/x4x: Fix a few things in set_enhanced_mode
......................................................................
Patch Set 6: Verified-1
Build Unstable
https://qa.coreboot.org/job/coreboot-gerrit/74924/ : UNSTABLE
https://qa.coreboot.org/job/coreboot-checkpatch/29024/ : SUCCESS
--
To view, visit https://review.coreboot.org/26565
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1afa637a16a083c3a945ba3e2a71292b005736fd
Gerrit-Change-Number: 26565
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 14 Jun 2018 08:49:34 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26563 )
Change subject: nb/intel/x4x: Add the option for stacked channel map settings
......................................................................
Patch Set 4: Verified-1
Build Unstable
https://qa.coreboot.org/job/coreboot-gerrit/74922/ : UNSTABLE
https://qa.coreboot.org/job/coreboot-checkpatch/29022/ : SUCCESS
--
To view, visit https://review.coreboot.org/26563
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I95965bfea129b37f64163159fefa1c8f16331b62
Gerrit-Change-Number: 26563
Gerrit-PatchSet: 4
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 14 Jun 2018 08:45:19 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes