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

[Ronak Kanabar (Code Review)]

Ronak Kanabar 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:

(4 comments)

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

https://review.coreboot.org/c/coreboot/+/39401/1/src/soc/intel/tigerlake/mem... 
PS1, Line 50: memset(&mem_cfg->SpdAddressTable, 0, sizeof(mem_cfg->SpdAddressTable));
> I will do change so get_spd_data() will no change.

Done

https://review.coreboot.org/c/coreboot/+/39401/1/src/soc/intel/tigerlake/mem... 
PS1, Line 120: spd_data_ptr
> I will do change so get_spd_data() will no change.

Done

https://review.coreboot.org/c/coreboot/+/39401/1/src/soc/intel/tigerlake/mem... 
PS1, Line 122: get_spd_data
> go it

Done

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

https://review.coreboot.org/c/coreboot/+/39401/3/src/soc/intel/tigerlake/mem... 
PS3, Line 122: get_spd_data
> got your point will change this.

Done