Attention is currently required from: Hung-Te Lin, Yu-Ping Wu. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59245 )
Change subject: soc/mediatek/mt8186: Add support for PMIC MT6366 ......................................................................
Patch Set 2:
(2 comments)
File src/soc/mediatek/mt8186/mt6366.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133113): https://review.coreboot.org/c/coreboot/+/59245/comment/b7c2cf85_764ce811 PS2, Line 708: while (!stopwatch_expired(&voltage_settled)) { suspect code indent for conditional statements (8, 49)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-133113): https://review.coreboot.org/c/coreboot/+/59245/comment/5858c019_fb0e3370 PS2, Line 708: while (!stopwatch_expired(&voltage_settled)) { braces {} are not necessary for single statement blocks