<font size="4">hi Paul, <br clear="all"></font><div><font class="Apple-style-span" size="4"><span class="Apple-style-span" style="font-size: small; "><div>i tested by my opensuse 11.2 without X windows. the patch is applicable to the former two patches which i sent before.</div>
<div>And it does not depend on any other patches by AMD at this moment.</div><div>And i am trying to get familiar with gerrit at this moment. I just learned how to submit the patches, still does not know</div><div>how to use "git format-patch" things.</div>
<div><br></div><div>The attachment is a simple boot log with the board. I will cost more time to make this better.</div></span></font><br><br>Best wishes<br>Wang Qing Pei<br>Phone: 86+018930528086<br>
<br><br><div class="gmail_quote">On Tue, Sep 6, 2011 at 10:28 PM, Paul Menzel <span dir="ltr"><<a href="mailto:paulepanter@users.sourceforge.net">paulepanter@users.sourceforge.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Dear QingPei,<br>
<br>
<br>
Am Dienstag, den 06.09.2011, 15:12 +0200 schrieb QingPei Wang:<br>
<div class="im"><br>
> commit deb6f0f225bddd2fa786f94689555a21add12d13<br>
> Author: QingPei Wang <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
> Date:   Tue Sep 6 21:09:40 2011 +0800<br>
><br>
>     Add ASUS M5A88-V mainboard support<br>
><br>
>     it's a AMD 880+800 mainboard. I port the code<br>
>     based on the AMD reference code.<br>
<br>
</div>nice job! Thank you for doing that.<br>
<br>
Could you please add to the commit message what you have tested, i. e.<br>
what does work and what does not work? Additionally what operating<br>
system did you test? Is that patch applicable to master with your other<br>
two(?) patches or does it depend on other patches send by AMD not yet in<br>
master?<br></blockquote><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><br>
>     Change-Id: I9725ccdbb25365c4007621318efee80b131fec29<br>
>     Signed-off-by: QingPei Wang <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
> ---<br>
>  src/mainboard/asus/Kconfig                  |    3 +<br>
>  src/mainboard/asus/m5a88-v/Kconfig          |  107 ++<br>
>  src/mainboard/asus/m5a88-v/Makefile.inc     |   20 +<br>
>  src/mainboard/asus/m5a88-v/acpi/cpstate.asl |   75 ++<br>
>  src/mainboard/asus/m5a88-v/acpi/ide.asl     |  244 ++++<br>
>  src/mainboard/asus/m5a88-v/acpi/routing.asl |  398 ++++++<br>
>  src/mainboard/asus/m5a88-v/acpi/sata.asl    |  149 +++<br>
>  src/mainboard/asus/m5a88-v/acpi/usb.asl     |  161 +++<br>
>  src/mainboard/asus/m5a88-v/acpi_tables.c    |  274 ++++<br>
>  src/mainboard/asus/m5a88-v/chip.h           |   22 +<br>
>  src/mainboard/asus/m5a88-v/cmos.layout      |   98 ++<br>
>  src/mainboard/asus/m5a88-v/devicetree.cb    |  124 ++<br>
>  src/mainboard/asus/m5a88-v/dsdt.asl         | 1824 +++++++++++++++++++++++++++<br>
>  src/mainboard/asus/m5a88-v/fadt.c           |  201 +++<br>
>  src/mainboard/asus/m5a88-v/get_bus_conf.c   |  147 +++<br>
>  src/mainboard/asus/m5a88-v/irq_tables.c     |  111 ++<br>
>  src/mainboard/asus/m5a88-v/mainboard.c      |  150 +++<br>
>  src/mainboard/asus/m5a88-v/mb_sysconf.h     |   43 +<br>
>  src/mainboard/asus/m5a88-v/mptable.c        |  160 +++<br>
>  src/mainboard/asus/m5a88-v/pmio.c           |   53 +<br>
>  src/mainboard/asus/m5a88-v/pmio.h           |   33 +<br>
>  src/mainboard/asus/m5a88-v/reset.c          |   63 +<br>
>  src/mainboard/asus/m5a88-v/resourcemap.c    |  278 ++++<br>
>  src/mainboard/asus/m5a88-v/romstage.c       |  280 ++++<br>
>  24 files changed, 5018 insertions(+), 0 deletions(-)<br>
<br>
</div></div>[…]<br>
<br>
The diff is pretty big. I guess a lot of files have been copied over?<br>
Can Gerrit send smaller diffs by using `git format-patch -M -C` (?) or<br>
so?<br>
<br>
<br>
Thanks,<br>
<font color="#888888"><br>
Paul<br>
</font><br>--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br></blockquote></div><br></div>