Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/19981 )
Change subject: google/slippy: populate PEI SPD data for all channels ......................................................................
Patch Set 1:
not directly, no. On BDW, mrc.bin populates the pei_data struct which is used to populate a CBMEM_INFO table and ultimately the SMBIOS table. Os HSW, the mrc.bin doesn't do this, so I've added a patch to populate the data: #19958. Even still, this doesn't eliminate the need for the spd_data struct in the pei_data to be populated for all/both sodimms/channels, which is what this patch does (basically bringing it in line with BDW)