ron minnich has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/81596?usp=email )
Change subject: riscv: remove test for OPENSBI_TEXT_START == linker address ......................................................................
riscv: remove test for OPENSBI_TEXT_START == linker address
In older OpenSBI, the opensbi linker address had to equal its location in cbfs. Since OpenSBI can now relocate itself, this restriction can be removed.
Change-Id: Ib16ae611ebaa9e532c86bc8a88e2352665e5b2fd Signed-off-by: Ronald G Minnich rminnich@gmail.com --- M src/arch/riscv/include/arch/memlayout.h 1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/81596/1
diff --git a/src/arch/riscv/include/arch/memlayout.h b/src/arch/riscv/include/arch/memlayout.h index 0c539d2..b100c41 100644 --- a/src/arch/riscv/include/arch/memlayout.h +++ b/src/arch/riscv/include/arch/memlayout.h @@ -22,8 +22,6 @@ #endif
#define OPENSBI(addr, size) \ - _ = ASSERT(addr == CONFIG_OPENSBI_TEXT_START, \ - "opensbi linker address must equal CONFIG_OPENSBI_TEXT_START"); \ REGION(opensbi, addr, size, 4K)
/* TODO: Need to add DMA_COHERENT region like on ARM? */