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

[Arthur Heymans (Code Review)]

Arthur Heymans 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: Code-Review+2

(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;
             : 			}
So a DIMM in only channel 1 would never boot? Same for different DIMM's? I'd print some warning about that.