From 4b6f262eadeddfd1f333f7ce05b2a46a5f39b8bf Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Tue, 9 Oct 2018 07:31:24 -0600 Subject: [PATCH] lib: increase part number size in memory_info/dimm_info The dimm_info and memory_info structs are used in devices that don't strictly use DIMMs. Those platforms have the DRAM soldered down so there's no DIMM part number that encapsulates the DRAMs used in the DIMM. The full part number is desired to be exposed in the SMBIOS tables. As such extend DIMM_INFO_PART_NUMBER_SIZE to 33 to accommodate longer part numbers 'MT53B256M32D1NP-053 WT:C' is one of the longer part numbers that are desired to be maintained. BUG=b:115697578 Change-Id: I0c39dd1d1c2f0776d70d8c4d8d56719779ff82ae Signed-off-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/28978 Tested-by: build bot (Jenkins) Reviewed-by: Jett Rink Reviewed-by: Furquan Shaikh --- src/include/memory_info.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/include/memory_info.h b/src/include/memory_info.h index 1b30702c33..5976e319f8 100644 --- a/src/include/memory_info.h +++ b/src/include/memory_info.h @@ -19,7 +19,7 @@ #include #define DIMM_INFO_SERIAL_SIZE 4 -#define DIMM_INFO_PART_NUMBER_SIZE 21 +#define DIMM_INFO_PART_NUMBER_SIZE 33 #define DIMM_INFO_TOTAL 8 /* Maximum num of dimm is 8 */ /**