[coreboot] [PATCH]Jetway PA78VM5
Marc Jones
marcj303 at gmail.com
Mon Jul 19 23:19:25 CEST 2010
On Thu, Jul 15, 2010 at 2:01 AM, Qing Pei Wang <wangqingpei at gmail.com> wrote:
> hi all,
> in order to make the code much clear, i update the patch which sent
> yesterday to make all messages fit for "PA78VM5".
>
> Signed-off-by Wang Qing Pei <wangqingpei at gmail.com>
>
Qing Pei,
Can you break up this patch into smaller parts, fintek sio and
mainboard? That would make it easier to review.
Also, Can you give a summary of what the code supports. What works and
what doesn't. What about internal graphics, pci e slots etc?
Here are a few items to look at:
Do you need the mb_sysconf.h file? I don't think that the 8132 and
8111 stuff is needed on your platform.
In devicetree.cb, You don't need multiple 18.0 devices.
romstage.c
+//used by incoherent_ht
+#define FAM10_SCAN_PCI_BUS 0
+#define FAM10_ALLOCATE_IO_RANGE 0
Are these used? I think that they may be outdated. Maybe make a patch
for all fam10 platforms?
+//#include "northbridge/amd/amdht/ht_wrapper.c"
+//#include "northbridge/amd/amdfam10/raminit_amdmct.c"
+//#include "cpu/amd/model_10xxx/fidvid.c"
+//#include "spd_addr.h"
Remove these old includes that are commented out.
Remove spd_addr.h file since it isn't used.
chip.h
+// int fixup_scsi;
+// int fixup_vga;
Remove these old lines. I don't think that they are used for anything
in 780/700.
Marc
--
http://se-eng.com
More information about the coreboot
mailing list