[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Support to initialize Memory

[Varun Joshi (Code Review)]

Varun Joshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39847 )

Change subject: soc/intel/tigerlake: Support to initialize Memory
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39847/5/src/soc/intel/tigerlake/mem... 
File src/soc/intel/tigerlake/meminit_tgl.c:

https://review.coreboot.org/c/coreboot/+/39847/5/src/soc/intel/tigerlake/mem... 
PS5, Line 179: meminit_ddr4x_sodimm
> Here you go: https://review.coreboot.org/c/coreboot/+/39865 […]

I am working on this, I saw you defined DIMMS_PER_CHANNEL to 2, shouldn't it be 1 for deltan?