<font style="font-family: tahoma,sans-serif;" size="2">the content of 010000bf.c comes from <br clear="all"></font><font style="font-family: tahoma,sans-serif;" size="2"><br></font><font size="2"><span style="font-family: tahoma,sans-serif;">src/vendorcode/amd/agesa/f12/Proc/CPU/Family/0x10/RevE/F10MicrocodePatch010000bf.c</span><br style="font-family: tahoma,sans-serif;">
</font><font style="font-family: tahoma,sans-serif;" size="2"><br>but </font><font size="2"><span style="font-family: tahoma,sans-serif;">F10MicrocodePatch010000bf.c is used by amd agesa, could not be used directly. i </span><br style="font-family: tahoma,sans-serif;">
<span style="font-family: tahoma,sans-serif;">port the code. just mentioned in the commit message that where is comes from</span></font><br><font size="4"><br><br style="font-family:georgia,serif"><span style="font-family:georgia,serif">Best wishes</span><br style="font-family:georgia,serif">
<span style="font-family:georgia,serif">QingPei Wang</span><br style="font-family:georgia,serif"><span style="font-family:georgia,serif">Phone: 86+018930528086</span></font><br>
<br><br><div class="gmail_quote">On Tue, Sep 13, 2011 at 6:18 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 QuingPei,<br>
<br>
<br>
thank you for updating the patches.<br>
<br>
Am Dienstag, den 13.09.2011, 12:05 +0200 schrieb QingPei Wang:<br>
<div class="im">> QingPei Wang (<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>) just uploaded a new patch set to gerrit, which you can find at <a href="http://review.coreboot.org/202" target="_blank">http://review.coreboot.org/202</a><br>

><br>
> -gerrit<br>
><br>
> commit b0cd5ca9e6a0d7d46caab6fedf491e00f70dc532<br>
> Author: QingPei Wang <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
> Date:   Tue Sep 13 18:04:22 2011 +0800<br>
><br>
>     Add AMD Family 10h PH-EO support<br>
><br>
>     the patch file comes from<br>
<br>
</div>1. Could you start sentences in the commit message body with a capital<br>
letter please?<br>
<div class="im"><br>
>     src/vendorcode/amd/agesa/f12/Proc/CPU/Family/0x10/RevE<br>
>     /F10MicrocodePatch010000bf.c<br>
<br>
</div>2. I do not understand the commit message. What do you mean with patch<br>
file? In this patch you are only adding a header file.<br>
<div class="im"><br>
>     Change-Id: If701c8a908edf1c486665d3ce4df65da0f65c802<br>
>     Signed-off-by: QingPei Wang <<a href="mailto:wangqingpei@gmail.com">wangqingpei@gmail.com</a>><br>
> ---<br>
>  src/cpu/amd/model_10xxx/mc_patch_010000bf.h |  165 +++++++++++++++++++++++++++<br>
>  1 files changed, 165 insertions(+), 0 deletions(-)<br>
<br>
</div>[…]<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>