Patchwork build service results for r4399

login
register
about
Submitter coreboot information
Date 2009-07-05 16:43:40
Message ID <E1MNUoW-0001gm-QY@ra.coresystems.de>
Download mbox | patch
Permalink /patch/6/
State Not Applicable
Headers show

Comments

coreboot information - 2009-07-05 16:43:40
Dear coreboot readers!

This is the automatic build system of coreboot.

The developer "uwe" checked in revision 4399 to
the coreboot repository. This caused the following 
changes:

Change Log:
Enable onboard VGA on the MS-6178 (i810 chipset) board (trivial).

Tested on hardware with the patch from r4398 and works fine as soon
as Linux boots (no VGA in FILO for some reason, will investigate).

In order to make the 'i810.vga' VGA blob from the vendor BIOS work
you have to make the check for PCI device ID mismatches non-fatal
(for now) in the src/devices/pci_rom.c file like this:


The reason is that the VGA blob thinks the proper VGA device ID is 0x7123
whereas it really is 0x7121 on hardware. There are multiple ways to work
around this (there have been many discussions in the past), we'll see which
method will be used in future...

Note: This has been tested against r4393 only for now to make sure there
are no problems because of the recent resource allocator changes, see
http://www.coreboot.org/pipermail/coreboot/2009-July/050486.html.
Tests with trunk will follow.

Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>



Build Log:
Compilation of asus:mew-am is still broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=4399&device=mew-am&vendor=asus&num=2
Compilation of asus:mew-vm is still broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=4399&device=mew-vm&vendor=asus&num=2
Compilation of motorola:sandpointx3_altimus_mpc7410 is still broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=4399&device=sandpointx3_altimus_mpc7410&vendor=motorola&num=2
Compilation of msi:ms6178 has been fixed
Compilation of nec:powermate2000 is still broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=4399&device=powermate2000&vendor=nec&num=2
Compilation of via:epia-m700 is still broken
See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=4399&device=epia-m700&vendor=via&num=2


If something broke during this checkin please be a pain 
in uwe's neck until the issue is fixed.

If this issue is not fixed within 24h the revision should 
be backed out.

   Best regards,
     coreboot automatic build system

Patch

Index: src/devices/pci_rom.c
===================================================================
--- src/devices/pci_rom.c       (Revision 4393)
+++ src/devices/pci_rom.c       (Arbeitskopie)
@@ -87,7 +87,7 @@ 
        if (dev->vendor != rom_data->vendor || dev->device != rom_data->device) {
                printk_err("Device or Vendor ID mismatch Vendor %04x, Device %04x\n",
                           rom_data->vendor, rom_data->device);
-               return NULL;
+               // return NULL;
        }

        printk_spew("PCI ROM Image,  Class Code %04x%02x, Code Type %02x\n",