Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/21194 )
Change subject: drivers/i2c/lm96000: Add new hardware-monitoring IC ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/#/c/21194/7/src/drivers/i2c/lm96000/lm96000.c File src/drivers/i2c/lm96000/lm96000.c:
https://review.coreboot.org/#/c/21194/7/src/drivers/i2c/lm96000/lm96000.c@18... PS7, Line 184: timeout = 1000; : lm_config = lm96000_read(dev, LM96000_CONFIG); : while ((lm_config < 0 || !(lm_config & LM96000_READY)) && timeout--) { : mdelay(1); : lm_config = lm96000_read(dev, LM96000_CONFIG); : } : if (lm_config < 0 || !(lm_config & LM96000_READY)) { : printk(BIOS_INFO, "lm96000: Not ready after 1s.\n"); : return; : } Use the stopwatch infrastructure?