[coreboot] [PATCH] Heterogenous dual channel support, part 2
Marc Jones
marcj303 at gmail.com
Wed Jan 21 02:55:27 CET 2009
On Tue, Jan 20, 2009 at 5:32 AM, Carl-Daniel Hailfinger
<c-d.hailfinger.devel.2006 at gmx.net> wrote:
> Marc,
>
> I believe you are the person who is most proficient in K8 RAM init (no
> offense to anyone else intended). Could you take a look at the patch and
> give me a few comments?
>
>
I think that the code flow is equivalent. Maybe add a comment about
the return values for the functions. The case statement could have a
default to catch an unexpected return. Maybe make the hw_error the
default?
> if (latencies == 0) {
That should probably cause a disable dimm.
I think that spd_set_memclk() has the same CAS checking so you could
do the fix their as well. Search for SPD_CAS_LAT.
Marc
More information about the coreboot
mailing list