Attention is currently required from: Tim Wawrzynczak.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68665 )
Change subject: mb/google/brask/var/kuldax: Revise PsysPL2 to 150W for Pentium CPU ......................................................................
Patch Set 1:
(3 comments)
File src/mainboard/google/brya/variants/kuldax/ramstage.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160963): https://review.coreboot.org/c/coreboot/+/68665/comment/0ccaf56f_5cf630b3 PS1, Line 69: if (fw_config_probe(FW_CONFIG(BJ_POWER, BJ_POWER_65W))) { braces {} are not necessary for any arm of this statement
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160963): https://review.coreboot.org/c/coreboot/+/68665/comment/e6c57685_00767fb7 PS1, Line 70: variant_update_psys_power_limits(limits, sys_limits, total_entries, &psys_config); line length of 98 exceeds 96 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160963): https://review.coreboot.org/c/coreboot/+/68665/comment/c4eae086_fcb2387d PS1, Line 72: variant_update_psys_power_limits(limits, revise_sys_limits, total_entries, &psys_config); line length of 105 exceeds 96 columns