[coreboot-gerrit] Change in coreboot[master]: Revert "soc/intel/cannonlake: Enable ACPI timer emulation if PM timer...

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45000 )

Change subject: Revert "soc/intel/cannonlake: Enable ACPI timer emulation if PM timer is disabled"
......................................................................

Patch Set 6: Code-Review+1

(1 comment)

Hey folks, does anyone have an opinion on this? I'd prefer to
get it in, simply because it avoids complexity. I don't think,
but can't prove, that it's really harmless to have both the
emulation and the actual timer available.

https://review.coreboot.org/c/coreboot/+/45000/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/45000/1//COMMIT_MSG@11 
PS1, Line 11: BIOS spec says, emulation shall be always enabled.
> thanks, reworded this

It's not unambiguous... Could also mean that this particular order is
required if it's going to be enabled.