<font size="4">thanks for your kindly suggestion.<br>the blog post is really useful for me.<br clear="all"></font><font size="4"><br style="font-family:georgia,serif"><br style="font-family:georgia,serif"><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 Wed, Sep 14, 2011 at 2:47 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;">
Am Mittwoch, den 14.09.2011, 14:12 +0800 schrieb QingPei Wang:<br>
<div class="im">> the content of 010000bf.c comes from<br>
><br>
> src/vendorcode/amd/agesa/f12/Proc/CPU/Family/0x10/RevE/F10MicrocodePatch010000bf.c<br>
><br>
> but F10MicrocodePatch010000bf.c is used by amd agesa, could not be used<br>
> directly. i<br>
> port the code. just mentioned in the commit message that where is comes from<br>
<br>
</div>Thank you for the explanation. Please add such elaborate explanations to<br>
the commit message. Mark Brown has a nice blog post about this [1].<br>
<br>
<br>
Thanks,<br>
<br>
Paul<br>
<br>
<br>
[1] <a href="http://www.sirena.org.uk/log/2011/09/09/making-patches-easy-to-review/" target="_blank">http://www.sirena.org.uk/log/2011/09/09/making-patches-easy-to-review/</a><br>
<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>