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

[Subrata Banik (Code Review)]

Subrata Banik 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 3:

(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 and "y" in case of PAYLOAD_SEABIOS to know only disable for seabios payload to boot ?