[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Define enum cpu_perf_eff_type type for core types

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/61459 )

Change subject: soc/intel/common: Define enum cpu_perf_eff_type type for core types
......................................................................

soc/intel/common: Define enum cpu_perf_eff_type type for core types

The patch defines enum values for small and big cores and uses them
to indicate the big or small core.

TEST=Verify the build for Brya

Signed-off-by: Sridhar Siricilla 
Change-Id: I740984a437da9d0518652f43180faf9b6ed4255e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61459
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Maulik V Vaghela 
---
M src/soc/intel/common/block/acpi/cpu_hybrid.c
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Maulik V Vaghela: Looks good to me, approved
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/soc/intel/common/block/acpi/cpu_hybrid.c b/src/soc/intel/common/block/acpi/cpu_hybrid.c
index 70e6329..4335a46 100644
--- a/src/soc/intel/common/block/acpi/cpu_hybrid.c
+++ b/src/soc/intel/common/block/acpi/cpu_hybrid.c
@@ -11,6 +11,11 @@
 #define CPPC_NOM_FREQ_IDX	22
 #define CPPC_NOM_PERF_IDX	3

+enum cpu_perf_eff_type {
+	CPU_TYPE_SMALL,
+	CPU_TYPE_BIG,
+};
+
 DECLARE_SPIN_LOCK(cpu_lock);
 static u8 global_cpu_type[CONFIG_MAX_CPUS];

@@ -46,7 +51,7 @@
    u8 cpu_index = get_cpu_index();

if (is_big_core())
-		global_cpu_type[cpu_index] = 1;
+		global_cpu_type[cpu_index] = CPU_TYPE_BIG;

spin_unlock(&cpu_lock);
 }
@@ -86,7 +91,7 @@

acpi_get_cpu_nomi_perf(&small_core_nom_perf, &big_core_nom_perf);

-	if (global_cpu_type[core_id])
+	if (global_cpu_type[core_id] == CPU_TYPE_BIG)
    	acpigen_set_package_element_int(pkg_path, CPPC_NOM_PERF_IDX, big_core_nom_perf);
    else
    	acpigen_set_package_element_int(pkg_path, CPPC_NOM_PERF_IDX,