ArmPlatformPkg: remove ArmPlatformInitializeSystemMemory
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 30 Nov 2017 14:00:28 +0000 (14:00 +0000)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 30 Nov 2017 16:08:33 +0000 (16:08 +0000)
The function ArmPlatformInitializeSystemMemory() is defined by
ArmPlatformLib, but is only ever called when using the PrePeiCore
flavor of the startup code. Also, none of the remaining upstream
platforms actually implement anything in that function in the first
place. So let's just remove it altogether.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
ArmPlatformPkg/ArmPlatformPkg.dec
ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibCTA15-A7/CTA15-A7.c
ArmPlatformPkg/ArmVExpressPkg/Library/ArmVExpressLibRTSM/RTSM.c
ArmPlatformPkg/Include/Library/ArmPlatformLib.h
ArmPlatformPkg/Library/ArmPlatformLibNull/ArmPlatformLibNull.c
ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.c
ArmPlatformPkg/MemoryInitPei/MemoryInitPeim.inf
BeagleBoardPkg/Library/BeagleBoardLib/BeagleBoard.c

index e282e76..9d47e45 100644 (file)
@@ -48,7 +48,6 @@
   gArmPlatformTokenSpaceGuid.PcdNorFlashRemapping|FALSE|BOOLEAN|0x00000012\r
 \r
   gArmPlatformTokenSpaceGuid.PcdStandalone|TRUE|BOOLEAN|0x00000001\r
-  gArmPlatformTokenSpaceGuid.PcdSystemMemoryInitializeInSec|FALSE|BOOLEAN|0x00000002\r
   gArmPlatformTokenSpaceGuid.PcdSendSgiToBringUpSecondaryCores|FALSE|BOOLEAN|0x00000004\r
 \r
   gArmPlatformTokenSpaceGuid.PcdNorFlashCheckBlockLocked|FALSE|BOOLEAN|0x0000003C\r
index 93f4d82..a6ddd1b 100644 (file)
@@ -149,19 +149,6 @@ ArmPlatformInitialize (
   return RETURN_SUCCESS;\r
 }\r
 \r
-/**\r
-  Initialize the system (or sometimes called permanent) memory\r
-\r
-  This memory is generally represented by the DRAM.\r
-\r
-**/\r
-VOID\r
-ArmPlatformInitializeSystemMemory (\r
-  VOID\r
-  )\r
-{\r
-}\r
-\r
 EFI_STATUS\r
 PrePeiCoreGetMpCoreInfo (\r
   OUT UINTN                   *CoreCount,\r
index 11dd7ff..7760e82 100644 (file)
@@ -155,20 +155,6 @@ ArmPlatformInitialize (
   return RETURN_SUCCESS;\r
 }\r
 \r
-/**\r
-  Initialize the system (or sometimes called permanent) memory\r
-\r
-  This memory is generally represented by the DRAM.\r
-\r
-**/\r
-VOID\r
-ArmPlatformInitializeSystemMemory (\r
-  VOID\r
-  )\r
-{\r
-  // Nothing to do here\r
-}\r
-\r
 EFI_STATUS\r
 PrePeiCoreGetMpCoreInfo (\r
   OUT UINTN                   *CoreCount,\r
index c74d535..3ad4c16 100644 (file)
@@ -111,17 +111,6 @@ ArmPlatformInitialize (
   IN  UINTN                     MpId\r
   );\r
 \r
-/**\r
-  Initialize the system (or sometimes called permanent) memory\r
-\r
-  This memory is generally represented by the DRAM.\r
-\r
-**/\r
-VOID\r
-ArmPlatformInitializeSystemMemory (\r
-  VOID\r
-  );\r
-\r
 /**\r
   Return the Virtual Memory Map of your platform\r
 \r
index f80a276..038e955 100644 (file)
@@ -106,20 +106,6 @@ ArmPlatformInitialize (
   return RETURN_SUCCESS;\r
 }\r
 \r
-/**\r
-  Initialize the system (or sometimes called permanent) memory\r
-\r
-  This memory is generally represented by the DRAM.\r
-\r
-**/\r
-VOID\r
-ArmPlatformInitializeSystemMemory (\r
-  VOID\r
-  )\r
-{\r
-  //TODO: Implement me\r
-}\r
-\r
 EFI_STATUS\r
 PrePeiCoreGetMpCoreInfo (\r
   OUT UINTN                   *CoreCount,\r
index e7880d3..389a2e6 100644 (file)
@@ -103,14 +103,6 @@ InitializeMemory (
 \r
   DEBUG ((EFI_D_LOAD | EFI_D_INFO, "Memory Init PEIM Loaded\n"));\r
 \r
-  //\r
-  // Initialize the System Memory (DRAM)\r
-  //\r
-  if (!FeaturePcdGet (PcdSystemMemoryInitializeInSec)) {\r
-    // In case the DRAM has not been initialized by the secure firmware\r
-    ArmPlatformInitializeSystemMemory ();\r
-  }\r
-\r
   // Ensure PcdSystemMemorySize has been set\r
   ASSERT (PcdGet64 (PcdSystemMemorySize) != 0);\r
   ASSERT (PcdGet64 (PcdSystemMemoryBase) < (UINT64)MAX_ADDRESS);\r
index 2c14a9c..e9b4ea1 100644 (file)
@@ -49,7 +49,6 @@
 \r
 [FeaturePcd]\r
   gEmbeddedTokenSpaceGuid.PcdPrePiProduceMemoryTypeInformationHob\r
-  gArmPlatformTokenSpaceGuid.PcdSystemMemoryInitializeInSec\r
 \r
 [FixedPcd]\r
   gArmTokenSpaceGuid.PcdFdBaseAddress\r
index 3b02440..d8eec72 100644 (file)
@@ -101,20 +101,6 @@ ArmPlatformInitialize (
   return RETURN_SUCCESS;\r
 }\r
 \r
-/**\r
-  Initialize the system (or sometimes called permanent) memory\r
-\r
-  This memory is generally represented by the DRAM.\r
-\r
-**/\r
-VOID\r
-ArmPlatformInitializeSystemMemory (\r
-  VOID\r
-  )\r
-{\r
-  // We do not need to initialize the System Memory on RTSM\r
-}\r
-\r
 VOID\r
 ArmPlatformGetPlatformPpiList (\r
   OUT UINTN                   *PpiListSize,\r