Attention is currently required from: Tim Wawrzynczak, Zhuohao Lee. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59173 )
Change subject: mb/google/brya/variants/primus: enable ALC5682I-VS ......................................................................
Patch Set 3:
(4 comments)
File src/mainboard/google/brya/variants/primus/variant.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132939): https://review.coreboot.org/c/coreboot/+/59173/comment/bf3a5107_c5103d94 PS3, Line 40: trailing whitespace
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132939): https://review.coreboot.org/c/coreboot/+/59173/comment/83d4b44c_3177ba5f PS3, Line 40: please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132939): https://review.coreboot.org/c/coreboot/+/59173/comment/f86c6152_d3f3cd85 PS3, Line 51: alc_dev=child; spaces required around that '=' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-132939): https://review.coreboot.org/c/coreboot/+/59173/comment/6878f7cc_571a1e56 PS3, Line 55: alc_dev->enabled=1; spaces required around that '=' (ctx:VxV)