From 98d4836105748906cfcdd67f4fb1818ff20b1508 Mon Sep 17 00:00:00 2001 From: Jesus Sanchez-Palencia Date: Thu, 9 Feb 2017 11:12:36 -0800 Subject: [PATCH] linker quark_se: Fix operator precedence bug Without this patch, the final linker script had a memory region defined as: RAM (wx) : ORIGIN = 0xA8006400, LENGTH = 55*1K - 0x20 +0x4 resulting on the wrong RAM length. Fix BSP_SHARED_RAM_SIZE definition so the sum is always processed first. Change-Id: I16b93adbaf27bb84b6e5d1ac433ef80f854b218c Signed-off-by: Jesus Sanchez-Palencia --- arch/x86/soc/intel_quark/quark_se/linker.ld | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/soc/intel_quark/quark_se/linker.ld b/arch/x86/soc/intel_quark/quark_se/linker.ld index 12549428494..7efc77095cc 100644 --- a/arch/x86/soc/intel_quark/quark_se/linker.ld +++ b/arch/x86/soc/intel_quark/quark_se/linker.ld @@ -25,7 +25,7 @@ /* total shared RAM size (restore info + gdt) */ #define BSP_SHARED_RAM_SIZE \ - CONFIG_BSP_SHARED_GDT_RAM_SIZE+CONFIG_BSP_SHARED_RESTORE_INFO_SIZE + (CONFIG_BSP_SHARED_GDT_RAM_SIZE+CONFIG_BSP_SHARED_RESTORE_INFO_SIZE) /* * When DEEP_SLEEP is not enabled, the shared memory can be re-used after boot. * Since the memory regions area are at fixed address locations, only the