Attention is currently required from: Lance Zhao, Tim Wawrzynczak. Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/59358 )
Change subject: src/include/acpi: Move CPPC_PACKAGE_NAME macro definition ......................................................................
src/include/acpi: Move CPPC_PACKAGE_NAME macro definition
The patch move CPPC_PACKAGE_NAME macro definition from acpi/acpigen.c to include/acpi/acpigen.h.
TEST=Verified on Brya.
Signed-off-by: Sridhar Siricilla sridhar.siricilla@intel.com Change-Id: Ic547445cdbe2b1a3efe44390bd127f577386e7fc --- M src/acpi/acpigen.c M src/include/acpi/acpigen.h 2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/59358/1
diff --git a/src/acpi/acpigen.c b/src/acpi/acpigen.c index 2b74261..ff04d0f 100644 --- a/src/acpi/acpigen.c +++ b/src/acpi/acpigen.c @@ -10,8 +10,6 @@
#define ACPIGEN_MAXLEN 0xfffff
-#define CPPC_PACKAGE_NAME "GCPC" - #include <lib.h> #include <string.h> #include <acpi/acpigen.h> diff --git a/src/include/acpi/acpigen.h b/src/include/acpi/acpigen.h index 463c16d..027a0e3 100644 --- a/src/include/acpi/acpigen.h +++ b/src/include/acpi/acpigen.h @@ -235,6 +235,8 @@ #define CPPC_VERSION_2 2 #define CPPC_VERSION_3 3
+#define CPPC_PACKAGE_NAME "GCPC" + /*version 1 has 15 fields, version 2 has 19, and version 3 has 21 */ enum cppc_fields { CPPC_HIGHEST_PERF, /* can be DWORD */