[coreboot-gerrit] Change in coreboot[master]: soc/intel/{skl,cnl,dnv,xeon_sp}: disable PM ACPI timer if chosen

[Nico Huber (Code Review)]

Attention is currently required from:  Felix Singer, Furquan Shaikh, Mariusz Szafrański, Jonathan Zhang, Paul Menzel, Subrata Banik, Arthur Heymans, Michael Niewöhner, Patrick Rudolph, Lance Zhao, Anjaneya "Reddy" Chagam, Johnny Lin, Tim Wawrzynczak, Christian Walter, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Tim Chu.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57932 )

Change subject: soc/intel/{skl,cnl,dnv,xeon_sp}: disable PM ACPI timer if chosen
......................................................................

Patch Set 33:

(1 comment)

File src/soc/intel/xeon_sp/pmc.c:

https://review.coreboot.org/c/coreboot/+/57932/comment/99fff39b_68fa5aa3 
PS33, Line 60: Disabling ACPI PM timer also switches off TCO.
> FSP will disable ACPI PM timer if TCOtimer upd is set to disabled right? , that's not a hardware beh […]

AIUI, it's hardware behavior, one timer depends on the other. If it
wasn't, why should it be covered by a single UPD?