Change in coreboot[master]: soc/fsp_broadwell_de: Implement SMBus read/write over IMC

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34679 )

Change subject: soc/fsp_broadwell_de: Implement SMBus read/write over IMC
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34679/4/src/soc/intel/fsp_broadwell... 
File src/soc/intel/fsp_broadwell_de/smbus-imc.c:

https://review.coreboot.org/c/coreboot/+/34679/4/src/soc/intel/fsp_broadwell... 
PS4, Line 24: return 0
aren't we returning the word data on success, 0 on failure?

https://review.coreboot.org/c/coreboot/+/34679/4/src/soc/intel/fsp_broadwell... 
PS4, Line 36: 
            : /* read byte, return 0 on success */
aren't we returning the byte data on success, 0 on failure?