[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: set use of legacy 8254 timer

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33512 )

Change subject: soc/intel/cannonlake: set use of legacy 8254 timer
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/33512/3/src/soc/intel/cannonlake/Kconfig 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/#/c/33512/3/src/soc/intel/cannonlake/Kconfig@320 
PS3, Line 320: 	default n if PAYLOAD_SEABIOS
             : 	default y
> is this correct? as per your commit msg, i guess default should be "n" that mean 8254 is not disable […]

nope, reversed the meaning of the Kconfig but not the logic. fixed in #4