[coreboot-gerrit] New patch to review for coreboot: 348fcb8 intel/wtm2/i915.c: Do not include non-existent "hda.h"

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Sun Sep 21 11:41:28 CEST 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6945

-gerrit

commit 348fcb8391f171029a2820600612da9c357031c5
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sun Sep 21 11:28:39 2014 +0200

    intel/wtm2/i915.c: Do not include non-existent "hda.h"
    
    Commit 75c83870 (azalia: Shrink boilerplate) [1] removed the header
    file `src/mainboard/intel/wtm2/hda_verb.h`, which is still included in
    `src/mainboard/intel/wtm2/i915.c`, causing the following build error,
    when native graphics initialization is enabled.
    
            CC         mainboard/intel/wtm2/i915.ramstage.o
        src/mainboard/intel/wtm2/i915.c:34:22: fatal error: hda_verb.h: No such file or directory
    
    This was not caught, as native graphics initialization is not enabled
    for the build tests.
    
    It turns out that the array `mainboard_cim_verb_data` is not used in
    `src/mainboard/intel/wtm2/hda_verb.h`, so fix the problem by removing
    the inclusion.
    
    [1] http://review.coreboot.org/6840
    
    Change-Id: Ic902581c6809a1069e169cc874678146a24d75f3
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/mainboard/intel/wtm2/i915.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/mainboard/intel/wtm2/i915.c b/src/mainboard/intel/wtm2/i915.c
index 64fa6b1..f542a13 100644
--- a/src/mainboard/intel/wtm2/i915.c
+++ b/src/mainboard/intel/wtm2/i915.c
@@ -31,7 +31,6 @@
 #include <arch/io.h>
 #include <arch/interrupt.h>
 #include <boot/coreboot_tables.h>
-#include "hda_verb.h"
 #include <smbios.h>
 #include <device/pci.h>
 #include <ec/google/chromeec/ec.h>



More information about the coreboot-gerrit mailing list