[coreboot] [PATCH] Convert all Intel i810 boards to CAR
Uwe Hermann
uwe at hermann-uwe.de
Wed Oct 13 21:14:40 CEST 2010
On Wed, Oct 13, 2010 at 09:00:00PM +0200, Uwe Hermann wrote:
> But you're right, the CACHE_BASE variables differ, but I guess they both
> work (0xc0000 works, I can test the other later just to make sure).
Actually, scratch that. It seems intel/car/cache_as_ram.inc hardcodes
the base to:
#define CacheBase (0xd0000 - CacheSize)
I.e. the DCACHE_RAM_BASE option is never used for this CAR
implementation. We have multiple possibilities to fix this:
- Drop DCACHE_RAM_BASE for these CPUs/sockets, and leave in the
hardcoded CacheBase, which means all of them will use the same base.
- Or, actually use DCACHE_RAM_BASE in the cache_as_ram.inc file,
which allows us to use different bases per-CPU or per-socket.
No idea if it makes sense to be able to select the base for these CPUs
at all (?)
Uwe.
--
http://hermann-uwe.de | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
More information about the coreboot
mailing list