From fd8c596c406ad23484e293dd3892ccf99c95e25b Mon Sep 17 00:00:00 2001
From: Subrata Banik <subratabanik@google.com>
Date: Mon, 19 Dec 2022 18:02:42 +0530
Subject: [PATCH] soc/intel/jasperlake: 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/Kano.

Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: Ibb31ab29c803dde70ef9ccf2b7c7c2ca0845b568
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71106
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/jasperlake/dptf.c             |  7 ++++---
 src/soc/intel/jasperlake/include/soc/dptf.h | 14 ++++++++++++++
 2 files changed, 18 insertions(+), 3 deletions(-)
 create mode 100644 src/soc/intel/jasperlake/include/soc/dptf.h

diff --git a/src/soc/intel/jasperlake/dptf.c b/src/soc/intel/jasperlake/dptf.c
index 4ee38de2a1..ef5291677a 100644
--- a/src/soc/intel/jasperlake/dptf.c
+++ b/src/soc/intel/jasperlake/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 jsl_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/jasperlake/include/soc/dptf.h b/src/soc/intel/jasperlake/include/soc/dptf.h
new file mode 100644
index 0000000000..9917697af8
--- /dev/null
+++ b/src/soc/intel/jasperlake/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 Jasper 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_ */