switch-coreboot/arch/x86/geodelx
Carl-Daniel Hailfinger 5a19770667 northbridge/amd/geodelx/raminit.c:auto_size_dimm() checks for the
mathematically impossible condition of a value being above and below the
specified range at the same time. Change it to check for out-of-range.
arch/x86/geodelx/geodelx.c:set_delay_control() is missing a break, it
will keep going and mess up DRAM timings.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>

Both changes look right to me.
Acked-by: Marc Jones <marc.jones@amd.com>


The raminit in v2 was fixed in r2899 | rminnich | 2007-10-26 with this
log:
> The lxraminit change fixes a bug (&& used instead of ||) [...]
> Signed-off-by: Ronald G. Minnich <rminnich@gmail.com>
> Acked-by: Peter Stuge <peter@stuge.se>


git-svn-id: svn://coreboot.org/repository/coreboot-v3@659 f3766cd6-281f-0410-b1cd-43a5c92072e9
2008-04-16 16:40:45 +00:00
..
cpu.c This started out as a trivial change and turned into a big change. This 2008-02-16 04:13:44 +00:00
geodelx.c northbridge/amd/geodelx/raminit.c:auto_size_dimm() checks for the 2008-04-16 16:40:45 +00:00
stage0.S Make printk() log to a buffer. 2008-02-13 15:48:37 +00:00
stage1.c This change moves the geodelxinit code from stage2 to stage1, which in 2008-02-04 16:16:16 +00:00