Hi Achin,
On 4 January 2017 at 15:50, achin.gupta@arm.com wrote:
From: Achin Gupta achin.gupta@arm.com
This patch reduces the size of the DRAM available to UEFI so that the freed memory can be used to load the Standalone MM image in the secure world.
What is "MM"?
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Achin Gupta achin.gupta@arm.com
Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc index cf4b803..82acd87 100644 --- a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +++ b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc @@ -108,8 +108,13 @@ gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0
# System Memory (2GB - 16MB of Trusted DRAM at the top of the 32bit address space)
- gArmTokenSpaceGuid.PcdSystemMemoryBase|0x80000000
+!if $(ARM_STANDALONE_MM_ENABLE) == FALSE
Why did you not make it check for the new flag being set and defaulting to the new behaviour if the platform hasn't set it to false, rather than change the behaviour only if the platform specifically enabled the new feature?
gArmTokenSpaceGuid.PcdSystemMemorySize|0x7F000000 +!else
- gArmTokenSpaceGuid.PcdSystemMemorySize|0x74000000
+!endif
# Size of the region used by UEFI in permanent memory (Reserved 64MB) gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000 -- 1.9.1
Linaro-uefi mailing list Linaro-uefi@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-uefi