Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45535 )
Change subject: soc/intel/common/block: add code for ACPI CPPC entries generation ......................................................................
Patch Set 35:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45535/35/src/soc/intel/common/block... File src/soc/intel/common/block/acpi/acpi.c:
https://review.coreboot.org/c/coreboot/+/45535/35/src/soc/intel/common/block... PS35, Line 399: #if CONFIG(SOC_INTEL_COMMON_BLOCK_ACPI_CPPC) Do we need this? AFAICS, just dropping this would make `static` work just fine.