[coreboot] Fwd: DBE62 v3 strangeness since v3 rev 649
Marc Jones
Marc.Jones at amd.com
Tue Apr 29 17:44:54 CEST 2008
ron minnich wrote:
> If somebody could ack this I can close out the recent dbe62 problems.
>
> ron
>
>
Acked-by: Marc Jones <marc.jones at amd.com>
> ---------- Forwarded message ----------
> From: ron minnich <rminnich at gmail.com>
> Date: Fri, Apr 18, 2008 at 10:09 PM
> Subject: Re: DBE62 v3 strangeness since v3 rev 649
> To: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
> Cc: Coreboot <coreboot at coreboot.org>
>
>
> On Tue, Apr 15, 2008 at 2:52 PM, Carl-Daniel Hailfinger
>
> <c-d.hailfinger.devel.2006 at gmx.net> wrote:
>
>
>> Hi Ron,
> >
> > this one is for you: Your commit 649 had a few probably unintended side
> > effects:
> > - set_delay_control thinks zero DIMMs are installed because
> > SPD_PRIMARY_SDRAM_WIDTH does not exist in the SPD table.
> > - Due to that, the GLCP_DELAY_CONTROLS msr is set to 0.
> >
> > Please confirm that you really wanted these settings.
> >
>
> This mistake arose out of my attempt to clean up the namings of things
> in the original initram. It had names from two sources.
>
> This patch cleans up that error, and is my attempt at cleaning up the
> null pointer
> issue in device/device.c, which I actually don't understand.
>
> This patch allows me to almost boot a Plan 9 kernel from flash.
>
> ron
>
--
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors
More information about the coreboot
mailing list