Attention is currently required from: Kapil Porwal, Pranava Y N, Subrata Banik.
Hello Kapil Porwal, Pranava Y N, Subrata Banik, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85554?usp=email
to look at the new patch set (#10).
The following approvals got outdated and were removed: Verified+1 by build bot (Jenkins)
Change subject: soc/intel/pantherlake: Add core scaling factors read support ......................................................................
soc/intel/pantherlake: Add core scaling factors read support
This commit adds support for reading core scaling factors via the PCODE mailbox interface.
Starting with Lunar Lake, the scaling factor information is centralized in the power control unit (PCU) firmware. In order to keep all firmware in sync, it is recommended to read the scaling factors from the PCU firmware instead of using hard-coded values.
The following changes were made:
- Updated the Kconfig file to select SOC_INTEL_COMMON_BLOCK_RUNTIME_CORE_SCALING_FACTORS option
- Modified the acpi.h header file to export the cpu_perf_eff_type enumeration for CPU performance/efficiency types.
- Added a new function to the pantherlake systemagent.c file, soc_read_core_scaling_factors(), which reads the core scaling factors from the PCODE mailbox interface. The pcode READ_CORE_SCALING_FACTOR is presented in document 829201 Panther Lake Processor Mailbox Command.
The performance impact on boot time is minimal. It took 12 us to read the scaling factors on a fatcat device.
TEST=Successfully read performance and efficient scaling factors on a fatcat board.
Change-Id: I7a8e1e66a02e4bf6b1a41277e83c6dec786fe169 Signed-off-by: Jeremy Compostella jeremy.compostella@intel.com --- M src/soc/intel/common/block/acpi/cpu_hybrid.c M src/soc/intel/common/block/include/intelblocks/acpi.h M src/soc/intel/pantherlake/Kconfig M src/soc/intel/pantherlake/include/soc/systemagent.h M src/soc/intel/pantherlake/systemagent.c 5 files changed, 148 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/85554/10