Attention is currently required from: Jérémy Compostella.

Alper Nebi Yasak has uploaded this change for review.

View Change

drivers/pc80/vga: Allow building for non-x86 architectures

The Cirrus driver uses the VGA support code for its initialization,
which in turn depends on port I/O functions to talk to the PCI device.
Previous commits implement these and fix build errors with VGA code on
more architectures, so it's possible to have things also work there to
an extent.

However, building VGA support code is restricted to x86 architectures in
its Makefile, so trying to build the Cirrus driver for e.g. ARM64
results in linker errors. Remove the Makefile restriction.

Change-Id: Ic3199fda500c51aa3261cd9eff9a5b0b052fd742
Signed-off-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
---
M src/drivers/pc80/vga/Makefile.mk
1 file changed, 0 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/64/82064/1
diff --git a/src/drivers/pc80/vga/Makefile.mk b/src/drivers/pc80/vga/Makefile.mk
index 63ec6ba..db103d8 100644
--- a/src/drivers/pc80/vga/Makefile.mk
+++ b/src/drivers/pc80/vga/Makefile.mk
@@ -1,7 +1,5 @@
## SPDX-License-Identifier: GPL-2.0-only

-ifeq ($(CONFIG_ARCH_X86),y)
-
romstage-$(CONFIG_ROMSTAGE_VGA) += vga_io.c
romstage-$(CONFIG_ROMSTAGE_VGA) += vga_palette.c
romstage-$(CONFIG_ROMSTAGE_VGA) += vga_font_8x16.c
@@ -11,5 +9,3 @@
ramstage-$(CONFIG_VGA) += vga_palette.c
ramstage-$(CONFIG_VGA) += vga_font_8x16.c
ramstage-$(CONFIG_VGA) += vga.c
-
-endif

To view, visit change 82064. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ic3199fda500c51aa3261cd9eff9a5b0b052fd742
Gerrit-Change-Number: 82064
Gerrit-PatchSet: 1
Gerrit-Owner: Alper Nebi Yasak <alpernebiyasak@gmail.com>
Gerrit-Reviewer: Jérémy Compostella <jeremy.compostella@intel.com>
Gerrit-Attention: Jérémy Compostella <jeremy.compostella@intel.com>
Gerrit-MessageType: newchange