summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-12-05 19:11:25 +0100
committerHung-Te Lin <hungte@chromium.org>2020-12-17 13:52:59 +0000
commitc8be0947f1d2665f9ddaca77c9739f55980551ac (patch)
treef6c2747d788639a887cd80a0f7f274dfcd2971cc
parentb8602aa1842044f0801afc61de931f1733272ba5 (diff)
mb/intel/coffeelake_rvp: Stop using headers for HDA verbs
One of the variants lacks an hda_verb.h, and hda_verb.c can't be built. Follow-up changes will make mainboard hda_verb.c files always get built through AZALIA_PLUGIN_SUPPORT, and breaks building this contraption. Turn the headers into standalone compilation units to prevent this issue. Since they contain definitions, including them from multiple compilation units wasn't a good idea anyway. Change-Id: I00d968563539a4e1b8d1e12145293439d8358555 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/48360 Reviewed-by: Patrick Rudolph <siro@das-labor.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/mainboard/intel/coffeelake_rvp/Makefile.inc2
-rw-r--r--src/mainboard/intel/coffeelake_rvp/hda_verb.c3
-rw-r--r--src/mainboard/intel/coffeelake_rvp/variants/cfl_h/hda_verb.c (renamed from src/mainboard/intel/coffeelake_rvp/variants/cfl_h/include/variant/hda_verb.h)4
-rw-r--r--src/mainboard/intel/coffeelake_rvp/variants/cfl_s/hda_verb.c (renamed from src/mainboard/intel/coffeelake_rvp/variants/cfl_s/include/variant/hda_verb.h)4
-rw-r--r--src/mainboard/intel/coffeelake_rvp/variants/cml_u/hda_verb.c (renamed from src/mainboard/intel/coffeelake_rvp/variants/whl_u/include/variant/hda_verb.h)4
-rw-r--r--src/mainboard/intel/coffeelake_rvp/variants/whl_u/hda_verb.c (renamed from src/mainboard/intel/coffeelake_rvp/variants/cml_u/include/variant/hda_verb.h)4
6 files changed, 1 insertions, 20 deletions
diff --git a/src/mainboard/intel/coffeelake_rvp/Makefile.inc b/src/mainboard/intel/coffeelake_rvp/Makefile.inc
index a3061e3db5..77f06913fa 100644
--- a/src/mainboard/intel/coffeelake_rvp/Makefile.inc
+++ b/src/mainboard/intel/coffeelake_rvp/Makefile.inc
@@ -10,7 +10,7 @@ romstage-y += memory.c
ramstage-$(CONFIG_CHROMEOS) += chromeos.c
ramstage-y += mainboard.c
-ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_HDA_VERB) += hda_verb.c
+ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_HDA_VERB) += variants/$(VARIANT_DIR)/hda_verb.c
subdirs-y += variants/baseboard
CPPFLAGS_common += -I$(src)/mainboard/$(MAINBOARDDIR)/variants/baseboard/include
diff --git a/src/mainboard/intel/coffeelake_rvp/hda_verb.c b/src/mainboard/intel/coffeelake_rvp/hda_verb.c
deleted file mode 100644
index c26029774e..0000000000
--- a/src/mainboard/intel/coffeelake_rvp/hda_verb.c
+++ /dev/null
@@ -1,3 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#include "variant/hda_verb.h"
diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/include/variant/hda_verb.h b/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/hda_verb.c
index e276cf7f6e..cbe8f0be21 100644
--- a/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/include/variant/hda_verb.h
+++ b/src/mainboard/intel/coffeelake_rvp/variants/cfl_h/hda_verb.c
@@ -1,8 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#ifndef HDA_VERB_H
-#define HDA_VERB_H
-
#include <device/azalia_device.h>
const u32 cim_verb_data[] = {
@@ -183,4 +180,3 @@ const u32 pc_beep_verbs[] = {
};
AZALIA_ARRAY_SIZES;
-#endif
diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/include/variant/hda_verb.h b/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/hda_verb.c
index e276cf7f6e..cbe8f0be21 100644
--- a/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/include/variant/hda_verb.h
+++ b/src/mainboard/intel/coffeelake_rvp/variants/cfl_s/hda_verb.c
@@ -1,8 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#ifndef HDA_VERB_H
-#define HDA_VERB_H
-
#include <device/azalia_device.h>
const u32 cim_verb_data[] = {
@@ -183,4 +180,3 @@ const u32 pc_beep_verbs[] = {
};
AZALIA_ARRAY_SIZES;
-#endif
diff --git a/src/mainboard/intel/coffeelake_rvp/variants/whl_u/include/variant/hda_verb.h b/src/mainboard/intel/coffeelake_rvp/variants/cml_u/hda_verb.c
index 7233250105..01012dde1b 100644
--- a/src/mainboard/intel/coffeelake_rvp/variants/whl_u/include/variant/hda_verb.h
+++ b/src/mainboard/intel/coffeelake_rvp/variants/cml_u/hda_verb.c
@@ -1,8 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#ifndef HDA_VERB_H
-#define HDA_VERB_H
-
#include <device/azalia_device.h>
const u32 cim_verb_data[] = {
@@ -684,4 +681,3 @@ const u32 cim_verb_data[] = {
const u32 pc_beep_verbs[] = {
};
AZALIA_ARRAY_SIZES;
-#endif
diff --git a/src/mainboard/intel/coffeelake_rvp/variants/cml_u/include/variant/hda_verb.h b/src/mainboard/intel/coffeelake_rvp/variants/whl_u/hda_verb.c
index 7233250105..01012dde1b 100644
--- a/src/mainboard/intel/coffeelake_rvp/variants/cml_u/include/variant/hda_verb.h
+++ b/src/mainboard/intel/coffeelake_rvp/variants/whl_u/hda_verb.c
@@ -1,8 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#ifndef HDA_VERB_H
-#define HDA_VERB_H
-
#include <device/azalia_device.h>
const u32 cim_verb_data[] = {
@@ -684,4 +681,3 @@ const u32 cim_verb_data[] = {
const u32 pc_beep_verbs[] = {
};
AZALIA_ARRAY_SIZES;
-#endif