[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Select USE_LEGACY_8254_TIMER

[Frans Hendriks (Code Review)]

Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34709 )

Change subject: soc/intel/common: Select USE_LEGACY_8254_TIMER
......................................................................

Patch Set 2: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/34709/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34709/2//COMMIT_MSG@9 
PS2, Line 9: Always select USE_LEGACY_8254_TIMER if we know we have to.
> It also selects USE_LEGACY_8254_TIMER in specific cases to prevent a hang.

Done

https://review.coreboot.org/c/coreboot/+/34709/2/src/device/Kconfig 
File src/device/Kconfig:

https://review.coreboot.org/c/coreboot/+/34709/2/src/device/Kconfig@89 
PS2, Line 89: 	select USE_LEGACY_8254_TIMER if SOC_INTEL_COMMON_BLOCK
> It should be enabled for all Oproms, but we only support VGA Oproms in coreboot.

Done