[coreboot-gerrit] [M] Change in coreboot[main]: soc/intel/pantherlake: Populate and pass DRAM info for SMBIOS

[Subrata Banik (Code Review)]

Attention is currently required from: Paul Menzel.

Subrata Banik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/84903?usp=email )

Change subject: soc/intel/pantherlake: Populate and pass DRAM info for SMBIOS
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/pantherlake/romstage/romstage.c:

https://review.coreboot.org/c/coreboot/+/84903/comment/3f0b3a00_b40c57ca?usp... :
PS1, Line 28: static void save_dimm_info(void)
> This looks very similar to other Intel SOC implementations. Isn’t this only dependent on the FSP version, and could be unified?

yes, I will consider moving this into some common location inside FSP.

marking it resolved till then.