[coreboot-gerrit] [M] Change in coreboot[master]: arch/x86: Introduce DUMP_SMBIOS_TYPE17 config

[Kapil Porwal (Code Review)]

Attention is currently required from: Arthur Heymans, Eric Lai, Felix Singer, Paul Menzel, Subrata Banik, Tarun Tuli.

Kapil Porwal has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75756?usp=email )

Change subject: arch/x86: Introduce DUMP_SMBIOS_TYPE17 config
......................................................................

Patch Set 13:

(1 comment)

File src/arch/x86/smbios.c:

https://review.coreboot.org/c/coreboot/+/75756/comment/70e94615_30c581af :
PS9, Line 226: /* skip old type for save static memory
             : 		"Other",
             : 		"Unknown",
             : 		"DRAM",
             : 		"EDRAM",
             : 		"VRAM",
             : 		"SRAM",
             : 		"RAM",
             : 		"ROM",
             : 		"Flash",
             : 		"EEPROM",
             : 		"FEPROM",
             : 		"EPROM",
             : 		"CDRAM",
             : 		"3DRAM",
             : 		"SDRAM",
             : 		"SGRAM",
             : 		"RDRAM",
             : 		"DDR",
             : 		"DDR2",
             : 		"DDR2 FB-DIMM",
             : 		"Reserved",
             : 		"Reserved",
             : 		"Reserved",
             : 		"DDR3",
             : 		"FBD2",
             : 	*/
> Anyone else have idea here?

I think we should have all the known types included. Even this `"below MEMORY_TYPE_DDR4 or exceed MEMORY_TYPE_HBM3"` string is occupying some static area.