Change in coreboot[master]: soc/intel/skl,acpi/acpigen: convert global CPPC package to local one

[Matt Delco (Code Review)]

Matt Delco has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46465 )

Change subject: soc/intel/skl,acpi/acpigen: convert global CPPC package to local one
......................................................................

Patch Set 16: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/46465/16/src/acpi/acpigen.c 
File src/acpi/acpigen.c:

https://review.coreboot.org/c/coreboot/+/46465/16/src/acpi/acpigen.c@1588 
PS16, Line 1588: 	acpigen_emit_namestring("\_SB.CP00.GCPC");
Seems alright though an approach like the following approach on hard-coding might be easier to swallow (while being within reason for some compilers to compute at build-time instead of run-time).

#define CPPC_PACKAGE_NAME "GCPC"
// acpigen_write_CPPC_package()
acpigen_write_name(CPPC_PACKAGE_NAME);
// acpigen_write_CPPC_method()
char pscope[16];
snprintf(pscope, sizeof(pscope), CONFIG_ACPI_CPU_STRING "." CPPC_PACKAGE_NAME, 0);