[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: add support to read spd data from SMBUS

[Aamir Bohra (Code Review)]

Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39401 )

Change subject: soc/intel/tigerlake: add support to read spd data from SMBUS
......................................................................

Patch Set 5:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/39401/5/src/soc/intel/tigerlake/mem... 
PS5, Line 118: meminit_channels
you would need dqs map and rcomp setting even for the SMBUS case right?
In that case spd_data_ptr needs to be set to zero and meminit_channels needs to be called outside the condition check , common to both