[coreboot-gerrit] Change in coreboot[master]: include/memory_info.h: Change part number field from 19 bytes to 21

Richard Spiegel (Code Review) gerrit at coreboot.org
Fri Apr 13 01:18:02 CEST 2018


Richard Spiegel has uploaded this change for review. ( https://review.coreboot.org/25650


Change subject: include/memory_info.h: Change part number field from 19 bytes to 21
......................................................................

include/memory_info.h: Change part number field from 19 bytes to 21

DDR4 reserves 20 bytes for part number without termination. Therefor
coreboot buffer for part number has to be increased to 21 bytes, to
account termination.

BUG=b:77943312
TEST=Build kahlee.

Change-Id: I57aa8e06d907a1e6b11ce88f612a0d6c14b7c65b
Signed-off-by: Richard Spiegel <richard.spiegel at silverbackltd.com>
---
M src/include/memory_info.h
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/25650/1

diff --git a/src/include/memory_info.h b/src/include/memory_info.h
index d26d789..c9554b8 100644
--- a/src/include/memory_info.h
+++ b/src/include/memory_info.h
@@ -20,7 +20,7 @@
 #include <compiler.h>
 
 #define DIMM_INFO_SERIAL_SIZE		4
-#define DIMM_INFO_PART_NUMBER_SIZE	19
+#define DIMM_INFO_PART_NUMBER_SIZE	21
 #define DIMM_INFO_TOTAL			8	/* Maximum num of dimm is 8 */
 
 /**

-- 
To view, visit https://review.coreboot.org/25650
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: I57aa8e06d907a1e6b11ce88f612a0d6c14b7c65b
Gerrit-Change-Number: 25650
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Spiegel <richard.spiegel at silverbackltd.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180412/8b3351b4/attachment.html>


More information about the coreboot-gerrit mailing list