intel/baytrail: use fmap information for code caching
Instead of using CBFS_SIZE from Kconfig, use values generated from fmap. While at it, make sure that the cached region size is a power of two. fmap_config is also added to cpu_incs-y, but that doesn't hurt (except for some miniscule increase in compile time) because it's #if-guarded. The upside is that dependencies are tracked properly. Change-Id: I03a919e1381ca3d0e972780b2c7d76c590aaa994 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Reviewed-on: https://review.coreboot.org/14573 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
parent
58a150a1a3
commit
06a0b567ce
|
@ -1,4 +1,5 @@
|
|||
cpu_incs-y += $(src)/soc/intel/baytrail/romstage/cache_as_ram.inc
|
||||
cpu_incs-y += $(obj)/fmap_config.h
|
||||
romstage-y += romstage.c
|
||||
romstage-y += raminit.c
|
||||
romstage-$(CONFIG_ENABLE_BUILTIN_COM1) += uart.c
|
||||
|
|
|
@ -19,6 +19,8 @@
|
|||
#include <cpu/x86/post_code.h>
|
||||
#include <cbmem.h>
|
||||
|
||||
#include "fmap_config.h"
|
||||
|
||||
/* The full cache-as-ram size includes the cache-as-ram portion from coreboot
|
||||
* and the space used by the reference code. These 2 values combined should
|
||||
* be a power of 2 because the MTRR setup assumes that. */
|
||||
|
@ -27,7 +29,7 @@
|
|||
#define CACHE_AS_RAM_BASE CONFIG_DCACHE_RAM_BASE
|
||||
|
||||
/* Cache all of CBFS just below 4GiB as Write-Protect type. */
|
||||
#define CODE_CACHE_SIZE (CONFIG_CBFS_SIZE)
|
||||
#define CODE_CACHE_SIZE _ALIGN_UP_POW2(___FMAP__COREBOOT_SIZE)
|
||||
#define CODE_CACHE_BASE (-CODE_CACHE_SIZE)
|
||||
#define CODE_CACHE_MASK (~(CODE_CACHE_SIZE - 1))
|
||||
#define CPU_PHYSMASK_HI ((1 << (CONFIG_CPU_ADDR_BITS - 32)) - 1)
|
||||
|
|
Loading…
Reference in New Issue