[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/254d6e22_5064199c 
PS3, Line 14: bool (*detect)(struct device *dev, unsigned int addr);
I think what Julius might have been getting at is should we just use the i2c_simple interface instead? e.g.

```
static inline int i2c_write_detect(unsigned int bus, uint8_t addr)
{
        struct i2c_msg seg;

seg.flags = 0;
        seg.slave = addr;
        seg.buf   = NULL;
        seg.len   = 0;

return i2c_transfer(bus, &seg, 1);
}

```