Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33512 )
Change subject: soc/intel/cannonlake: fix use of legacy 8254 timer ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/#/c/33512/7/src/soc/intel/cannonlake/Kconfig File src/soc/intel/cannonlake/Kconfig:
https://review.coreboot.org/#/c/33512/7/src/soc/intel/cannonlake/Kconfig@319 PS7, Line 319: bool please add a string so it's exposed to users (and can be changed through .config files). Not everybody uses the integrated payload mechanism.