diff options
author | Sridhar Siricilla <sridhar.siricilla@intel.com> | 2023-01-03 11:29:08 +0530 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2023-01-06 21:34:54 +0000 |
commit | 4c3014feb276cd14ce4fbea4f75d7b0952841cf3 (patch) | |
tree | d6f77985592550c461149f5d2ead87a002a8f830 | |
parent | 63fcc4acc20116241b007fc5bb6d4e48925b277d (diff) |
soc/intel/: Rename small and big cores references
The patch addresses Intel heterogeneous cores as `Efficient` and
`Performance` cores instead of `small` and `big` cores. It is to ensure
coreboot code has uniform reference to the heterogeneous cores. So, the
patch renames all `small` and `big` core references to `efficient` (eff)
and `performance` (perf) cores respectively.
TEST=Build the code for Brya and Rex boards
Signed-off-by: Sridhar Siricilla <sridhar.siricilla@intel.com>
Change-Id: I98c9c0ed86b211d736a0a1738b47410faa13a39f
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71639
Reviewed-by: Kapil Porwal <kapilporwal@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
-rw-r--r-- | src/soc/intel/alderlake/cpu.c | 6 | ||||
-rw-r--r-- | src/soc/intel/common/block/acpi/cpu_hybrid.c | 36 | ||||
-rw-r--r-- | src/soc/intel/common/block/include/intelblocks/acpi.h | 4 | ||||
-rw-r--r-- | src/soc/intel/meteorlake/cpu.c | 6 |
4 files changed, 26 insertions, 26 deletions
diff --git a/src/soc/intel/alderlake/cpu.c b/src/soc/intel/alderlake/cpu.c index 4bc0af762f..768f2d533d 100644 --- a/src/soc/intel/alderlake/cpu.c +++ b/src/soc/intel/alderlake/cpu.c @@ -99,10 +99,10 @@ enum core_type get_soc_cpu_type(void) return CPUID_CORE_TYPE_INTEL_CORE; } -void soc_get_scaling_factor(u16 *big_core_scal_factor, u16 *small_core_scal_factor) +void soc_get_scaling_factor(u16 *perf_core_scal_factor, u16 *eff_core_scal_factor) { - *big_core_scal_factor = 127; - *small_core_scal_factor = 100; + *perf_core_scal_factor = 127; + *eff_core_scal_factor = 100; } bool soc_is_nominal_freq_supported(void) diff --git a/src/soc/intel/common/block/acpi/cpu_hybrid.c b/src/soc/intel/common/block/acpi/cpu_hybrid.c index 4335a468d2..e5ccf468df 100644 --- a/src/soc/intel/common/block/acpi/cpu_hybrid.c +++ b/src/soc/intel/common/block/acpi/cpu_hybrid.c @@ -12,14 +12,14 @@ #define CPPC_NOM_PERF_IDX 3 enum cpu_perf_eff_type { - CPU_TYPE_SMALL, - CPU_TYPE_BIG, + CPU_TYPE_EFF, + CPU_TYPE_PERF, }; DECLARE_SPIN_LOCK(cpu_lock); static u8 global_cpu_type[CONFIG_MAX_CPUS]; -static bool is_big_core(void) +static bool is_perf_core(void) { return get_soc_cpu_type() == CPUID_CORE_TYPE_INTEL_CORE; } @@ -39,9 +39,9 @@ static u32 get_cpu_index(void) } /* - * This function determines the type (big or small) of the CPU that is executing - * it and stores the information (in a thread-safe manner) in an global_cpu_type - * array. + * This function determines the type (performance or efficient) of the CPU that + * is executing it and stores the information (in a thread-safe manner) in an + * global_cpu_type array. * It requires the SoC to implement a function `get_soc_cpu_type()` which will be * called in a critical section to determine the type of the executing CPU. */ @@ -50,8 +50,8 @@ static void set_cpu_type(void *unused) spin_lock(&cpu_lock); u8 cpu_index = get_cpu_index(); - if (is_big_core()) - global_cpu_type[cpu_index] = CPU_TYPE_BIG; + if (is_perf_core()) + global_cpu_type[cpu_index] = CPU_TYPE_PERF; spin_unlock(&cpu_lock); } @@ -64,16 +64,16 @@ static void run_set_cpu_type(void *unused) } } -static void acpi_get_cpu_nomi_perf(u16 *small_core_nom_perf, u16 *big_core_nom_perf) +static void acpi_get_cpu_nomi_perf(u16 *eff_core_nom_perf, u16 *perf_core_nom_perf) { - u16 big_core_scal_factor, small_core_scal_factor; + u16 perf_core_scal_factor, eff_core_scal_factor; u8 max_non_turbo_ratio = cpu_get_max_non_turbo_ratio(); - soc_get_scaling_factor(&big_core_scal_factor, &small_core_scal_factor); + soc_get_scaling_factor(&perf_core_scal_factor, &eff_core_scal_factor); - *big_core_nom_perf = (u16)((max_non_turbo_ratio * big_core_scal_factor) / 100); + *perf_core_nom_perf = (u16)((max_non_turbo_ratio * perf_core_scal_factor) / 100); - *small_core_nom_perf = (u16)((max_non_turbo_ratio * small_core_scal_factor) / 100); + *eff_core_nom_perf = (u16)((max_non_turbo_ratio * eff_core_scal_factor) / 100); } static u16 acpi_get_cpu_nominal_freq(void) @@ -84,18 +84,18 @@ static u16 acpi_get_cpu_nominal_freq(void) /* Updates Nominal Frequency and Nominal Performance */ static void acpigen_cppc_update_nominal_freq_perf(const char *pkg_path, s32 core_id) { - u16 small_core_nom_perf, big_core_nom_perf; + u16 eff_core_nom_perf, perf_core_nom_perf; if (!soc_is_nominal_freq_supported()) return; - acpi_get_cpu_nomi_perf(&small_core_nom_perf, &big_core_nom_perf); + acpi_get_cpu_nomi_perf(&eff_core_nom_perf, &perf_core_nom_perf); - if (global_cpu_type[core_id] == CPU_TYPE_BIG) - acpigen_set_package_element_int(pkg_path, CPPC_NOM_PERF_IDX, big_core_nom_perf); + if (global_cpu_type[core_id] == CPU_TYPE_PERF) + acpigen_set_package_element_int(pkg_path, CPPC_NOM_PERF_IDX, perf_core_nom_perf); else acpigen_set_package_element_int(pkg_path, CPPC_NOM_PERF_IDX, - small_core_nom_perf); + eff_core_nom_perf); /* Update CPU's nominal frequency */ acpigen_set_package_element_int(pkg_path, CPPC_NOM_FREQ_IDX, diff --git a/src/soc/intel/common/block/include/intelblocks/acpi.h b/src/soc/intel/common/block/include/intelblocks/acpi.h index c149c6939e..63f38bf308 100644 --- a/src/soc/intel/common/block/include/intelblocks/acpi.h +++ b/src/soc/intel/common/block/include/intelblocks/acpi.h @@ -18,8 +18,8 @@ enum core_type { CPUID_UNKNOWN = 0xff, }; -/* Gets the scaling factor for small and big core */ -void soc_get_scaling_factor(u16 *big_core_scal_factor, u16 *small_core_scal_factor); +/* Gets the scaling factor for Efficient and Performance core */ +void soc_get_scaling_factor(u16 *perf_core_scal_factor, u16 *eff_core_scal_factor); /* Generates ACPI code to define _CPC control method */ void acpigen_write_CPPC_hybrid_method(int core_id); diff --git a/src/soc/intel/meteorlake/cpu.c b/src/soc/intel/meteorlake/cpu.c index e0ac726542..b299cc89cf 100644 --- a/src/soc/intel/meteorlake/cpu.c +++ b/src/soc/intel/meteorlake/cpu.c @@ -87,10 +87,10 @@ enum core_type get_soc_cpu_type(void) return CPUID_CORE_TYPE_INTEL_CORE; } -void soc_get_scaling_factor(u16 *big_core_scal_factor, u16 *small_core_scal_factor) +void soc_get_scaling_factor(u16 *perf_core_scal_factor, u16 *eff_core_scal_factor) { - *big_core_scal_factor = 125; - *small_core_scal_factor = 100; + *perf_core_scal_factor = 125; + *eff_core_scal_factor = 100; } bool soc_is_nominal_freq_supported(void) |