Attention is currently required from: Patrick Rudolph, Christian Walter.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/66616 )
Change subject: mb/prodrive/hermes: Prevent SGPIO cross-powering 5V rail ......................................................................
Patch Set 4:
(7 comments)
Commit Message:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/39adbf00_070aa503 PS4, Line 8: Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/4c738d85_2da5b2ce PS4, Line 9: The PCH's SGPIO pads are connected to a buffer chip which is powered from Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/a3a8d734_3f36742d PS4, Line 10: the always-on +3V3_AUX rail. For some reason, if the SGPIO pads remain in Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/9f177cba_58deeb38 PS4, Line 11: their native function when a Poseidon system shuts down, a bit of voltage Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/6da7148b_aedbe6b4 PS4, Line 12: from the +3V3_AUX-powered buffer chip will leak into the +5V rail through Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/2f98d534_6b93ae1a PS4, Line 15: Possible unwrapped commit description (prefer a maximum 72 chars per line)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-159867): https://review.coreboot.org/c/coreboot/+/66616/comment/481bdc1d_1f84509e PS4, Line 16: This issue has only been observed in S5, but it is very likely other sleep Possible unwrapped commit description (prefer a maximum 72 chars per line)