[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma: put controller in separate header

[Matt DeVillier (Code Review)]

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/40592 )

Change subject: drivers/intel/gma: put controller in separate header
......................................................................

drivers/intel/gma: put controller in separate header

Including i915.h just for the GMA/SSDT related functions means
dragging along all of i915_reg.h as well, which is problematic
since some platforms (like Apollolake) use overlapping symbols.
To avoid this conflict, break out the GMA/SSDT bits into their
own header which can be included without conflict.

Change-Id: I73fb7ef01abaafdcdbc44f1e3f5eb1883fc31616
Signed-off-by: Matt DeVillier 
---
M src/drivers/intel/gma/i915.h
A src/drivers/intel/gma/i915_gma.h
2 files changed, 25 insertions(+), 15 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/92/40592/1

diff --git a/src/drivers/intel/gma/i915.h b/src/drivers/intel/gma/i915.h
index f8721a2..cede8ee 100644
--- a/src/drivers/intel/gma/i915.h
+++ b/src/drivers/intel/gma/i915.h
@@ -4,6 +4,7 @@
 #ifndef INTEL_I915_H
 #define INTEL_I915_H 1

+#include <drivers/intel/gma/i915_gma.h>
 #include <drivers/intel/gma/i915_reg.h>
 #include <drivers/intel/gma/drm_dp_helper.h>
 #include <edid.h>
@@ -75,21 +76,6 @@
 void gtt_write(u32 reg, u32 data);
 u32 gtt_read(u32 reg);

-struct i915_gpu_controller_info
-{
-	int use_spread_spectrum_clock;
-	int ndid;
-	u32 did[5];
-};
-
-#define GMA_STATIC_DISPLAYS(ssc) {			\
-	.use_spread_spectrum_clock = (ssc),		\
-	.ndid = 3, .did = { 0x0100, 0x0240, 0x0410, }	\
-}
-
-void
-drivers_intel_gma_displays_ssdt_generate(const struct i915_gpu_controller_info *conf);
-
 /* vbt.c */
 struct device;
 void
diff --git a/src/drivers/intel/gma/i915_gma.h b/src/drivers/intel/gma/i915_gma.h
new file mode 100644
index 0000000..4b4da95
--- /dev/null
+++ b/src/drivers/intel/gma/i915_gma.h
@@ -0,0 +1,24 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/* This file is part of the coreboot project. */
+
+#ifndef _I915_GMA_H_
+#define _I915_GMA_H_
+
+#include <stdint.h>
+
+struct i915_gpu_controller_info
+{
+	int use_spread_spectrum_clock;
+	int ndid;
+	u32 did[5];
+};
+
+#define GMA_STATIC_DISPLAYS(ssc) {			\
+	.use_spread_spectrum_clock = (ssc),		\
+	.ndid = 3, .did = { 0x0100, 0x0240, 0x0410, }	\
+}
+
+void
+drivers_intel_gma_displays_ssdt_generate(const struct i915_gpu_controller_info *conf);
+
+#endif