Mario Scheithauer has posted comments on this change. ( https://review.coreboot.org/25585 )
Change subject: siemens/mc_apl1: Fix for using IDT PMIC ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/25585/1/src/mainboard/siemens/mc_apl1/mainbo... File src/mainboard/siemens/mc_apl1/mainboard.c:
https://review.coreboot.org/#/c/25585/1/src/mainboard/siemens/mc_apl1/mainbo... PS1, Line 146: printk(BIOS_ERR, "PMIC: Mailbox interface busy!\n");
is this delay required here? if not, you might be better off by using the stopwatch API from src/inc […]
Hello Patrick, thanks for the review! I'm using stopwatch API now, not quite as you suggested, but that should also work.