<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 8/13/10 9:50 AM, Qing Pei Wang wrote:
<blockquote
cite="mid:AANLkTi=i3CnL7k9CiiA8-U97CdDcXgdYsm6rUa4jZ+hU@mail.gmail.com"
type="cite">hi,<br>
the attach file added the support of Gigabyte MA785GMT
mainboard.<br>
the details of the hardware configuration can be found at<br>
<a moz-do-not-send="true"
href="http://www.gigabyte.com/products/product-page.aspx?pid=3478">http://www.gigabyte.com/products/product-page.aspx?pid=3478</a>
<br>
a breif configuration is:<br>
1. CPU:Support for AM3 processors: AMD PhenomTM II processor/ AMD
Athlon™ II processor<br>
2. North Bridge: AMD 785G<br>
3. South Bridge: AMD SB710 <br>
4: Super IO : ITE8718F<br clear="all">
<br>
The mainboard has two bios flashchip. Coreboot ROM should be
flashed into the M_BIOS(which means main bios).<br>
signed-off-by Wang Qing Pei <<a moz-do-not-send="true"
href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
</blockquote>
<br>
Hi Qing Pei!<br>
<br>
Thanks for this patch! It's great to have more boards in the tree.<br>
<br>
<blockquote type="cite">---
src/mainboard/gigabyte/ma785gmt/mb_sysconf.h (revision 0)<br>
+++ src/mainboard/gigabyte/ma785gmt/mb_sysconf.h (revision
0)<br>
<br>
</blockquote>
..<br>
<blockquote type="cite">+#ifndef MB_SYSCONF_H<br>
+<br>
+#define MB_SYSCONF_H<br>
+<br>
+struct mb_sysconf_t {<br>
+ u8 bus_isa;<br>
+ u8 bus_8132_0;<br>
+ u8 bus_8132_1;<br>
+ u8 bus_8132_2;<br>
+ u8 bus_8111_0;<br>
+ u8 bus_8111_1;<br>
+ u8 bus_8132a[31][3];<br>
+ u8 bus_8151[31][2];<br>
+<br>
+ u32 apicid_8111;<br>
+ u32 apicid_8132_1;<br>
+ u32 apicid_8132_2;<br>
+ u32 apicid_8132a[31][2];<br>
+ u32 sbdn3;<br>
+ u32 sbdn3a[31];<br>
+ u32 sbdn5[31];<br>
+ u32 bus_type[256];<br>
+};<br>
+<br>
+#endif<br>
</blockquote>
Can you check if this file is really needed?<br>
<br>
I would expect that anything that references AMD8132, AMD8111 and
AMD8151 is not needed on an AMD785/SB710 board?<br>
<br>
Also, a lot of files in the patch are exact copies of the AMD
Tilapia board. I counted 15 files where the only change from Tilapia
is the following:<br>
<br>
<blockquote type="cite">@@ -1,7 +1,7 @@<br>
/*<br>
* This file is part of the coreboot project.<br>
- * Copyright (C) 2010 Advanced Micro Devices, Inc.<br>
+ * Copyright (C) 2010 Wang Qing Pei <a class="moz-txt-link-rfc2396E" href="mailto:wangqingpei@gmail.com"><wangqingpei@gmail.com></a><br>
*<br>
* This program is free software; you can redistribute it and/or
modify<br>
* it under the terms of the GNU General Public License as
published by<br>
</blockquote>
<br>
Some more change the name from AMD Tilapia to Gigabyte GA785GMT in
addition to that.<br>
<br>
Those files should keep the AMD copyright in there I believe. The
ones that you didn't change should not have your copyright added.<br>
<br>
I think with above issues adressed we could commit this patch.<br>
<br>
Thanks,<br>
<br>
Stefan<br>
<br>
<br>
</body>
</html>