[coreboot] [patch][v2]:fam10 microcode

Ward Vandewege ward at gnu.org
Tue Jun 16 04:24:40 CEST 2009

Hi Marc,

On Mon, Jun 15, 2009 at 04:14:32PM -0600, Marc Jones wrote:
> Very good catch. I am suprised we didn't see this sooner.
> The bug is that the code was always updating the passed value to the
> next link offset even when it was on the requested link (cap_count). I
> think that your patch has a slight problem in that it skips the CapID
> and CapType check on the first link, link_no == 0.
> Attached is a slightly different fix (untested). I think that this
> function could be rewritten to be more clear but this is what you get
> when tying to keep code similarity when going from asm to C.....
> Please review and test.

This does not appear to fix the hang for my board. Here's a boot log with
lots of register dumping enabled:


And here's one without the very lengthy dumps:


I see very little difference with the older dumps.


Ward Vandewege <ward at gnu.org>

More information about the coreboot mailing list