[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Save DIMM info by available nodes

[Jes Klinke (Code Review)]

Jes Klinke has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39359 )

Change subject: soc/intel/tigerlake: Save DIMM info by available nodes
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39359/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39359/1//COMMIT_MSG@9 
PS1, Line 9: TEST= make build image returns correct dimm info by mosys and dmidecode
           :       with TGL-U and TGL-Y + MICA + LPDDR4
> Based on MAX_NODE, MAX_CH and ARRAY_SIZE(mem_info->dimm), 8 entries seem correct. […]

I have used dmidecode to verify that public coreboot with this patch displays the same information in the Memory Device section, as do the private intel repo.

Assuming we want the public coreboot to be "on par" with Intels private repo, in order to stop using the latter, I guess that is sufficient.

I cannot speak to whether the displayed information is correct, nor whether array sizes should be adjusted.  Perhaps we can have a followup CL.