[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/i2c: Return negative values on error

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Paul Menzel, Felix Held.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59539 )

Change subject: soc/mediatek/i2c: Return negative values on error
......................................................................

Patch Set 3:

(1 comment)

Patchset:

PS2: 
Done.
> so the proper fix would be to make platform_i2c_transfer return a negative value in those failure cases, so that the same bug can't happen in other drivers

However, note that there are many existing calls to i2c_*() that assume errors when receiving non-zero return values:

if (i2c_writeb(...))
     printk(BIOS_ERR, ...);