Attention is currently required from: Eric Lai, Paul Menzel, Tarun Tuli.
Arthur Heymans 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 9:
(2 comments)
File src/arch/x86/Kconfig:
https://review.coreboot.org/c/coreboot/+/75756/comment/e81c0681_4eea0069 : PS9, Line 362: choice : prompt "Dump SMBIOS type17" : default n : depends on GENERATE_SMBIOS_TABLES drop the choice.
File src/arch/x86/smbios.c:
https://review.coreboot.org/c/coreboot/+/75756/comment/86d64af5_8dbcc560 : 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", : */ Is printing SMBIOS information for older DRAM specs not desirable? It will print unsupported type, which is incorrect.