Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38606 )
Change subject: soc/intel/tigerlake: add memory configuration support ......................................................................
Patch Set 8:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38606/8/src/soc/intel/tigerlake/mem... File src/soc/intel/tigerlake/meminit_tgl.c:
https://review.coreboot.org/c/coreboot/+/38606/8/src/soc/intel/tigerlake/mem... PS8, Line 153: board_cfg->half_populated, Why is this required if board_cfg is already passed into meminit_channels_dimm0()?