Change in coreboot[master]: soc/intel/common: add Kconfig to enable/disable the ACPI PM timer
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45952 )
Change subject: soc/intel/common: add Kconfig to enable/disable the ACPI PM timer ......................................................................
Patch Set 20:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45952/17/src/soc/intel/common/block... File src/soc/intel/common/block/pmc/Kconfig:
https://review.coreboot.org/c/coreboot/+/45952/17/src/soc/intel/common/block... PS17, Line 45: TCO
Sorry for overlooking this. […]
"This timer is required for ACPI and for total cost of ownership (TCO), [...]". IMO, that would hurt, badly. Explaining why the TCO block is named TCO is really out of scope, and without explanation it's more than confusing.
-
Nico Huber (Code Review)