device/dram: Update RDIMM classification from RIMM to DIMM
Registered DIMM should be 'FORMFACTOR_DIMM' with 'DETAIL_REGISTERED'
instead of 'FORMFACTOR_RIMM', RIMM has been EOL for so many years.
Memory form factor info is now correct on 4th Gen Xeon server platform
with registered DIMM.
Signed-off-by: Ziang Wang <ziang.wang@intel.com>
Signed-off-by: Kehong Chen <kehong.chen@intel.com>
Change-Id: I1eea4717a2d60c6100c262a2284a2ac5109f114a
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75489
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
diff --git a/src/device/dram/spd.c b/src/device/dram/spd.c
index 564a4ea..4a1e882 100644
--- a/src/device/dram/spd.c
+++ b/src/device/dram/spd.c
@@ -57,7 +57,7 @@
switch (module_type) {
case SPD_DDR2_DIMM_TYPE_RDIMM:
case SPD_DDR2_DIMM_TYPE_MINI_RDIMM:
- info->form_factor = MEMORY_FORMFACTOR_RIMM;
+ info->form_factor = MEMORY_FORMFACTOR_DIMM;
info->type_detail = MEMORY_TYPE_DETAIL_REGISTERED;
break;
case SPD_DDR2_DIMM_TYPE_UDIMM:
@@ -85,7 +85,7 @@
switch (module_type) {
case SPD_DDR3_DIMM_TYPE_RDIMM:
case SPD_DDR3_DIMM_TYPE_MINI_RDIMM:
- info->form_factor = MEMORY_FORMFACTOR_RIMM;
+ info->form_factor = MEMORY_FORMFACTOR_DIMM;
info->type_detail = MEMORY_TYPE_DETAIL_REGISTERED;
break;
case SPD_DDR3_DIMM_TYPE_UDIMM:
@@ -114,7 +114,7 @@
switch (module_type) {
case DDR4_SPD_RDIMM:
case DDR4_SPD_MINI_RDIMM:
- info->form_factor = MEMORY_FORMFACTOR_RIMM;
+ info->form_factor = MEMORY_FORMFACTOR_DIMM;
info->type_detail = MEMORY_TYPE_DETAIL_REGISTERED;
break;
case DDR4_SPD_UDIMM:
@@ -139,7 +139,7 @@
switch (module_type) {
case DDR5_SPD_RDIMM:
case DDR5_SPD_MINI_RDIMM:
- info->form_factor = MEMORY_FORMFACTOR_RIMM;
+ info->form_factor = MEMORY_FORMFACTOR_DIMM;
info->type_detail = MEMORY_TYPE_DETAIL_REGISTERED;
break;
case DDR5_SPD_UDIMM: