summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2014-11-24 22:15:09 +0100
committerPatrick Georgi <pgeorgi@google.com>2014-11-30 12:20:03 +0100
commit5f967492e351ea6bc473a6d192fe4e2aafc00b1e (patch)
tree133f4b87931f3a130f4f1d9531ad627de2a42ad9
parent4a145052a3d6cbe89f779fbc317fff8ca3f76601 (diff)
intel/sandybridge: make sure to stay in HLT until reboot
It also tells the compiler that we never leave here. Change-Id: I824569efd46b577588387b29fc7781abf8c42385 Found-by: Coverity Scan Signed-off-by: Patrick Georgi <patrick@georgi-clan.de> Reviewed-on: http://review.coreboot.org/7579 Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
-rw-r--r--src/northbridge/intel/sandybridge/raminit_native.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/northbridge/intel/sandybridge/raminit_native.c b/src/northbridge/intel/sandybridge/raminit_native.c
index 5c1af4452f..9de39c6768 100644
--- a/src/northbridge/intel/sandybridge/raminit_native.c
+++ b/src/northbridge/intel/sandybridge/raminit_native.c
@@ -3751,7 +3751,9 @@ void init_dram_ddr3(spd_raw_data * spds, int mobile, int min_tck,
if (!mrc_cache || mrc_cache->mrc_data_size < sizeof (ctrl)) {
/* Failed S3 resume, reset to come up cleanly */
outb(0x6, 0xcf9);
- hlt();
+ while (1) {
+ hlt();
+ }
}
memcpy(&ctrl, mrc_cache->mrc_data, sizeof (ctrl));
}