Attention is currently required from: Piotr Król.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67940 )
Change subject: mb/protectli/vault_cml: Add Comet Lake 6 port board support ......................................................................
Patch Set 7:
(3 comments)
File src/mainboard/protectli/vault_cml/devicetree.cb:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159604): https://review.coreboot.org/c/coreboot/+/67940/comment/93b95b0f_866f78e0 PS7, Line 58: # Enable SERIRQ continous 'continous' may be misspelled - perhaps 'continuous'?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159604): https://review.coreboot.org/c/coreboot/+/67940/comment/da83ed13_1dbbfcad PS7, Line 269: device pnp 2e.4 on # Enviroment Controller 'Enviroment' may be misspelled - perhaps 'Environment'?
File src/mainboard/protectli/vault_cml/mainboard.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159604): https://review.coreboot.org/c/coreboot/+/67940/comment/716b7438_9209ada6 PS7, Line 64: for (i = 0; i < 10000; i++) inb(0x61); trailing statements should be on next line