Am Dienstag, den 29.09.2009, 08:21 -0600 schrieb Myles Watson: > Index: src/mainboard/intel/xe7501devkit/Makefile.inc > =================================================================== > --- src/mainboard/intel/xe7501devkit/Makefile.inc (Revision 4683) > +++ src/mainboard/intel/xe7501devkit/Makefile.inc (Arbeitskopie) > @@ -1,7 +1,7 @@ > ROMCCFLAGS := -mcpu=p4 > obj-$(CONFIG_HAVE_ACPI_TABLES) += acpi_tables.o > ifeq ($(CONFIG_PCI_ROM_RUN),y) > - ifeq ($(CONFIG_PCI_ROM_RUN),y) > + ifeq ($(CONFIG_CONSOLE_VGA),y) > I think this should be CONFIG_VGA_ROM_RUN. CONFIG_CONSOLE_VGA is for the > Coreboot console messages. It's the same thing Config.lb is doing. If this is broken, Config.lb should be fixed, too.
> This whole part can be removed since it includes the binary vga.rom which > isn't in the tree. CBFS can help us remove no_vgarom.S, vgarom.S, and the > reference to _vgarom_start in the device tree. Yes, the vgarom.S hack is really horrible, but it's a bit of work to get rid of it - in both Config.lb and this one. > > Signed-off-by: Patrick Georgi <[email protected]> > Acked-by: Myles Watson <[email protected]> Thanks, r4685 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
