[coreboot] [PATCH]es 7 assorted fixes

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Mar 19 12:35:54 CET 2009

Hi Jens,

can you please check if all of your patches have been applied? I want to
make sure no patches are lost.



On 21.10.2008 11:18, Jens Rottmann wrote:
> route_irq15.diff (changes pirq_routing.c):
> Fixes a off-by-one error when routing the IRQs. This led to IRQ15 not
> getting assigned.
> autoboot_delay.diff (changes filo.c):
> Fixes compile error when AUTOBOOT_DELAY=0.
> dword_copy.diff (changes crt0.S.lb, cache_as_ram.inc (Geode LX)):
> Speed up copying coreboot to ram by using "movsl" instead of "movsb".
> Also use different console messages for copying and uncompressing, like
> it's already done in similar code in other places.
> speed_calc.diff (changes raminit.c (Geode LX)):
> 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.
> await_ide.diff (changes ide.c):
> Made await_ide(), which polls for an ide status change, check the status
> reg much more often. In my case this reduced the time spent in coreboot
> by 1.5 sec!
> The timeout values of course aren't changed, only the granularity. Also,
> I didn't see any udelay() implementation that looked like it couldn't
> cope with 10 us delays. (Most are written as for (...) inb(0x80) loops.)
> fs_arch.diff (changes ext2fs.c, fat.c):
> #if ARCH == 'i386' results in a compile error: character constant too
> long (or something alike). Changed it to
> #ifdef __i386
> I'm unsure if this is correct, though! Why didn't anyone hit this
> problem before? Is this some ROMCC-special?
> it8712_gpio.diff (changes superio.c (IT8712F)):
> Added the missing I/O resources for IT8712F GPIOs. Our boards need these
> e.g. to switch the com ports between RS232 and RS485.
> Signed-off-by: Jens Rottmann <JRottmann at LiPPERTEmbedded.de>


More information about the coreboot mailing list