Attention is currently required from: Tim Wawrzynczak, Werner Zeh, Kyösti Mälkki. Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55088 )
Change subject: drivers/i2c/designware: Report I2C timings for additional bus speeds ......................................................................
Patch Set 2:
(1 comment)
File src/drivers/i2c/designware/dw_i2c.c:
https://review.coreboot.org/c/coreboot/+/55088/comment/e8d3b796_a9dfb7f0 PS2, Line 834: Now check if there are more speed settings available and report them as well. Would it be bad to expose information about all different I2C speeds to the OS? We can skip the ones where it is not possible to calculate sane values i.e. revert CB:34385. The problem that CB:34385 was trying to address is "dw_i2c: bad counts" message. We can either lower the severity of the print or drop it completely: https://review.coreboot.org/plugins/gitiles/coreboot/+/refs/heads/master/src...