soc/intel/xeon_sp: Rename cpx_generate_p_state_entries()
Prepare for common ACPI. Rename cpx_generated_p_state_entries() to the common soc_power_states_generation() function. Add empty soc_power_states_generation() to skx. Change-Id: Ib7e8dfd2bb602f3e6ccdb5b221bc65236f66a875 Signed-off-by: Marc Jones <marcjones@sysproconsulting.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/46598 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
parent
3fc04842cb
commit
7a25fb8e69
|
@ -261,8 +261,9 @@ void generate_cpu_entries(const struct device *device)
|
||||||
|
|
||||||
/* NOTE: Intel idle driver doesn't use ACPI C-state tables */
|
/* NOTE: Intel idle driver doesn't use ACPI C-state tables */
|
||||||
|
|
||||||
/* Generate P-state tables */
|
/* Soc specific power states generation */
|
||||||
cpx_generate_p_state_entries(core_id, threads_per_package);
|
soc_power_states_generation(core_id, threads_per_package);
|
||||||
|
|
||||||
acpigen_pop_len();
|
acpigen_pop_len();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -123,8 +123,8 @@ void uncore_inject_dsdt(const struct device *device)
|
||||||
acpigen_pop_len();
|
acpigen_pop_len();
|
||||||
}
|
}
|
||||||
|
|
||||||
/* To be renamed soc_power_states_generation() */
|
/* TODO: See if we can use the common generate_p_state_entries */
|
||||||
void cpx_generate_p_state_entries(int core, int cores_per_package)
|
void soc_power_states_generation(int core, int cores_per_package)
|
||||||
{
|
{
|
||||||
int ratio_min, ratio_max, ratio_turbo, ratio_step;
|
int ratio_min, ratio_max, ratio_turbo, ratio_step;
|
||||||
int coord_type, power_max, power_unit, num_entries;
|
int coord_type, power_max, power_unit, num_entries;
|
||||||
|
|
|
@ -22,7 +22,6 @@ unsigned long northbridge_write_acpi_tables(const struct device *device,
|
||||||
|
|
||||||
void motherboard_fill_fadt(acpi_fadt_t *fadt);
|
void motherboard_fill_fadt(acpi_fadt_t *fadt);
|
||||||
|
|
||||||
void cpx_generate_p_state_entries(int core, int cores_per_package);
|
|
||||||
int calculate_power(int tdp, int p1_ratio, int ratio);
|
int calculate_power(int tdp, int p1_ratio, int ratio);
|
||||||
void uncore_inject_dsdt(const struct device *device);
|
void uncore_inject_dsdt(const struct device *device);
|
||||||
unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current);
|
unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current);
|
||||||
|
|
|
@ -171,6 +171,10 @@ void uncore_inject_dsdt(const struct device *device)
|
||||||
acpigen_pop_len();
|
acpigen_pop_len();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void soc_power_states_generation(int core, int cores_per_package)
|
||||||
|
{
|
||||||
|
}
|
||||||
|
|
||||||
unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current)
|
unsigned long xeonsp_acpi_create_madt_lapics(unsigned long current)
|
||||||
{
|
{
|
||||||
struct device *cpu;
|
struct device *cpu;
|
||||||
|
|
Loading…
Reference in New Issue