Meera Ravindranath has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36034 )
Change subject: soc/intel/common/block/timer: Add config to enable/disable PM timer support ......................................................................
Patch Set 6:
(4 comments)
https://review.coreboot.org/c/coreboot/+/36034/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36034/4//COMMIT_MSG@9 PS4, Line 9: Added
Add
Done
https://review.coreboot.org/c/coreboot/+/36034/4//COMMIT_MSG@10 PS4, Line 10:
Why is such an option needed?
This Kconfig would be further used to support TianoCore boot which requires PM timer to be enabled.
https://review.coreboot.org/c/coreboot/+/36034/4/src/soc/intel/common/block/... File src/soc/intel/common/block/timer/Kconfig:
https://review.coreboot.org/c/coreboot/+/36034/4/src/soc/intel/common/block/... PS4, Line 19: ACPI PM Timer
But it says *Use ACPI PM Timer* in the description?
Ack
https://review.coreboot.org/c/coreboot/+/36034/4/src/soc/intel/common/block/... PS4, Line 19: disables the ACPI PM Timer by default.
Allows a provision to enable or disable ACPI PM timer?
Ack