On 4 January 2017 at 17:27, Ryan Harkin ryan.harkin@linaro.org wrote:
On 4 January 2017 at 17:22, Ryan Harkin ryan.harkin@linaro.org wrote:
Hi Achin,
On 4 January 2017 at 15:50, achin.gupta@arm.com wrote:
From: Achin Gupta achin.gupta@arm.com
This patch includes the DXE runtime driver that exports the EFI_MM_COMMUNICATION_PROTOCOL for the FVP platform. This driver is responsible for handling communication with the MM environment in the secure world. An implementation of the ARM SVC lib is also include in the FVP build.
s/include/included/
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Achin Gupta achin.gupta@arm.com
Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc | 8 ++++++++ Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.fdf | 5 +++++ 2 files changed, 13 insertions(+)
diff --git a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc index c76da5d..bc115e1 100644 --- a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc +++ b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.dsc @@ -40,6 +40,9 @@
[LibraryClasses.common] ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf +!if $(ARM_STANDALONE_MM_ENABLE) == TRUE
- ArmSvcLib|ArmPkg/Library/ArmSvcLib/ArmSvcLib.inf
+!endif ArmCpuLib|ArmPkg/Drivers/ArmCpuLib/ArmCortexAEMv8Lib/ArmCortexAEMv8Lib.inf ArmPlatformLib|ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibRTSM/ArmVExpressLib.inf ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf @@ -294,6 +297,11 @@
MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
+!if $(ARM_STANDALONE_MM_ENABLE) == TRUE
- # Standalone MM Support
- ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.inf
I've just spotted the .inf filename. I think "convention" tells us the .inf is the same name as the higher level directory name, ie. MmCommunicationDxe.inf. Not sure if it's anything more than common practice, but it's worth checking.
Actually, I can't find MmCommunicationDxe added in this patch series and it isn't already in the tree. I don't see any other reference to it in my inbox either... :-/
OK, so I've now read 0/4 and assume it's in the github tree... sorry about that.
+!endif
- # # Semi-hosting filesystem #
diff --git a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.fdf b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.fdf index 2ace4d8..0d9895b 100644 --- a/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.fdf +++ b/Platforms/ARM/VExpress/ArmVExpress-FVP-AArch64.fdf @@ -142,6 +142,11 @@ FvNameGuid = 87940482-fc81-41c3-87e6-399cf85ac8a0 # INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
+!if $(ARM_STANDALONE_MM_ENABLE) == TRUE
- # Standalone MM Support
- INF ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.inf
+!endif
- # # Platform Driver #
-- 1.9.1
Linaro-uefi mailing list Linaro-uefi@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-uefi