diff options
author | Angel Pons <th3fanbus@gmail.com> | 2020-12-05 19:11:25 +0100 |
---|---|---|
committer | Hung-Te Lin <hungte@chromium.org> | 2020-12-17 13:52:59 +0000 |
commit | c8be0947f1d2665f9ddaca77c9739f55980551ac (patch) | |
tree | f6c2747d788639a887cd80a0f7f274dfcd2971cc | |
parent | b8602aa1842044f0801afc61de931f1733272ba5 (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.inc | 2 | ||||
-rw-r--r-- | src/mainboard/intel/coffeelake_rvp/hda_verb.c | 3 | ||||
-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 |