Patchwork build service results for r6367

login
register
about
Submitter Alexandru Gagniuc
Date 2011-02-16 15:03:12
Message ID <4D5BE730.7060204@gmail.com>
Download mbox | patch
Permalink /patch/2646/
State Accepted
Headers show

Comments

Alexandru Gagniuc - 2011-02-16 15:03:12
Fix build errors from r6367.

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
---
Belay my last patch. This patch fixes both types of errors that happen
in r6367. Sorry about this.

Alex

On 02/16/2011 04:33 PM, Alex G. wrote:
> This patch should add the missing file.
> 
> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
> ---
> 
> On 02/16/2011 04:21 PM, repository service wrote:
>> Dear coreboot readers!
>>
>> This is the automatic build system of coreboot.
>>
>> The developer "stuge" checked in revision 6367 to
>> the coreboot repository. This caused the following 
>> changes:
>>
>> Change Log:
>> Extended K8T890 driver to include the K8T800 and K8M800 northbridges
>>
>> The K8T800 is almost identical to the K8T800Pro, also added to this patch.
>> The K8T800_OLD is also defined, which is an older version of the K8T800,
>> but which has no driver and early HT code yet. Also extended the K8M890 VGA
>> driver to work for the K8M800 (not tested). According to the datasheet, the
>> K8T890 and K8T800 are similar enough to be able to use the same
>> initialization code. At least for the K8T800, this is sufficient to have
>> a working HT link with the CPU, and to initialise the V-Link to the
>> southbridge.
>>
>> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
>> Acked-by: Peter Stuge <peter@stuge.se>
>>
>>
>> Build Log:
>> Compilation of asus:a8v-e_deluxe has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=a8v-e_deluxe&vendor=asus&num=2
>> Compilation of asus:a8v-e_se has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=a8v-e_se&vendor=asus&num=2
>> Compilation of asus:m2v has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=m2v&vendor=asus&num=2
>> Compilation of asus:m2v-mx_se has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=m2v-mx_se&vendor=asus&num=2
>> Compilation of bcom:winnetp680 has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=winnetp680&vendor=bcom&num=2
>> Compilation of jetway:j7f24 has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=j7f24&vendor=jetway&num=2
>> Compilation of via:epia-cn has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=epia-cn&vendor=via&num=2
>> Compilation of via:pc2500e has been broken
>> See the error log at http://qa.coreboot.org/log_buildbrd.php?revision=6367&device=pc2500e&vendor=via&num=2
>>
>>
>> If something broke during this checkin please be a pain 
>> in stuge'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
>>
>>
>>
>
Peter Stuge - 2011-02-16 17:40:24
Alex G. wrote:
> Fix build errors from r6367.
> 
> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>

Acked-by: Peter Stuge <peter@stuge.se>

r6369

Patch

Index: src/southbridge/via/vt8237r/lpc.c
===================================================================
--- src/southbridge/via/vt8237r/lpc.c	(revision 6367)
+++ src/southbridge/via/vt8237r/lpc.c	(working copy)
@@ -300,7 +300,7 @@ 
 	pci_write_config8(dev, 0x48, 0x0c);
 #else
 	
-  #if CONFIG_SOUTHBRIDGE_VIA_K8T800
+  #if defined(CONFIG_SOUTHBRIDGE_VIA_K8T800)
 	/* It seems that when we pair with the K8T800, we need to disable
 	 * the A2 mask
 	 */
Index: src/southbridge/via/k8t890/k8x8xx.h
===================================================================
--- src/southbridge/via/k8t890/k8x8xx.h	(revision 0)
+++ src/southbridge/via/k8t890/k8x8xx.h	(revision 0)
@@ -0,0 +1,25 @@ 
+/*
+ * This file is part of the coreboot project.
+ * 
+ * Copyright (C) 2011  Alexandru Gagniuc <mr.nuke.me@gmail.com>
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#ifndef SOUTHBRIDGE_VIA_K8T890_K8X8XX_H
+#define SOUTHBRIDGE_VIA_K8T890_K8X8XX_H
+
+#include "k8t890.h"
+
+#endif//SOUTHBRIDGE_VIA_K8T890_K8X8XX_H
\ No newline at end of file