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

[Felix Held (Code Review)]

Attention is currently required from: Paul Menzel, Yu-Ping Wu.
Felix Held 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: Code-Review+2

(2 comments)

File src/soc/mediatek/common/i2c.c:

https://review.coreboot.org/c/coreboot/+/59539/comment/6b1ddf55_1020c219 
PS3, Line 282: 	return ret;
> Those enum values have been changed to negative in this patch. […]

oh, sorry, somehow missed that part of the updated patch. yes, then this is correct

https://review.coreboot.org/c/coreboot/+/59539/comment/add13139_cf574894 
PS3, Line 317: 		if (ret < 0)
> since mtk_i2c_transfer will return 0 or positive, this condition will always be false. […]

missed the part of the patch that made the defined constants negative numbers, so this is correct