On 18.03.2009 18:32, Mart Raudsepp wrote: > Ühel kenal päeval, N, 2008-10-23 kell 03:41, kirjutas Carl-Daniel > Hailfinger: > >> On 23.10.2008 01:38, ron minnich wrote: >> >>> Jens, I am very happy to see these contributions from Lippert. I hope >>> you or someone from this list can look at bringing them forward to v3 >>> as well. >>> >> This is the GeodeLX RAM speed calculation patch for v3. >> >> Changed RAM speed calculation to fix RAM modules getting rejected only >> due to integer rounding errors. Previously, the formula was: >> speed = 2 * (10000/spd_value) >> For spd_value=60 this means speed = 2 * 166 = 332, which is less than >> 333 and coreboot died saying RAM was incompatible. The new formula is: >> speed = 20000 / spd_value >> For spd_value=60, speed=333, which is fine. >> >> Signed-off-by: Jens Rottmann <[email protected]> >> Acked-by: Carl-Daniel Hailfinger <[email protected]> >> > > Tested on ThinCan DBE63 to fix the issue of 333 > 332 comparison for RAM > modules I had problems with before due to bailing out in the > overclocking check. > > Acked-by: Mart Raudsepp <[email protected]> >
Thanks, committed in r1157. Regards, Carl-Daniel -- http://www.hailfinger.org/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
