Remove XIP_ROM_BASE
The base is now calculated automatically, and all mentions of that config option were typical anyway (4GB - XIP_ROM_SIZE). Change-Id: Icdf908dc043719f3810f7b5b85ad9938f362ea40 Signed-off-by: Patrick Georgi <patrick@georgi-clan.de> Reviewed-on: http://review.coreboot.org/366 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
parent
36c04e8a5c
commit
784544b934
|
@ -40,10 +40,6 @@ config CDB
|
||||||
hex
|
hex
|
||||||
default 0x18
|
default 0x18
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -57,11 +57,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_AGESA_FAMILY12
|
depends on CPU_AMD_AGESA_FAMILY12
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_AGESA_FAMILY12
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -57,11 +57,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_AGESA_FAMILY14
|
depends on CPU_AMD_AGESA_FAMILY14
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_AGESA_FAMILY14
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -279,7 +279,7 @@ clear_fixed_var_mtrr_out:
|
||||||
movl $(((CONFIG_RAMTOP) + TOP_MEM_MASK) & ~TOP_MEM_MASK) , %eax
|
movl $(((CONFIG_RAMTOP) + TOP_MEM_MASK) & ~TOP_MEM_MASK) , %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)
|
#if CONFIG_XIP_ROM_SIZE
|
||||||
|
|
||||||
/* Enable write base caching so we can do execute in place (XIP)
|
/* Enable write base caching so we can do execute in place (XIP)
|
||||||
* on the flash ROM.
|
* on the flash ROM.
|
||||||
|
@ -302,7 +302,7 @@ clear_fixed_var_mtrr_out:
|
||||||
wbcache_post_fam10_setup:
|
wbcache_post_fam10_setup:
|
||||||
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
#endif /* CONFIG_XIP_ROM_SIZE && CONFIG_XIP_ROM_BASE */
|
#endif /* CONFIG_XIP_ROM_SIZE */
|
||||||
|
|
||||||
/* Set the default memory type and enable fixed and variable MTRRs. */
|
/* Set the default memory type and enable fixed and variable MTRRs. */
|
||||||
movl $MTRRdefType_MSR, %ecx
|
movl $MTRRdefType_MSR, %ecx
|
||||||
|
|
|
@ -30,11 +30,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_SOCKET_AM2R2
|
depends on CPU_AMD_SOCKET_AM2R2
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_SOCKET_AM2R2
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -30,11 +30,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_SOCKET_AM3
|
depends on CPU_AMD_SOCKET_AM3
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_SOCKET_AM3
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -30,11 +30,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_SOCKET_ASB2
|
depends on CPU_AMD_SOCKET_ASB2
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_SOCKET_ASB2
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -30,11 +30,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_SOCKET_C32
|
depends on CPU_AMD_SOCKET_C32
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_SOCKET_C32
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -29,11 +29,6 @@ config CDB
|
||||||
default 0x18
|
default 0x18
|
||||||
depends on CPU_AMD_SOCKET_F_1207
|
depends on CPU_AMD_SOCKET_F_1207
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xfff80000
|
|
||||||
depends on CPU_AMD_SOCKET_F_1207
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x80000
|
default 0x80000
|
||||||
|
|
|
@ -229,7 +229,7 @@ clear_fixed_var_mtrr_out:
|
||||||
simplemask CacheSize, 0
|
simplemask CacheSize, 0
|
||||||
wrmsr
|
wrmsr
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)
|
#if CONFIG_XIP_ROM_SIZE
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Enable write base caching so we can do execute in place (XIP)
|
* Enable write base caching so we can do execute in place (XIP)
|
||||||
|
@ -250,7 +250,7 @@ clear_fixed_var_mtrr_out:
|
||||||
movl $0x0000000f, %edx
|
movl $0x0000000f, %edx
|
||||||
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
#endif /* CONFIG_XIP_ROM_SIZE && CONFIG_XIP_ROM_BASE */
|
#endif /* CONFIG_XIP_ROM_SIZE */
|
||||||
|
|
||||||
/* Enable cache. */
|
/* Enable cache. */
|
||||||
movl %cr0, %eax
|
movl %cr0, %eax
|
||||||
|
|
|
@ -98,7 +98,7 @@ clear_mtrrs:
|
||||||
orl $(1 << 30), %eax
|
orl $(1 << 30), %eax
|
||||||
movl %eax, %cr0
|
movl %eax, %cr0
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)
|
#if CONFIG_XIP_ROM_SIZE
|
||||||
/* Enable cache for our code in Flash because we do XIP here */
|
/* Enable cache for our code in Flash because we do XIP here */
|
||||||
movl $MTRRphysBase_MSR(1), %ecx
|
movl $MTRRphysBase_MSR(1), %ecx
|
||||||
xorl %edx, %edx
|
xorl %edx, %edx
|
||||||
|
@ -115,7 +115,7 @@ clear_mtrrs:
|
||||||
xorl %edx, %edx
|
xorl %edx, %edx
|
||||||
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | 0x800), %eax
|
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | 0x800), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
#endif /* CONFIG_XIP_ROM_SIZE && CONFIG_XIP_ROM_BASE */
|
#endif /* CONFIG_XIP_ROM_SIZE */
|
||||||
|
|
||||||
/* Enable cache. */
|
/* Enable cache. */
|
||||||
movl %cr0, %eax
|
movl %cr0, %eax
|
||||||
|
|
|
@ -98,7 +98,7 @@ clear_mtrrs:
|
||||||
orl $(1 << 30), %eax
|
orl $(1 << 30), %eax
|
||||||
movl %eax, %cr0
|
movl %eax, %cr0
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)
|
#if CONFIG_XIP_ROM_SIZE
|
||||||
/* Enable cache for our code in Flash because we do XIP here */
|
/* Enable cache for our code in Flash because we do XIP here */
|
||||||
movl $MTRRphysBase_MSR(1), %ecx
|
movl $MTRRphysBase_MSR(1), %ecx
|
||||||
xorl %edx, %edx
|
xorl %edx, %edx
|
||||||
|
@ -115,7 +115,7 @@ clear_mtrrs:
|
||||||
movl $0x0000000f, %edx
|
movl $0x0000000f, %edx
|
||||||
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
#endif /* CONFIG_XIP_ROM_SIZE && CONFIG_XIP_ROM_BASE */
|
#endif /* CONFIG_XIP_ROM_SIZE */
|
||||||
|
|
||||||
/* Enable cache. */
|
/* Enable cache. */
|
||||||
movl %cr0, %eax
|
movl %cr0, %eax
|
||||||
|
|
|
@ -105,7 +105,7 @@ clear_mtrrs:
|
||||||
orl $(1 << 30), %eax
|
orl $(1 << 30), %eax
|
||||||
movl %eax, %cr0
|
movl %eax, %cr0
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && defined(CONFIG_XIP_ROM_BASE)
|
#if CONFIG_XIP_ROM_SIZE
|
||||||
/* Enable cache for our code in Flash because we do XIP here */
|
/* Enable cache for our code in Flash because we do XIP here */
|
||||||
movl $MTRRphysBase_MSR(1), %ecx
|
movl $MTRRphysBase_MSR(1), %ecx
|
||||||
xorl %edx, %edx
|
xorl %edx, %edx
|
||||||
|
@ -122,7 +122,7 @@ clear_mtrrs:
|
||||||
movl $0x0000000f, %edx
|
movl $0x0000000f, %edx
|
||||||
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
movl $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
#endif /* CONFIG_XIP_ROM_SIZE && CONFIG_XIP_ROM_BASE */
|
#endif /* CONFIG_XIP_ROM_SIZE */
|
||||||
|
|
||||||
/* Enable cache. */
|
/* Enable cache. */
|
||||||
movl %cr0, %eax
|
movl %cr0, %eax
|
||||||
|
|
|
@ -240,7 +240,7 @@ testok:
|
||||||
movl $(~(CONFIG_RAMTOP - CONFIG_RAMBASE - 1) | MTRRphysMaskValid), %eax
|
movl $(~(CONFIG_RAMTOP - CONFIG_RAMBASE - 1) | MTRRphysMaskValid), %eax
|
||||||
wrmsr
|
wrmsr
|
||||||
|
|
||||||
/* Cache XIP_ROM_BASE-SIZE to speedup coreboot code. */
|
/* Cache XIP_ROM area to speedup coreboot code. */
|
||||||
movl $MTRRphysBase_MSR(1), %ecx
|
movl $MTRRphysBase_MSR(1), %ecx
|
||||||
xorl %edx, %edx
|
xorl %edx, %edx
|
||||||
/*
|
/*
|
||||||
|
|
|
@ -27,10 +27,6 @@ config TSC_CALIBRATE_WITH_IO
|
||||||
bool
|
bool
|
||||||
default n
|
default n
|
||||||
|
|
||||||
config XIP_ROM_BASE
|
|
||||||
hex
|
|
||||||
default 0xffff0000
|
|
||||||
|
|
||||||
config XIP_ROM_SIZE
|
config XIP_ROM_SIZE
|
||||||
hex
|
hex
|
||||||
default 0x10000
|
default 0x10000
|
||||||
|
|
|
@ -45,23 +45,13 @@ void set_var_mtrr_resource(void *gp, struct device *dev, struct resource *res);
|
||||||
void x86_setup_fixed_mtrrs(void);
|
void x86_setup_fixed_mtrrs(void);
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
/* Validate CONFIG_XIP_ROM_SIZE and CONFIG_XIP_ROM_BASE */
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && !defined(CONFIG_XIP_ROM_BASE)
|
|
||||||
# error "CONFIG_XIP_ROM_SIZE without CONFIG_XIP_ROM_BASE"
|
|
||||||
#endif
|
|
||||||
#if defined(CONFIG_XIP_ROM_BASE) && !defined(CONFIG_XIP_ROM_SIZE)
|
|
||||||
# error "CONFIG_XIP_ROM_BASE without CONFIG_XIP_ROM_SIZE"
|
|
||||||
#endif
|
|
||||||
#if !defined(CONFIG_RAMTOP)
|
#if !defined(CONFIG_RAMTOP)
|
||||||
# error "CONFIG_RAMTOP not defined"
|
# error "CONFIG_RAMTOP not defined"
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && ((CONFIG_XIP_ROM_SIZE & (CONFIG_XIP_ROM_SIZE -1)) != 0)
|
#if ((CONFIG_XIP_ROM_SIZE & (CONFIG_XIP_ROM_SIZE -1)) != 0)
|
||||||
# error "CONFIG_XIP_ROM_SIZE is not a power of 2"
|
# error "CONFIG_XIP_ROM_SIZE is not a power of 2"
|
||||||
#endif
|
#endif
|
||||||
#if defined(CONFIG_XIP_ROM_SIZE) && ((CONFIG_XIP_ROM_BASE % CONFIG_XIP_ROM_SIZE) != 0)
|
|
||||||
# error "CONFIG_XIP_ROM_BASE is not a multiple of CONFIG_XIP_ROM_SIZE"
|
|
||||||
#endif
|
|
||||||
|
|
||||||
#if (CONFIG_RAMTOP & (CONFIG_RAMTOP - 1)) != 0
|
#if (CONFIG_RAMTOP & (CONFIG_RAMTOP - 1)) != 0
|
||||||
# error "CONFIG_RAMTOP must be a power of 2"
|
# error "CONFIG_RAMTOP must be a power of 2"
|
||||||
|
|
|
@ -21,8 +21,3 @@ See also this thread: http://www.coreboot.org/pipermail/coreboot/2009-September/
|
||||||
|
|
||||||
Ward, 2009-09-22
|
Ward, 2009-09-22
|
||||||
|
|
||||||
mansoor@iwavesystems.com said, about the last issue:
|
|
||||||
|
|
||||||
Try enabling CONFIG_XIP_ROM_BASE. It solved the same problem for me in my board.
|
|
||||||
|
|
||||||
So, that's a todo.
|
|
||||||
|
|
Loading…
Reference in New Issue