Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45873 )
Change subject: mb, soc: change mainboard_get_dram_part_num() prototype ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45873/3/src/soc/intel/alderlake/rom... File src/soc/intel/alderlake/romstage/romstage.c:
https://review.coreboot.org/c/coreboot/+/45873/3/src/soc/intel/alderlake/rom... PS3, Line 92: part_name_overridden
Done
JFYI, I realize now I was looking at the logic at https://source.corp.google.com/chromeos_public/src/third_party/coreboot/src/... when I made my statement. I do see now that not all implementations did the same (respected each dimm's name). We should probably clean this up afterwards.