[coreboot-gerrit] Change in coreboot[master]: soc/intel/baytrail: Add MRC SMBus workaround

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44092 )

Change subject: soc/intel/baytrail: Add MRC SMBus workaround
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44092/4/src/soc/intel/baytrail/roms... 
File src/soc/intel/baytrail/romstage/raminit.c:

https://review.coreboot.org/c/coreboot/+/44092/4/src/soc/intel/baytrail/roms... 
PS4, Line 170: if (mp->mainboard.spd_addrs[i]) {
             : 				i2c_eeprom_read(mp->mainboard.spd_addrs[i],
             : 					0, SPD_SIZE, spd_buf[i]);
             : 				/* NOTE: the MRC expects both SPD pointers
             : 					to match */
             : 				mp->mainboard.dram_data[i] = spd_buf;
             : 			}
> Actually, the data buffers can be different, but the data for channel 0 needs to be at index 0, and  […]

Reading it again, I'd reword the comment:

Note: MRC looks for Channel 1 SPD at array index 1