Attention is currently required from: Kevin Chiu, Tim Wawrzynczak, Robert Chen, Nick Vaccaro. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60331 )
Change subject: mb/google/brya/var/vell: Add AMP driver setting ......................................................................
Patch Set 9:
(3 comments)
File src/drivers/i2c/cs35l53/cs35l53.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137140): https://review.coreboot.org/c/coreboot/+/60331/comment/610d407a_16e9d209 PS9, Line 70: if ((config->boost_peak_milliamp > 4500) ||(config->boost_peak_milliamp < 1600) || spaces required around that '||' (ctx:WxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137140): https://review.coreboot.org/c/coreboot/+/60331/comment/13243793_d71e9805 PS9, Line 71: (config->boost_peak_milliamp % 50)){ space required before the open brace '{'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137140): https://review.coreboot.org/c/coreboot/+/60331/comment/e3594c2d_fcf4ff08 PS9, Line 89: acpi_dp_add_integer(dsd, "cirrus,gpio2-output-enable", config->gpio2_output_enable ? 1 : 0); line over 96 characters