[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/cpu: add function to init cppc_config

Matt Delco (Code Review) gerrit at coreboot.org
Mon Aug 13 23:04:48 CEST 2018


Matt Delco has uploaded this change for review. ( https://review.coreboot.org/28068


Change subject: soc/intel/common/block/cpu: add function to init cppc_config
......................................................................

soc/intel/common/block/cpu: add function to init cppc_config

This change adds a method to init a cppc_config structure in a way that
should ideally work across Intel processors that support EIST.

Change-Id: Ib767df63d796bd1f21e36bcf575cf912e09090a1
Signed-off-by: Matt Delco <delco at chromium.org>
---
M src/soc/intel/common/block/cpu/cpulib.c
M src/soc/intel/common/block/include/intelblocks/cpulib.h
2 files changed, 187 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/28068/1

diff --git a/src/soc/intel/common/block/cpu/cpulib.c b/src/soc/intel/common/block/cpu/cpulib.c
index 112a049..9191fd5 100644
--- a/src/soc/intel/common/block/cpu/cpulib.c
+++ b/src/soc/intel/common/block/cpu/cpulib.c
@@ -216,6 +216,185 @@
 }
 
 /*
+ * Init cppc_config in a way that's appropriate for Intel
+ * processors with Intel Enhanced Speed Step Technology.
+ * NOTE: version 2 is expected to be the typical use case.
+ * For now this function 'punts' on version 3 and just
+ * populates the additional fields with 'unsupported'.
+ */
+void cpu_init_cppc_config(struct cppc_config *config, u32 version)
+{
+	acpi_addr_t msr = {
+		.space_id   = ACPI_ADDRESS_SPACE_FIXED,
+		.bit_width  = 8,
+		.bit_offset = 0,
+		{
+			.access_size = 4
+		},
+		.addrl      = 0,
+		.addrh      = 0,
+	};
+	static const acpi_addr_t unsupported = {
+		.space_id   = ACPI_ADDRESS_SPACE_MEMORY,
+		.bit_width  = 0,
+		.bit_offset = 0,
+		{
+			.resv = 0
+		},
+		.addrl      = 0,
+		.addrh      = 0,
+	};
+
+	config->version = version;
+
+	msr.addrl = MSR_HWP_CAPABILITIES;
+
+	/*
+	 * Highest Performance:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x00, 0x771, 0x04,)},
+	 */
+	config->regs[CPPC_HIGHEST_PERF] = msr;
+
+	/*
+	 * Nominal Performance -> Guaranteed Performance:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x08, 0x771, 0x04,)},
+	 */
+	msr.bit_offset = 8;
+	config->regs[CPPC_NOMINAL_PERF] = msr;
+
+	/*
+	 * Lowest Nonlinear Performance -> Most Efficient Performance:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x10, 0x771, 0x04,)},
+	 */
+	msr.bit_offset = 16;
+	config->regs[CPPC_LOWEST_NONL_PERF] = msr;
+
+	/*
+	 * Lowest Performance:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x18, 0x771, 0x04,)},
+	 */
+	msr.bit_offset = 24;
+	config->regs[CPPC_LOWEST_PERF] = msr;
+
+	/*
+	 * Guaranteed Performance Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x08, 0x771, 0x04,)},
+	 */
+	msr.bit_offset = 8;
+	config->regs[CPPC_GUARANTEED_PERF] = msr;
+
+	msr.addrl = MSR_HWP_REQUEST;
+
+	/*
+	 * Desired Performance Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x10, 0x774, 0x04,)},
+	 */
+	msr.bit_offset = 16;
+	config->regs[CPPC_DESIRED_PERF] = msr;
+
+	/*
+	 * Minimum Performance Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x00, 0x774, 0x04,)},
+	 */
+	msr.bit_offset = 0;
+	config->regs[CPPC_MIN_PERF] = msr;
+
+	/*
+	 * Maximum Performance Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x08, 0x08, 0x774, 0x04,)},
+	 */
+	msr.bit_offset = 8;
+	config->regs[CPPC_MAX_PERF] = msr;
+
+	/*
+	 * Performance Reduction Tolerance Register:
+	 * ResourceTemplate(){Register(SystemMemory, 0x00, 0x00, 0x0,,)},
+	 */
+	config->regs[CPPC_PERF_REDUCE_TOLERANCE] = unsupported;
+
+	/*
+	 * Time Window Register:
+	 * ResourceTemplate(){Register(SystemMemory, 0x00, 0x00, 0x0,,)},
+	 */
+	config->regs[CPPC_TIME_WINDOW] = unsupported;
+
+	/*
+	 * Counter Wraparound Time:
+	 * ResourceTemplate(){Register(SystemMemory, 0x00, 0x00, 0x0,,)},
+	 */
+	config->regs[CPPC_COUNTER_WRAP] = unsupported;
+
+	msr.addrl = MSR_MPERF;
+
+	/*
+	 * Reference Performance Counter Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x40, 0x00, 0x0E7, 0x04,)},
+	 */
+	msr.bit_width = 64;
+	msr.bit_offset = 0;
+	config->regs[CPPC_REF_PERF_COUNTER] = msr;
+
+	msr.addrl = MSR_APERF;
+
+	/*
+	 * Delivered Performance Counter Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x40, 0x00, 0x0E8, 0x04,)},
+	 */
+	config->regs[CPPC_DELIVERED_PERF_COUNTER] = msr;
+
+	msr.addrl = MSR_HWP_STATUS;
+
+	/*
+	 * Performance Limited Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x01, 0x02, 0x777, 0x04,)},
+	 */
+	msr.bit_width = 1;
+	msr.bit_offset = 2;
+	config->regs[CPPC_PERF_LIMITED] = msr;
+
+	msr.addrl = MSR_PM_ENABLE;
+
+	/*
+	 * CPPC Enable Register:
+	 * ResourceTemplate(){Register(FFixedHW, 0x01, 0x00, 0x770, 0x04,)},
+	 */
+	msr.bit_offset = 0;
+	config->regs[CPPC_ENABLE] = msr;
+
+	if (version >= 2) {
+		/* Autonomous Selection Enable is populated below */
+
+		/* Autonomous Activity Window Register */
+		config->regs[CPPC_AUTO_ACTIVITY_WINDOW] = unsupported;
+
+		/* Energy Performance Preference Register */
+		config->regs[CPPC_PERF_PREF] = unsupported;
+
+		/* Reference Performance */
+		config->regs[CPPC_REF_PERF] = unsupported;
+
+		if (version >= 3) {
+			/* Lowest Frequency */
+			config->regs[CPPC_LOWEST_FREQ] = unsupported;
+			/* Nominal Frequency */
+			config->regs[CPPC_NOMINAL_FREQ] = unsupported;
+		}
+
+		/*
+		 * Autonomous Selection Enable = 1
+		 * This field is actually the first addition in version 2 but
+		 * it's so unlike the others I'm populating it last.
+		 */
+		msr.space_id    = ACPI_ADDRESS_SPACE_MEMORY;
+		msr.bit_width   = 32;
+		msr.bit_offset  = 0;
+		msr.access_size = 0;
+		msr.addrl       = 1;
+		config->regs[CPPC_AUTO_SELECT] = msr;
+	}
+}
+
+/*
  * Set Bit 6 (ENABLE_IA_UNTRUSTED_MODE) of MSR 0x120
  * UCODE_PCR_POWER_MISC MSR to enter IA Untrusted Mode.
  */
diff --git a/src/soc/intel/common/block/include/intelblocks/cpulib.h b/src/soc/intel/common/block/include/intelblocks/cpulib.h
index 88f04b4..0071033 100644
--- a/src/soc/intel/common/block/include/intelblocks/cpulib.h
+++ b/src/soc/intel/common/block/include/intelblocks/cpulib.h
@@ -109,6 +109,14 @@
 void cpu_disable_eist(void);
 
 /*
+ * Init CPPC block with MSRs for Intel Enhanced Speed Step Technology.
+ * Version 2 is suggested--this function's implementation of version 3
+ * may have room for improvment.
+ */
+struct cppc_config;
+void cpu_init_cppc_config(struct cppc_config *config, u32 version);
+
+/*
  * Set Bit 6 (ENABLE_IA_UNTRUSTED_MODE) of MSR 0x120
  * UCODE_PCR_POWER_MISC MSR to enter IA Untrusted Mode.
  */

-- 
To view, visit https://review.coreboot.org/28068
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib767df63d796bd1f21e36bcf575cf912e09090a1
Gerrit-Change-Number: 28068
Gerrit-PatchSet: 1
Gerrit-Owner: Matt Delco <delco at chromium.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180813/535cba31/attachment.html>


More information about the coreboot-gerrit mailing list