[coreboot-gerrit] Change in coreboot[master]: acpigen,soc/intel/{common,skl}: rework generation of the CPPC table

[Michael Niewöhner (Code Review)]

Attention is currently required from:  Felix Singer, Furquan Shaikh, Paul Menzel, Angel Pons, Subrata Banik, Michael Niewöhner, Kyösti Mälkki, Patrick Rudolph, Lance Zhao, Matt Delco, Nico Huber, Tim Wawrzynczak.
Hello  Felix Singer, build bot (Jenkins), Furquan Shaikh, Paul Menzel, Angel Pons, Subrata Banik, Kyösti Mälkki, Patrick Rudolph, Lance Zhao, Matt Delco, Nico Huber, Tim Wawrzynczak, Tim Wawrzynczak,

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

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

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

Change subject: acpigen,soc/intel/{common,skl}: rework generation of the CPPC table
......................................................................

acpigen,soc/intel/{common,skl}: rework generation of the CPPC table

Rework generation of the CPPC table by the CPU code to use an
easily-readable array instead of multiple confusing assignments of
structs that only get filled partly.

Also, pass the CPPC version directly to the ACPI generator instead of
passing it around indirectly through a struct.

This results in requiring only one call in CPU code instead of two by
also moving the table init function call to acpigen.

Comments for documentation were added, where appropriate.

Change-Id: I844d191b1134b98e409240ede71e2751e51e2159
Signed-off-by: Michael Niewöhner 
---
M src/acpi/acpigen.c
M src/cpu/intel/common/common.h
M src/cpu/intel/common/common_init.c
M src/include/acpi/acpigen.h
M src/soc/intel/common/block/acpi/acpi.c
M src/soc/intel/skylake/acpi.c
6 files changed, 91 insertions(+), 213 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/57888/4