[coreboot-gerrit] Change in coreboot[master]: sandybridge/raminit_common.c: fix printram statement
Iru Cai (Code Review)
gerrit at coreboot.org
Thu Aug 16 10:43:00 CEST 2018
Iru Cai has uploaded this change for review. ( https://review.coreboot.org/28117
Change subject: sandybridge/raminit_common.c: fix printram statement
......................................................................
sandybridge/raminit_common.c: fix printram statement
Change-Id: Iddea8cc71dc1fb33d46b22dd19e39bf1c1257555
Signed-off-by: Iru Cai <mytbk920423 at gmail.com>
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/17/28117/1
diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index d806066..6944542 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -207,12 +207,15 @@
stretch = 3;
MCHBAR32_AND_OR(0x401c + 0x400 * channel, 0xffffc3ff,
(stretch << 12) | (stretch << 10));
- printram("OTHP Workaround [%x] = %x\n", addr, reg);
+ printram("OTHP Workaround [%x] = %x\n",
+ 0x401c + 0x400 * channel,
+ MCHBAR32(0x401c + 0x400 * channel));
} else {
// OTHP
MCHBAR32_AND_OR(0x400c + 0x400 * channel, 0xfff0ffff,
(stretch << 16) | (stretch << 18));
- printram("OTHP [%x] = %x\n", addr, reg);
+ printram("OTHP [%x] = %x\n", 0x400c + 0x400 * channel
+ , MCHBAR32(0x400c + 0x400 * channel));
}
}
--
To view, visit https://review.coreboot.org/28117
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: Iddea8cc71dc1fb33d46b22dd19e39bf1c1257555
Gerrit-Change-Number: 28117
Gerrit-PatchSet: 1
Gerrit-Owner: Iru Cai <mytbk920423 at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180816/e26f48b7/attachment-0001.html>
More information about the coreboot-gerrit
mailing list