dptf: Add support for IDSP

\_SB.DPTF.IDSP adverties to the DPTF daemon which policies the
implementation supports. Added a new acpigen function to figure out
which policies are used, and fills out IDSP appropriately.

Change-Id: Idf67a23bf38de4481c02f98ffb27afb8ca2d1b7b
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42081
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>

3 files changed, 58 insertions, 0 deletions

diff --git a/src/acpi/acpigen_dptf.c b/src/acpi/acpigen_dptf.c
index 59afa551a5..c8849261d6 100644
--- a/src/acpi/acpigen_dptf.c
+++ b/src/acpi/acpigen_dptf.c
@@ -9,6 +9,11 @@
 /* Defaults */
 #define DEFAULT_RAW_UNIT		"ma"
 
+/* DPTF-specific UUIDs */
+#define DPTF_PASSIVE_POLICY_1_0_UUID	"42A441D6-AE6A-462B-A84B-4A8CE79027D3"
+#define DPTF_CRITICAL_POLICY_UUID	"97C68AE7-15FA-499c-B8C9-5DA81D606E0A"
+#define DPTF_ACTIVE_POLICY_UUID		"3A95C389-E4B8-4629-A526-C52C88626BAE"
+
 enum {
 	ART_REVISION			= 0,
 	DEFAULT_PRIORITY		= 100,
@@ -432,3 +437,40 @@ void dptf_write_tsr_hysteresis(uint8_t hysteresis)
 
 	acpigen_write_name_integer("GTSH", hysteresis);
 }
+
+void dptf_write_enabled_policies(const struct dptf_active_policy *active_policies,
+				 int active_count,
+				 const struct dptf_passive_policy *passive_policies,
+				 int passive_count,
+				 const struct dptf_critical_policy *critical_policies,
+				 int critical_count)
+{
+	bool is_active_used;
+	bool is_passive_used;
+	bool is_critical_used;
+	int pkg_count;
+
+	is_active_used = (active_count && active_policies[0].target != DPTF_NONE);
+	is_passive_used = (passive_count && passive_policies[0].target != DPTF_NONE);
+	is_critical_used = (critical_count && critical_policies[0].source != DPTF_NONE);
+	pkg_count = is_active_used + is_passive_used + is_critical_used;
+
+	if (!pkg_count)
+		return;
+
+	acpigen_write_scope(TOPLEVEL_DPTF_SCOPE);
+	acpigen_write_name("IDSP");
+	acpigen_write_package(pkg_count);
+
+	if (is_active_used)
+		acpigen_write_uuid(DPTF_ACTIVE_POLICY_UUID);
+
+	if (is_passive_used)
+		acpigen_write_uuid(DPTF_PASSIVE_POLICY_1_0_UUID);
+
+	if (is_critical_used)
+		acpigen_write_uuid(DPTF_CRITICAL_POLICY_UUID);
+
+	acpigen_pop_len(); /* Package */
+	acpigen_pop_len(); /* Scope */
+}
diff --git a/src/drivers/intel/dptf/dptf.c b/src/drivers/intel/dptf/dptf.c
index 1fe9653eb7..1e4063e40f 100644
--- a/src/drivers/intel/dptf/dptf.c
+++ b/src/drivers/intel/dptf/dptf.c
@@ -69,6 +69,11 @@ static void dptf_fill_ssdt(const struct device *dev)
 	for (p = DPTF_TEMP_SENSOR_0, i = 0; p <= DPTF_TEMP_SENSOR_3; ++p, ++i)
 		tsr_en[i] = is_participant_used(config, p);
 
+	/* Policies */
+	dptf_write_enabled_policies(config->policies.active, DPTF_MAX_ACTIVE_POLICIES,
+				    config->policies.passive, DPTF_MAX_PASSIVE_POLICIES,
+				    config->policies.critical, DPTF_MAX_CRITICAL_POLICIES);
+
 	dptf_write_active_policies(config->policies.active,
 				   DPTF_MAX_ACTIVE_POLICIES);
 
diff --git a/src/include/acpi/acpigen_dptf.h b/src/include/acpi/acpigen_dptf.h
index 496840b8b4..89b64f3728 100644
--- a/src/include/acpi/acpigen_dptf.h
+++ b/src/include/acpi/acpigen_dptf.h
@@ -127,6 +127,17 @@ struct dptf_power_limits {
 };
 
 /*
+ * This function writes out \_SB.DPTF.IDSP, which describes the different DPTF policies that
+ * this implementation is using.
+ */
+void dptf_write_enabled_policies(const struct dptf_active_policy *active_policies,
+				 int active_count,
+				 const struct dptf_passive_policy *passive_policies,
+				 int passive_count,
+				 const struct dptf_critical_policy *critical_policies,
+				 int critical_count);
+
+/*
  * This function provides tables of temperature and corresponding fan or percent.  When the
  * temperature thresholds are met (_AC0 - _AC9), the fan is driven to corresponding percentage
  * of full speed.