]> git.proxmox.com Git - mirror_edk2.git/commitdiff
BeagleBoardPkg: switch to use MdeModulePkg ResetSystemLib
authorLeif Lindholm <leif.lindholm@linaro.org>
Mon, 3 Jul 2017 15:05:04 +0000 (16:05 +0100)
committerLeif Lindholm <leif.lindholm@linaro.org>
Mon, 3 Jul 2017 21:30:22 +0000 (22:30 +0100)
The BeagleBoard port used EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf
for its reset handling. With the arrival
MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf

As part of this, change BeagleBoardPkg/Library/ResetSystemLib to be an
implementation of ResetSystemLib instead of the previous
EfiResetSystemLib.

Wire all reset variants to ResetCold, except for ResetShutdown and
EnterS3WithImmediateWake, which return immediately.

Note: this ResetSystemLib never supported being called after
ExitBootservices, and this shortcoming is not addressed here.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
BeagleBoardPkg/BeagleBoardPkg.dsc
BeagleBoardPkg/BeagleBoardPkg.fdf
BeagleBoardPkg/Library/ResetSystemLib/ResetSystemLib.c
BeagleBoardPkg/Library/ResetSystemLib/ResetSystemLib.inf

index 90d6af444df0c76899230b10a767f737aacc7863..b22f814a28a4aa1126213114897b908c9123753c 100644 (file)
@@ -56,7 +56,7 @@
   BaseLib|MdePkg/Library/BaseLib/BaseLib.inf\r
   BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf\r
 \r
-  EfiResetSystemLib|BeagleBoardPkg/Library/ResetSystemLib/ResetSystemLib.inf\r
+  ResetSystemLib|BeagleBoardPkg/Library/ResetSystemLib/ResetSystemLib.inf\r
 \r
   PciLib|MdePkg/Library/BasePciLibCf8/BasePciLibCf8.inf\r
   PerformanceLib|MdePkg/Library/BasePerformanceLibNull/BasePerformanceLibNull.inf\r
 #      SerialPortLib|ArmPkg/Library/SemiHostingSerialPortLib/SemiHostingSerialPortLib.inf\r
 #  }\r
 \r
-  EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf\r
+  MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf\r
   EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf\r
   EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf\r
 \r
index f140d9019cac701e74be3023f2c4a6bb5a64eafc..c9c6afd714859ac8264c7fb68e6f59532c48155f 100644 (file)
@@ -115,7 +115,7 @@ FvNameGuid         = d0dd3e90-343d-4cb3-8f69-772214989282
   INF MdeModulePkg/Universal/SerialDxe/SerialDxe.inf\r
   INF MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf\r
 \r
-  INF EmbeddedPkg/ResetRuntimeDxe/ResetRuntimeDxe.inf\r
+  INF MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystemRuntimeDxe.inf\r
   INF EmbeddedPkg/RealTimeClockRuntimeDxe/RealTimeClockRuntimeDxe.inf\r
   INF EmbeddedPkg/MetronomeDxe/MetronomeDxe.inf\r
 \r
index 6b7879b02bd68bee209e4946b31b6066a0c58ce5..7bc6c6c329a8a5189ef5aba73dfad842f6947674 100644 (file)
@@ -2,6 +2,7 @@
   Do a generic Cold Reset for OMAP3550 and BeagleBoard specific Warm reset\r
 \r
   Copyright (c) 2008 - 2010, Apple Inc. All rights reserved.<BR>\r
+  Copyright (c) 2017, Linaro Ltd. All rights reserved.<BR>\r
 \r
   This program and the accompanying materials\r
   are licensed and made available under the terms and conditions of the BSD License\r
 \r
 #include <Uefi.h>\r
 \r
-#include <Library/ArmLib.h>\r
-#include <Library/CacheMaintenanceLib.h>\r
-#include <Library/MemoryAllocationLib.h>\r
 #include <Library/IoLib.h>\r
-#include <Library/PcdLib.h>\r
-#include <Library/DebugLib.h>\r
-#include <Library/UefiBootServicesTableLib.h>\r
+#include <Library/ResetSystemLib.h>\r
 \r
 #include <Omap3530/Omap3530.h>\r
 \r
+/**\r
+  This function causes a system-wide reset (cold reset), in which\r
+  all circuitry within the system returns to its initial state. This type of\r
+  reset is asynchronous to system operation and operates without regard to\r
+  cycle boundaries.\r
 \r
+  If this function returns, it means that the system does not support cold\r
+  reset.\r
+**/\r
 VOID\r
