Arthur Heymans has uploaded this change for review.

View Change

include/rules.h: Allow .data section on non-XIP stages

This allows platform that don't run stage XIP to have a .data section.

Change-Id: I40a651398dff374bf6ba75471b07662de96fb434
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
---
M src/include/rules.h
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/48157/1
diff --git a/src/include/rules.h b/src/include/rules.h
index f503914..a7fc77f 100644
--- a/src/include/rules.h
+++ b/src/include/rules.h
@@ -265,7 +265,7 @@
#define ENV_STAGE_XIP (ENV_CACHE_AS_RAM && \
!((ENV_ROMSTAGE || ENV_SEPARATE_VERSTAGE) && CONFIG(NO_XIP_EARLY_STAGES)))
/* No .data sections with execute-in-place from ROM. */
-#define ENV_STAGE_HAS_DATA_SECTION !ENV_CACHE_AS_RAM
+#define ENV_STAGE_HAS_DATA_SECTION !ENV_STAGE_XIP
#else
/* Both .data and .bss, sometimes SRAM not DRAM. */
#define ENV_STAGE_HAS_DATA_SECTION 1

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I40a651398dff374bf6ba75471b07662de96fb434
Gerrit-Change-Number: 48157
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-MessageType: newchange