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

[Michael Niewöhner (Code Review)]

Attention is currently required from: Arthur Heymans,  Felix Singer, Furquan Shaikh, Mariusz Szafrański, Jonathan Zhang, Paul Menzel, Johnny Li, Subrata Banik, Patrick Rudolph, Lance Zhao, Marc Jones, Nico Huber, Anjaneya "Reddy" Chagam, Johnny Lin, Tim Wawrzynczak, Christian Walter, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Tim Chu.
Hello Arthur Heymans,  Felix Singer, build bot (Jenkins), Furquan Shaikh, Mariusz Szafrański, Jonathan Zhang, Paul Menzel, Johnny Li, Subrata Banik, Arthur Heymans, Patrick Rudolph, Lance Zhao, Marc Jones, Anjaneya "Reddy" Chagam, Nico Huber, Johnny Lin, Christian Walter, Suresh Bellampalli, Tim Wawrzynczak, Vanessa Eusebio, Michal Motyl, Tim Chu,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/57932

to look at the new patch set (#38).

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

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

Disable the PM ACPI timer during PMC init, when `USE_PM_ACPI_TIMER` is
disabled. This is done to bring SKL, CNL, DNV in line with the other
platforms, in order to transition handling of the PM timer from FSP to
coreboot in the follow-up changes.

For SKL and CNL, this temporarly redundantly disables the PM Timer,
since FSP does that, too. This redundancy is resolved in the follow-up.

Change-Id: I47280cd670a96c8fa5af107986496234f04e1f77
Signed-off-by: Michael Niewöhner 
---
M src/soc/intel/cannonlake/pmc.c
M src/soc/intel/denverton_ns/Kconfig
M src/soc/intel/denverton_ns/include/soc/pmc.h
M src/soc/intel/denverton_ns/pmc.c
M src/soc/intel/skylake/pmc.c
5 files changed, 32 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/57932/38