-ShutdownEfi (\r
+EFIAPI\r
+ResetCold (\r
   VOID\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  UINTN                   MemoryMapSize;\r
-  EFI_MEMORY_DESCRIPTOR   *MemoryMap;\r
-  UINTN                   MapKey;\r
-  UINTN                   DescriptorSize;\r
-  UINTN                   DescriptorVersion;\r
-  UINTN                   Pages;\r
-\r
-  MemoryMap = NULL;\r
-  MemoryMapSize = 0;\r
-  do {\r
-    Status = gBS->GetMemoryMap (\r
-                    &MemoryMapSize,\r
-                    MemoryMap,\r
-                    &MapKey,\r
-                    &DescriptorSize,\r
-                    &DescriptorVersion\r
-                    );\r
-    if (Status == EFI_BUFFER_TOO_SMALL) {\r
-\r
-      Pages = EFI_SIZE_TO_PAGES (MemoryMapSize) + 1;\r
-      MemoryMap = AllocatePages (Pages);\r
-\r
-      //\r
-      // Get System MemoryMap\r
-      //\r
-      Status = gBS->GetMemoryMap (\r
-                      &MemoryMapSize,\r
-                      MemoryMap,\r
-                      &MapKey,\r
-                      &DescriptorSize,\r
-                      &DescriptorVersion\r
-                      );\r
-      // Don't do anything between the GetMemoryMap() and ExitBootServices()\r
-      if (!EFI_ERROR (Status)) {\r
-        Status = gBS->ExitBootServices (gImageHandle, MapKey);\r
-        if (EFI_ERROR (Status)) {\r
-          FreePages (MemoryMap, Pages);\r
-          MemoryMap = NULL;\r
-          MemoryMapSize = 0;\r
-        }\r
-      }\r
-    }\r
-  } while (EFI_ERROR (Status));\r
-\r
-  //Clean and invalidate caches.\r
-  WriteBackInvalidateDataCache();\r
-  InvalidateInstructionCache();\r
-\r
-  //Turning off Caches and MMU\r
-  ArmDisableDataCache ();\r
-  ArmDisableInstructionCache ();\r
-  ArmDisableMmu ();\r
+  //Perform cold reset of the system.\r
+  MmioOr32 (PRM_RSTCTRL, RST_DPLL3);\r
+  while ((MmioRead32(PRM_RSTST) & GLOBAL_COLD_RST) != 0x1);\r
 }\r
 \r
-typedef\r
+/**\r
+  This function causes a system-wide initialization (warm reset), in which all\r
+  processors are set to their initial state. Pending cycles are not corrupted.\r
+\r
+  If this function returns, it means that the system does not support warm\r
+  reset.\r
+**/\r
 VOID\r
-(EFIAPI *CALL_STUB)(\r
+EFIAPI\r
+ResetWarm (\r
   VOID\r
-);\r
-\r
+  )\r
+{\r
+  ResetCold ();\r
+}\r
 \r
 /**\r
-  Resets the entire platform.\r
-\r
-  @param  ResetType             The type of reset to perform.\r
-  @param  ResetStatus           The status code for the reset.\r
-  @param  DataSize              The size, in bytes, of WatchdogData.\r
-  @param  ResetData             For a ResetType of EfiResetCold, EfiResetWarm, or\r
-                                EfiResetShutdown the data buffer starts with a Null-terminated\r
-                                Unicode string, optionally followed by additional binary data.\r
+  This function causes the system to enter a power state equivalent\r
+  to the ACPI G2/S5 or G3 states.\r
 \r
+  If this function returns, it means that the system does not support shut down\r
+  reset.\r
 **/\r
-EFI_STATUS\r
+VOID\r
 EFIAPI\r
-LibResetSystem (\r
-  IN EFI_RESET_TYPE   ResetType,\r
-  IN EFI_STATUS       ResetStatus,\r
-  IN UINTN            DataSize,\r
-  IN CHAR16           *ResetData OPTIONAL\r
+ResetShutdown (\r
+  VOID\r
   )\r
 {\r
-  CALL_STUB   StartOfFv;\r
-\r
-  if (ResetData != NULL) {\r
-    DEBUG((EFI_D_ERROR, "%s", ResetData));\r
-  }\r
-\r
-  ShutdownEfi ();\r
-\r
-  switch (ResetType) {\r
-  case EfiResetWarm:\r
-    //Perform warm reset of the system by jumping to the begining of the FV\r
-    StartOfFv = (CALL_STUB)(UINTN)PcdGet64 (PcdFvBaseAddress);\r
-    StartOfFv ();\r
-    break;\r
-  case EfiResetCold:\r
-  case EfiResetShutdown:\r
-  default:\r
-    //Perform cold reset of the system.\r
-    MmioOr32 (PRM_RSTCTRL, RST_DPLL3);\r
-    while ((MmioRead32(PRM_RSTST) & GLOBAL_COLD_RST) != 0x1);\r
-    break;\r
-  }\r
-\r
-  // If the reset didn't work, return an error.\r
-  ASSERT (FALSE);\r
-  return EFI_DEVICE_ERROR;\r
+  // not implemented\r
 }\r
 \r
-\r
-\r
 /**\r
-  Initialize any infrastructure required for LibResetSystem () to function.\r
-\r
-  @param  ImageHandle   The firmware allocated handle for the EFI image.\r
-  @param  SystemTable   A pointer to the EFI System Table.\r
-\r
-  @retval EFI_SUCCESS   The constructor always returns EFI_SUCCESS.\r
+  This function causes the system to enter S3 and then wake up immediately.\r
 \r
+  If this function returns, it means that the system does not support S3\r
+  feature.\r
 **/\r
-EFI_STATUS\r
+VOID\r
 EFIAPI\r
-LibInitializeResetSystem (\r
-  IN EFI_HANDLE        ImageHandle,\r
-  IN EFI_SYSTEM_TABLE  *SystemTable\r
+EnterS3WithImmediateWake (\r
+  VOID\r
   )\r
 {\r
-  return EFI_SUCCESS;\r
+  // not implemented\r
 }\r
 \r
+/**\r
+  This function causes a systemwide reset. The exact type of the reset is\r
+  defined by the EFI_GUID that follows the Null-terminated Unicode string passed\r
+  into ResetData. If the platform does not recognize the EFI_GUID in ResetData\r
+  the platform must pick a supported reset type to perform.The platform may\r
+  optionally log the parameters from any non-normal reset that occurs.\r
+\r
+  @param[in]  DataSize   The size, in bytes, of ResetData.\r
+  @param[in]  ResetData  The data buffer starts with a Null-terminated string,\r
+                         followed by the EFI_GUID.\r
+**/\r
+VOID\r
+EFIAPI\r
+ResetPlatformSpecific (\r
+  IN UINTN   DataSize,\r
+  IN VOID    *ResetData\r
+  )\r
+{\r
+  ResetCold ();\r
+}\r
index 64638cb62cb84bbe4f7613856f93bfc2b4e3ccd1..b313cad967edb6f0df8fabb7362afad5907cf2c6 100644 (file)
@@ -2,6 +2,7 @@
 # Reset System lib to make it easy to port new platforms\r
 #\r
 # Copyright (c) 2008, Apple Inc. All rights reserved.<BR>\r
+# Copyright (c) 2017, Linaro Ltd. All rights reserved.<BR>\r
 #\r
 #  This program and the accompanying materials\r
 #  are licensed and made available under the terms and conditions of the BSD License\r
   FILE_GUID                      = 781371a2-3fdd-41d4-96a1-7b34cbc9e895\r
   MODULE_TYPE                    = BASE\r
   VERSION_STRING                 = 1.0\r
-  LIBRARY_CLASS                  = EfiResetSystemLib\r
+  LIBRARY_CLASS                  = ResetSystemLib\r
 \r
 \r
 [Sources.common]\r
   ResetSystemLib.c\r
 \r
 [Packages]\r
-  MdePkg/MdePkg.dec\r
   ArmPkg/ArmPkg.dec\r
-  EmbeddedPkg/EmbeddedPkg.dec\r
+  MdeModulePkg/MdeModulePkg.dec\r
+  MdePkg/MdePkg.dec\r
   Omap35xxPkg/Omap35xxPkg.dec\r
 \r
 [Pcd.common]\r
   gArmTokenSpaceGuid.PcdCpuResetAddress\r
-  gEmbeddedTokenSpaceGuid.PcdEmbeddedFdBaseAddress\r
 \r
 [LibraryClasses]\r
-  DebugLib\r
-  ArmLib\r
-  CacheMaintenanceLib\r
-  MemoryAllocationLib\r
-  UefiRuntimeServicesTableLib\r
-  TimerLib\r
-  UefiLib\r
-  UefiBootServicesTableLib\r
+  IoLib\r
 \r
 [Pcd]\r
   gArmTokenSpaceGuid.PcdFvBaseAddress\r