[coreboot-gerrit] Change in coreboot[master]: nb/intel/x4x: Fix computing page_size
Arthur Heymans (Code Review)
gerrit at coreboot.org
Wed Dec 27 00:21:35 CET 2017
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/22991
Change subject: nb/intel/x4x: Fix computing page_size
......................................................................
nb/intel/x4x: Fix computing page_size
This problem was introduced by 3cf94032b "nb/x4x/raminit: Rewrite SPD
decode and timing selection", but was probably not encountered because
such dimms are rather uncommon.
Change-Id: I2d57f5e584ac7fa1479791c239432005fe8c178d
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/intel/x4x/raminit.c
1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/22991/1
diff --git a/src/northbridge/intel/x4x/raminit.c b/src/northbridge/intel/x4x/raminit.c
index 0b4d4a5..3cd75be 100644
--- a/src/northbridge/intel/x4x/raminit.c
+++ b/src/northbridge/intel/x4x/raminit.c
@@ -188,9 +188,10 @@
* 1KB page size. For the x16 configuration, the page size is 2KB
* for all densities except the 256Mb device, which has a 1KB page
* size." Micron, 'TN-47-16 Designing for High-Density DDR2 Memory'
+ * The formula is pagesize in KiB = width * 2^col_bits / 8.
*/
- s->dimms[dimm_idx].page_size = s->dimms[dimm_idx].width *
- (1 << decoded_dimm.col_bits);
+ s->dimms[dimm_idx].page_size = decoded_dimm.width *
+ (1 << decoded_dimm.col_bits) / 8;
switch (decoded_dimm.banks) {
case 4:
--
To view, visit https://review.coreboot.org/22991
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: I2d57f5e584ac7fa1479791c239432005fe8c178d
Gerrit-Change-Number: 22991
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/20171226/2a455d5b/attachment-0001.html>
More information about the coreboot-gerrit
mailing list