[coreboot-gerrit] Change in coreboot[master]: arch/x86: Refactor the SMBIOS type 17 write function

[Subrata Banik (Code Review)]

Attention is currently required from: Jason Glenesk, Marshall Dawson, Subrata Banik, Angel Pons, Rob Barnes, Patrick Rudolph, Felix Held.
Hello build bot (Jenkins), Jason Glenesk, Raul Rangel, Furquan Shaikh, Marshall Dawson, Tim Wawrzynczak, Angel Pons, Rob Barnes, Patrick Rudolph, Felix Held,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/56628

to look at the new patch set (#16).

Change subject: arch/x86: Refactor the SMBIOS type 17 write function
......................................................................

arch/x86: Refactor the SMBIOS type 17 write function

List of changes:
1. Create Module Type macros as per Memory Type
(i.e. DDR2/DDR3/DDR4/DDR5/LPDDR4/LPDDR5) and fix compilation
issue due to renaming of existing macros due to scoping the Memory
Type.
2. Skip fixed module type to form factor conversion using DDR2 SPD4
specification (inside dimm_info_fill()).
3. Use dedicated Memory Type and Module type for `Form Factor`
conversion using `get_spd_info()` function.
4. Use dedicated Memory Type and Module type for `TypeDetail`
conversion using `get_spd_info()` function.
5. Create a new API (convert_form_factor_to_module_type()) for
`Form Factor` to 'Module type' conversion as per `Memory Type`.
6. Add DDR5 and LPDDR5 memory technology into the SMBIOS Memory Type
table.
7. Add new argument as `Memory Type`
smbios_form_factor_to_spd_mod_type().
8. smbios_form_factor_to_spd_mod_type() internally calls
convert_form_factor_to_module_type() for `Module Type` conversion.
9. Update `test_smbios_form_factor_to_spd_mod_type()` to
accommodate different memory types.

Refer to datasheet SPD4.1.2.M-1 for LPDDRx and SPD4.1.2.L-3 for DDRx.

BUG=b:194659789
TEST=Refer to dmidecode -t 17 output as below:
Without this code change:

Handle 0x0012, DMI type 17, 40 bytes
Memory Device
        Array Handle: 0x000A
        Error Information Handle: Not Provided
        Total Width: 16 bits
        Data Width: 16 bits
        Size: 2048 MB
        Form Factor: Unknown
        ....

With this code change:

Handle 0x0012, DMI type 17, 40 bytes
Memory Device
        Array Handle: 0x000A
        Error Information Handle: Not Provided
        Total Width: 16 bits
        Data Width: 16 bits
        Size: 2048 MB
        Form Factor: Row Of Chips
        ....

Change-Id: Ia337ac8f50b61ae78d86a07c7a86aa9c248bad50
Signed-off-by: Subrata Banik 
---
M src/arch/x86/smbios.c
M src/device/dram/ddr3.c
M src/device/dram/ddr4.c
M src/device/dram/spd.c
M src/include/device/dram/spd.h
M src/include/dimm_info_util.h
M src/include/smbios.h
M src/include/spd.h
M src/lib/dimm_info_util.c
M src/mainboard/scaleway/tagada/ramstage.c
M src/northbridge/intel/haswell/haswell_mrc/raminit.c
M src/soc/amd/common/fsp/dmi.c
M src/soc/amd/common/pi/amd_late_init.c
M src/soc/intel/common/smbios.c
M tests/lib/Makefile.inc
M tests/lib/dimm_info_util-test.c
16 files changed, 440 insertions(+), 111 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/56628/16