build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59737 )
Change subject: mb/google/brya/variants/primus: Swap TPM I2C with touchscreen I2C ......................................................................
Patch Set 3:
(3 comments)
File src/mainboard/google/brya/variants/primus/variant.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134705): https://review.coreboot.org/c/coreboot/+/59737/comment/4d74a9b7_ec80cc90 PS3, Line 43: if (board_ver <2) { spaces required around that '<' (ctx:WxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134705): https://review.coreboot.org/c/coreboot/+/59737/comment/07ae468a_f3578883 PS3, Line 52: else { else should follow close brace '}'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-134705): https://review.coreboot.org/c/coreboot/+/59737/comment/a9162207_2db25fbc PS3, Line 113: if (board_ver >1) { spaces required around that '>' (ctx:WxV)