From e4aee2b178820be7a68d2b370595b20fe3f0cbee Mon Sep 17 00:00:00 2001
From: Subrata Banik <subratabanik@google.com>
Date: Mon, 19 Dec 2022 17:56:03 +0530
Subject: [PATCH] soc/intel/cannonlake: Move DPTF ACPI Device IDs into header
 file

This patch moves DPTF ACPI Device IDs into the header file (soc/dptf.h)
so that upcoming patches in this patch train can achieve more
common code.

TEST=Able to build and boot Google/Hatch.

Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: I7a9218a41825d2fa40a1c1b96a333465b7f617c0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71105
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kapil Porwal <kapilporwal@google.com>
---
 src/soc/intel/cannonlake/dptf.c             |  7 ++++---
 src/soc/intel/cannonlake/include/soc/dptf.h | 14 ++++++++++++++
 2 files changed, 18 insertions(+), 3 deletions(-)
 create mode 100644 src/soc/intel/cannonlake/include/soc/dptf.h

diff --git a/src/soc/intel/cannonlake/dptf.c b/src/soc/intel/cannonlake/dptf.c
index 8f1f98a9e8..54fac4c857 100644
--- a/src/soc/intel/cannonlake/dptf.c
+++ b/src/soc/intel/cannonlake/dptf.c
@@ -1,15 +1,16 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
 #include <drivers/intel/dptf/dptf.h>
+#include <soc/dptf.h>
 
 static const struct dptf_platform_info cnl_dptf_platform_info = {
 	.use_eisa_hids = true,
 	/* _HID for the toplevel DPTF device, typically \_SB.DPTF */
-	.dptf_device_hid = "INT3400",
+	.dptf_device_hid = DPTF_DPTF_DEVICE,
 	/* _HID for Intel DPTF Generic Device (these require PTYP as well) */
-	.generic_hid = "INT3403",
+	.generic_hid = DPTF_GEN_DEVICE,
 	/* _HID for Intel DPTF Fan Device */
-	.fan_hid = "INT3404",
+	.fan_hid = DPTF_FAN_DEVICE,
 };
 
 const struct dptf_platform_info *soc_get_dptf_platform_info(void)
diff --git a/src/soc/intel/cannonlake/include/soc/dptf.h b/src/soc/intel/cannonlake/include/soc/dptf.h
new file mode 100644
index 0000000000..79bcde9955
--- /dev/null
+++ b/src/soc/intel/cannonlake/include/soc/dptf.h
@@ -0,0 +1,14 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef _SOC_DPTF_H_
+#define _SOC_DPTF_H_
+
+/* Below are the unique ACPI Device IDs for thermal/dptf on Cannon Lake SoC. */
+/* DPTF ACPI Device ID */
+#define DPTF_DPTF_DEVICE	"INT3400"
+/* Generic ACPI Device ID for TSR0/1/2/3 and charger */
+#define DPTF_GEN_DEVICE		"INT3403"
+/* Fan ACPI Device ID */
+#define DPTF_FAN_DEVICE		"INT3404"
+
+#endif /* _SOC_DPTF_H_ */