[coreboot-gerrit] Change in coreboot[master]: soc/intel/{cnl,icl,tgl,jsl,ehl,adl}/acpi: generate CPPC entries

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45536 )

Change subject: soc/intel/{cnl,icl,tgl,jsl,ehl,adl}/acpi: generate CPPC entries
......................................................................

Patch Set 10:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45536/10/src/soc/intel/alderlake/ac... 
File src/soc/intel/alderlake/acpi.c:

https://review.coreboot.org/c/coreboot/+/45536/10/src/soc/intel/alderlake/ac... 
PS10, Line 162: 		generate_cppc_entries(core_id);
Why jump through so many hoops here? soc calls generate_cpu_entries(), calls
back here, calls back there. I guess the hardware involves the soc-specific
config struct, but then again, why is it a devicetree config?