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 2:

(1 comment)

Patchset:

PS2:
> regardless if a new function is used, the new pointer in the ops struct is still needed, no?

For at least the DW i2c driver, a 0-byte transfer just immediately returns -1, https://review.coreboot.org/c/coreboot/+/63210/2/src/drivers/i2c/designware/..., and the transfer function would need a few tweaks.... but maybe that can be accommodated, let me hack something up.

Matt, then the i2c/generic driver code wouldn't call detect(), it would call transfer() but passing 0 for the size, it's not quite as descriptive that way, but I think that's what Julius means.