Change in coreboot[master]: soc/intel/fsp_broadwell_de: Populate SMBIOS tables with memory inform...
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34681 )
Change subject: soc/intel/fsp_broadwell_de: Populate SMBIOS tables with memory information ......................................................................
Patch Set 4:
(2 comments)
@Werner: Any objectives against always providing SMBIOS type17 at cost of slightly increased boot time from your side?
https://review.coreboot.org/c/coreboot/+/34681/4/src/soc/intel/fsp_broadwell... File src/soc/intel/fsp_broadwell_de/Kconfig:
https://review.coreboot.org/c/coreboot/+/34681/4/src/soc/intel/fsp_broadwell... PS4, Line 96: config SAVE_DRAM_INFO I don't see why it should be board specific.
https://review.coreboot.org/c/coreboot/+/34681/4/src/soc/intel/fsp_broadwell... File src/soc/intel/fsp_broadwell_de/romstage/romstage.c:
https://review.coreboot.org/c/coreboot/+/34681/4/src/soc/intel/fsp_broadwell... PS4, Line 124: if (CONFIG(SAVE_DRAM_INFO)) !CONFIG(FSP_MEMORY_DOWN)?
-
Patrick Rudolph (Code Review)