Attention is currently required from: Jeremy Soller, Tim Wawrzynczak, Angel Pons, Patrick Rudolph. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56256 )
Change subject: src/soc/intel/*: Change legacy_8254_timer to CMOS option ......................................................................
Patch Set 1:
(3 comments)
File src/soc/intel/cannonlake/fsp_params.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123943): https://review.coreboot.org/c/coreboot/+/56256/comment/fab48083_b853557c PS1, Line 431: params->Enable8254ClockGating = !legacy_8254_timer;; Statements terminations use 1 semicolon
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123943): https://review.coreboot.org/c/coreboot/+/56256/comment/2f3ac838_0caac21c PS1, Line 432: params->Enable8254ClockGatingOnS3 = !legacy_8254_timer;; Statements terminations use 1 semicolon
File src/soc/intel/jasperlake/fsp_params.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-123943): https://review.coreboot.org/c/coreboot/+/56256/comment/8c824125_8cf306ea PS1, Line 91: params->Enable8254ClockGating = !legacy_8254_timer;; Statements terminations use 1 semicolon