[coreboot-gerrit] Change in coreboot[master]: device/software_i2c: Add function to recover bus

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35725 )

Change subject: device/software_i2c: Add function to recover bus
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35725/1/src/device/software_i2c.c 
File src/device/software_i2c.c:

https://review.coreboot.org/c/coreboot/+/35725/1/src/device/software_i2c.c@3... 
PS1, Line 373: 	while (i++ < RECOVERY_CLK_CNT * 2) {
> True... […]

sending random start conditions might also make some devices unhappy; what the kernel does here seems to be the best option that is least likely to cause some weird problems. if i read the code you suggested correctly (only had a brief look at it and the functions it calls), it should do the trick