Marshall Dawson has uploaded this change for review.

View Change

program.ld: Qualify .bss linking with ENV_EARLY_RAM

Systems with ENV_EARLY_RAM need to position .bss in a consistent
location across early stages, and not necessarily within the
program region. Skip the default .bss section for these designs.
A subsequent patch will add the early linker script.

Change-Id: Icde1cfd02611b8cce634aa76f5c192d5b637cf44
Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com>
---
M src/lib/program.ld
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/37487/1
diff --git a/src/lib/program.ld b/src/lib/program.ld
index a9d4e48..1c8f99e 100644
--- a/src/lib/program.ld
+++ b/src/lib/program.ld
@@ -125,7 +125,7 @@
}
#endif

-#if ENV_STAGE_HAS_BSS_SECTION && !ENV_CACHE_AS_RAM
+#if (ENV_STAGE_HAS_BSS_SECTION && !ENV_CACHE_AS_RAM && !ENV_EARLY_RAM)
.bss . : {
. = ALIGN(ARCH_POINTER_ALIGN_SIZE);
_bss = .;

To view, visit change 37487. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icde1cfd02611b8cce634aa76f5c192d5b637cf44
Gerrit-Change-Number: 37487
Gerrit-PatchSet: 1
Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-MessageType: newchange