From febbe519b1fc754554b6d94cf4e7b17548c32fe9 Mon Sep 17 00:00:00 2001 From: Evan Lojewski Date: Tue, 29 Jan 2019 09:18:53 -0600 Subject: Update to allow DDR4-SORDIMM modules to be used [3]. Add support for the module type in the SPD parser. This change has been tested with the VR9FR2G7228JBKSBD4 module from Viking Technologies: 2666MHz DDR4-SORDIMM, rank 2, 16GB Note- Above mentioned module need an additional patch due to an unexpected SPD manufacturer being detected. Change-Id: Iaf8d78579381c48cbe6b735d02dd624855d0e57e Signed-off-by: Evan Lojewski Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71055 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Reviewed-by: Christian R. Geddes --- src/import/generic/memory/lib/spd/spd_checker.H | 3 +++ src/import/generic/memory/lib/spd/spd_factory_pattern.C | 3 +++ 2 files changed, 6 insertions(+) (limited to 'src/import/generic/memory/lib/spd') diff --git a/src/import/generic/memory/lib/spd/spd_checker.H b/src/import/generic/memory/lib/spd/spd_checker.H index cb4eb371a..0b2a33076 100644 --- a/src/import/generic/memory/lib/spd/spd_checker.H +++ b/src/import/generic/memory/lib/spd/spd_checker.H @@ -6,6 +6,7 @@ /* OpenPOWER HostBoot Project */ /* */ /* Contributors Listed Below - COPYRIGHT 2017 */ +/* [+] Evan Lojewski */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -186,6 +187,8 @@ static inline bool is_dimm_type_valid(const uint8_t i_dimm_type) case RDIMM: case LRDIMM: case DDIMM: + case SORDIMM: + case MINIRDIMM: l_result = true; break; diff --git a/src/import/generic/memory/lib/spd/spd_factory_pattern.C b/src/import/generic/memory/lib/spd/spd_factory_pattern.C index cb934265d..3127af5a7 100644 --- a/src/import/generic/memory/lib/spd/spd_factory_pattern.C +++ b/src/import/generic/memory/lib/spd/spd_factory_pattern.C @@ -6,6 +6,7 @@ /* OpenPOWER HostBoot Project */ /* */ /* Contributors Listed Below - COPYRIGHT 2018 */ +/* [+] Evan Lojewski */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -295,6 +296,8 @@ fapi2::ReturnCode factories::dimm_module_select_param(parameters& o_param) const switch(iv_dimm_type) { case RDIMM: + case SORDIMM: + case MINIRDIMM: o_param = RDIMM_MODULE; break; -- cgit v1.2.1