[coreboot-gerrit] Change in coreboot[master]: device/i2c_bus: Add detect function pointer in i2c_bus_ops

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Subrata Banik, Matt DeVillier, Julius Werner, Karthik Ramasubramanian.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63209 )

Change subject: device/i2c_bus: Add detect function pointer in i2c_bus_ops
......................................................................

Patch Set 3:

(1 comment)

File src/include/device/i2c_bus.h:

https://review.coreboot.org/c/coreboot/+/63209/comment/4f39412e_9eba11a1 
PS3, Line 14: bool (*detect)(struct device *dev, unsigned int addr);
> I meant we shouldn't have a separate detect() function pointer here. […]

I think we're roughly saying the same thing here; remove the detect callback pointer, and use transfer instead 👍 We could also add similar functionality to i2c simple for compatibility