hi marc,<br>thanks a lot for your kindly review.<br>i update the patch followed by your suggestion.<br>there are some issues with all of the fam10 problems. patch will be send later.<br><br><div class="gmail_quote">On Tue, Jul 20, 2010 at 5:19 AM, Marc Jones <span dir="ltr"><<a href="mailto:marcj303@gmail.com">marcj303@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">On Thu, Jul 15, 2010 at 2:01 AM, Qing Pei Wang <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>> wrote:<br>

> hi all,<br>
>    in order to make the code much clear, i update the patch which sent<br>
> yesterday to make all messages fit for "PA78VM5".<br>
><br>
> Signed-off-by Wang Qing Pei <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
><br>
<br>
<br>
</div>Qing Pei,<br>
<br>
Can you break up this patch into smaller parts, fintek sio and<br>
mainboard? That would make it easier to review.<br>
<br>
Also, Can you give a summary of what the code supports. What works and<br>
what doesn't. What about internal graphics, pci e slots etc?<br>
<br>
<br>
Here are a few items to look at:<br>
<br>
Do you need the mb_sysconf.h file? I don't think that the 8132 and<br>
8111 stuff is needed on your platform.<br>
<br>
In devicetree.cb, You don't need multiple 18.0 devices.<br>
<br>
romstage.c<br>
+//used by incoherent_ht<br>
+#define FAM10_SCAN_PCI_BUS 0<br>
+#define FAM10_ALLOCATE_IO_RANGE 0<br>
<br>
Are these used? I think that they may be outdated. Maybe make a patch<br>
for all fam10 platforms?<br>
<br>
+//#include "northbridge/amd/amdht/ht_wrapper.c"<br>
+//#include "northbridge/amd/amdfam10/raminit_amdmct.c"<br>
+//#include "cpu/amd/model_10xxx/fidvid.c"<br>
+//#include "spd_addr.h"<br>
<br>
Remove these old includes that are commented out.<br>
<br>
Remove spd_addr.h file since it isn't used.<br>
<br>
chip.h<br>
+//     int fixup_scsi;<br>
+//     int fixup_vga;<br>
<br>
Remove these old lines. I don't think that they are used for anything<br>
in 780/700.<br>
<br>
Marc<br>
<font color="#888888"><br>
<br>
--<br>
<a href="http://se-eng.com" target="_blank">http://se-eng.com</a><br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Wang Qing Pei <br>Phone: 86+13426369984<br>