]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/UefiLib/UefiLib.c
UefiCpuPkg: Move AsmRelocateApLoopStart from Mpfuncs.nasm to AmdSev.nasm
[mirror_edk2.git] / MdePkg / Library / UefiLib / UefiLib.c
index c2d143e821d5a8e7b30f3e741dba12e06f25a3b1..95d0319d3a337dece493909e6ea3543e3dfcf476 100644 (file)
@@ -10,7 +10,6 @@
 \r
 **/\r
 \r
-\r
 #include "UefiLibInternal.h"\r
 \r
 /**\r
@@ -54,10 +53,10 @@ CompareIso639LanguageCode (
   UINT32  Name1;\r
   UINT32  Name2;\r
 \r
-  Name1 = ReadUnaligned24 ((CONST UINT32 *) Language1);\r
-  Name2 = ReadUnaligned24 ((CONST UINT32 *) Language2);\r
+  Name1 = ReadUnaligned24 ((CONST UINT32 *)Language1);\r
+  Name2 = ReadUnaligned24 ((CONST UINT32 *)Language2);\r
 \r
-  return (BOOLEAN) (Name1 == Name2);\r
+  return (BOOLEAN)(Name1 == Name2);\r
 }\r
 \r
 /**\r
@@ -92,7 +91,7 @@ EfiGetSystemConfigurationTable (
   ASSERT (Table != NULL);\r
 \r
   SystemTable = gST;\r
-  *Table = NULL;\r
+  *Table      = NULL;\r
   for (Index = 0; Index < SystemTable->NumberOfTableEntries; Index++) {\r
     if (CompareGuid (TableGuid, &(SystemTable->ConfigurationTable[Index].VendorGuid))) {\r
       *Table = SystemTable->ConfigurationTable[Index].VendorTable;\r
@@ -132,7 +131,7 @@ EfiGetSystemConfigurationTable (
 **/\r
 EFI_EVENT\r
 EFIAPI\r
-EfiCreateProtocolNotifyEvent(\r
+EfiCreateProtocolNotifyEvent (\r
   IN  EFI_GUID          *ProtocolGuid,\r
   IN  EFI_TPL           NotifyTpl,\r
   IN  EFI_EVENT_NOTIFY  NotifyFunction,\r
@@ -226,7 +225,7 @@ EfiNamedEventListen (
                   EVT_NOTIFY_SIGNAL,\r
                   NotifyTpl,\r
                   NotifyFunction,\r
-                  (VOID *) NotifyContext,\r
+                  (VOID *)NotifyContext,\r
                   &Event\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
@@ -246,7 +245,7 @@ EfiNamedEventListen (
   //\r
 \r
   Status = gBS->RegisterProtocolNotify (\r
-                  (EFI_GUID *) Name,\r
+                  (EFI_GUID *)Name,\r
                   Event,\r
                   RegistrationLocal\r
                   );\r
@@ -277,12 +276,12 @@ EfiNamedEventSignal (
   EFI_STATUS  Status;\r
   EFI_HANDLE  Handle;\r
 \r
-  ASSERT(Name != NULL);\r
+  ASSERT (Name != NULL);\r
 \r
   Handle = NULL;\r
   Status = gBS->InstallProtocolInterface (\r
                   &Handle,\r
-                  (EFI_GUID *) Name,\r
+                  (EFI_GUID *)Name,\r
                   EFI_NATIVE_INTERFACE,\r
                   NULL\r
                   );\r
@@ -290,7 +289,7 @@ EfiNamedEventSignal (
 \r
   Status = gBS->UninstallProtocolInterface (\r
                   Handle,\r
-                  (EFI_GUID *) Name,\r
+                  (EFI_GUID *)Name,\r
                   NULL\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
@@ -313,11 +312,11 @@ EfiNamedEventSignal (
 EFI_STATUS\r
 EFIAPI\r
 EfiEventGroupSignal (\r
-  IN CONST EFI_GUID *EventGroup\r
+  IN CONST EFI_GUID  *EventGroup\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-  EFI_EVENT  Event;\r
+  EFI_STATUS  Status;\r
+  EFI_EVENT   Event;\r
 \r
   if (EventGroup == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -353,8 +352,8 @@ EfiEventGroupSignal (
 VOID\r
 EFIAPI\r
 EfiEventEmptyFunction (\r
-  IN EFI_EVENT              Event,\r
-  IN VOID                   *Context\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
   )\r
 {\r
 }\r
@@ -377,7 +376,7 @@ EfiGetCurrentTpl (
   VOID\r
   )\r
 {\r
-  EFI_TPL Tpl;\r
+  EFI_TPL  Tpl;\r
 \r
   Tpl = gBS->RaiseTPL (TPL_HIGH_LEVEL);\r
   gBS->RestoreTPL (Tpl);\r
@@ -385,7 +384,6 @@ EfiGetCurrentTpl (
   return Tpl;\r
 }\r
 \r
-\r
 /**\r
   Initializes a basic mutual exclusion lock.\r
 \r
@@ -406,15 +404,15 @@ EFI_LOCK *
 EFIAPI\r
 EfiInitializeLock (\r
   IN OUT EFI_LOCK  *Lock,\r
-  IN EFI_TPL        Priority\r
+  IN EFI_TPL       Priority\r
   )\r
 {\r
   ASSERT (Lock != NULL);\r
   ASSERT (Priority <= TPL_HIGH_LEVEL);\r
 \r
-  Lock->Tpl       = Priority;\r
-  Lock->OwnerTpl  = TPL_APPLICATION;\r
-  Lock->Lock      = EfiLockReleased ;\r
+  Lock->Tpl      = Priority;\r
+  Lock->OwnerTpl = TPL_APPLICATION;\r
+  Lock->Lock     = EfiLockReleased;\r
   return Lock;\r
 }\r
 \r
@@ -466,7 +464,6 @@ EfiAcquireLockOrFail (
   IN EFI_LOCK  *Lock\r
   )\r
 {\r
-\r
   ASSERT (Lock != NULL);\r
   ASSERT (Lock->Lock != EfiLockUninitialized);\r
 \r
@@ -503,7 +500,7 @@ EfiReleaseLock (
   IN EFI_LOCK  *Lock\r
   )\r
 {\r
-  EFI_TPL Tpl;\r
+  EFI_TPL  Tpl;\r
 \r
   ASSERT (Lock != NULL);\r
   ASSERT (Lock->Lock == EfiLockAcquired);\r
@@ -541,19 +538,19 @@ EfiReleaseLock (
 EFI_STATUS\r
 EFIAPI\r
 EfiTestManagedDevice (\r
-  IN CONST EFI_HANDLE       ControllerHandle,\r
-  IN CONST EFI_HANDLE       DriverBindingHandle,\r
-  IN CONST EFI_GUID         *ProtocolGuid\r
+  IN CONST EFI_HANDLE  ControllerHandle,\r
+  IN CONST EFI_HANDLE  DriverBindingHandle,\r
+  IN CONST EFI_GUID    *ProtocolGuid\r
   )\r
 {\r
-  EFI_STATUS     Status;\r
-  VOID           *ManagedInterface;\r
+  EFI_STATUS  Status;\r
+  VOID        *ManagedInterface;\r
 \r
   ASSERT (ProtocolGuid != NULL);\r
 \r
   Status = gBS->OpenProtocol (\r
                   ControllerHandle,\r
-                  (EFI_GUID *) ProtocolGuid,\r
+                  (EFI_GUID *)ProtocolGuid,\r
                   &ManagedInterface,\r
                   DriverBindingHandle,\r
                   ControllerHandle,\r
@@ -562,7 +559,7 @@ EfiTestManagedDevice (
   if (!EFI_ERROR (Status)) {\r
     gBS->CloseProtocol (\r
            ControllerHandle,\r
-           (EFI_GUID *) ProtocolGuid,\r
+           (EFI_GUID *)ProtocolGuid,\r
            DriverBindingHandle,\r
            ControllerHandle\r
            );\r
@@ -598,15 +595,15 @@ EfiTestManagedDevice (
 EFI_STATUS\r
 EFIAPI\r
 EfiTestChildHandle (\r
-  IN CONST EFI_HANDLE       ControllerHandle,\r
-  IN CONST EFI_HANDLE       ChildHandle,\r
-  IN CONST EFI_GUID         *ProtocolGuid\r
+  IN CONST EFI_HANDLE  ControllerHandle,\r
+  IN CONST EFI_HANDLE  ChildHandle,\r
+  IN CONST EFI_GUID    *ProtocolGuid\r
   )\r
 {\r
-  EFI_STATUS                            Status;\r
-  EFI_OPEN_PROTOCOL_INFORMATION_ENTRY   *OpenInfoBuffer;\r
-  UINTN                                 EntryCount;\r
-  UINTN                                 Index;\r
+  EFI_STATUS                           Status;\r
+  EFI_OPEN_PROTOCOL_INFORMATION_ENTRY  *OpenInfoBuffer;\r
+  UINTN                                EntryCount;\r
+  UINTN                                Index;\r
 \r
   ASSERT (ProtocolGuid != NULL);\r
 \r
@@ -616,7 +613,7 @@ EfiTestChildHandle (
   //\r
   Status = gBS->OpenProtocolInformation (\r
                   ControllerHandle,\r
-                  (EFI_GUID *) ProtocolGuid,\r
+                  (EFI_GUID *)ProtocolGuid,\r
                   &OpenInfoBuffer,\r
                   &EntryCount\r
                   );\r
@@ -630,7 +627,8 @@ EfiTestChildHandle (
   Status = EFI_UNSUPPORTED;\r
   for (Index = 0; Index < EntryCount; Index++) {\r
     if ((OpenInfoBuffer[Index].ControllerHandle == ChildHandle) &&\r
-        (OpenInfoBuffer[Index].Attributes & EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER) != 0) {\r
+        ((OpenInfoBuffer[Index].Attributes & EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER) != 0))\r
+    {\r
       Status = EFI_SUCCESS;\r
       break;\r
     }\r
@@ -653,18 +651,23 @@ EfiTestChildHandle (
 EFI_STATUS\r
 EFIAPI\r
 IsLanguageSupported (\r
-  IN CONST CHAR8 *SupportedLanguages,\r
-  IN CONST CHAR8 *TargetLanguage\r
+  IN CONST CHAR8  *SupportedLanguages,\r
+  IN CONST CHAR8  *TargetLanguage\r
   )\r
 {\r
-  UINTN Index;\r
+  UINTN  Index;\r
+\r
   while (*SupportedLanguages != 0) {\r
-    for (Index = 0; SupportedLanguages[Index] != 0 && SupportedLanguages[Index] != ';'; Index++);\r
-    if ((AsciiStrnCmp(SupportedLanguages, TargetLanguage, Index) == 0) && (TargetLanguage[Index] == 0)) {\r
+    for (Index = 0; SupportedLanguages[Index] != 0 && SupportedLanguages[Index] != ';'; Index++) {\r
+    }\r
+\r
+    if ((AsciiStrnCmp (SupportedLanguages, TargetLanguage, Index) == 0) && (TargetLanguage[Index] == 0)) {\r
       return EFI_SUCCESS;\r
     }\r
+\r
     SupportedLanguages += Index;\r
-    for (; *SupportedLanguages != 0 && *SupportedLanguages == ';'; SupportedLanguages++);\r
+    for ( ; *SupportedLanguages != 0 && *SupportedLanguages == ';'; SupportedLanguages++) {\r
+    }\r
   }\r
 \r
   return EFI_UNSUPPORTED;\r
@@ -712,7 +715,7 @@ LookupUnicodeString (
   //\r
   // Make sure the parameters are valid\r
   //\r
-  if (Language == NULL || UnicodeString == NULL) {\r
+  if ((Language == NULL) || (UnicodeString == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -720,7 +723,7 @@ LookupUnicodeString (
   // If there are no supported languages, or the Unicode String Table is empty, then the\r
   // Unicode String specified by Language is not supported by this Unicode String Table\r
   //\r
-  if (SupportedLanguages == NULL || UnicodeStringTable == NULL) {\r
+  if ((SupportedLanguages == NULL) || (UnicodeStringTable == NULL)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -729,13 +732,11 @@ LookupUnicodeString (
   //\r
   while (*SupportedLanguages != 0) {\r
     if (CompareIso639LanguageCode (Language, SupportedLanguages)) {\r
-\r
       //\r
       // Search the Unicode String Table for the matching Language specifier\r
       //\r
       while (UnicodeStringTable->Language != NULL) {\r
         if (CompareIso639LanguageCode (Language, UnicodeStringTable->Language)) {\r
-\r
           //\r
           // A matching string was found, so return it\r
           //\r
@@ -755,8 +756,6 @@ LookupUnicodeString (
   return EFI_UNSUPPORTED;\r
 }\r
 \r
-\r
-\r
 /**\r
   This function looks up a Unicode string in UnicodeStringTable.\r
 \r
@@ -807,14 +806,14 @@ LookupUnicodeString2 (
   IN BOOLEAN                         Iso639Language\r
   )\r
 {\r
-  BOOLEAN   Found;\r
-  UINTN     Index;\r
-  CHAR8     *LanguageString;\r
+  BOOLEAN  Found;\r
+  UINTN    Index;\r
+  CHAR8    *LanguageString;\r
 \r
   //\r
   // Make sure the parameters are valid\r
   //\r
-  if (Language == NULL || UnicodeString == NULL) {\r
+  if ((Language == NULL) || (UnicodeString == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -822,7 +821,7 @@ LookupUnicodeString2 (
   // If there are no supported languages, or the Unicode String Table is empty, then the\r
   // Unicode String specified by Language is not supported by this Unicode String Table\r
   //\r
-  if (SupportedLanguages == NULL || UnicodeStringTable == NULL) {\r
+  if ((SupportedLanguages == NULL) || (UnicodeStringTable == NULL)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -836,13 +835,13 @@ LookupUnicodeString2 (
         Found = TRUE;\r
         break;\r
       }\r
+\r
       SupportedLanguages += 3;\r
     }\r
   } else {\r
-    Found = !IsLanguageSupported(SupportedLanguages, Language);\r
+    Found = !IsLanguageSupported (SupportedLanguages, Language);\r
   }\r
 \r
-\r
   //\r
   // If Language is not a member of SupportedLanguages, then return EFI_UNSUPPORTED\r
   //\r
@@ -856,21 +855,25 @@ LookupUnicodeString2 (
   while (UnicodeStringTable->Language != NULL) {\r
     LanguageString = UnicodeStringTable->Language;\r
     while (0 != *LanguageString) {\r
-      for (Index = 0 ;LanguageString[Index] != 0 && LanguageString[Index] != ';'; Index++);\r
-      if (AsciiStrnCmp(LanguageString, Language, Index) == 0) {\r
+      for (Index = 0; LanguageString[Index] != 0 && LanguageString[Index] != ';'; Index++) {\r
+      }\r
+\r
+      if (AsciiStrnCmp (LanguageString, Language, Index) == 0) {\r
         *UnicodeString = UnicodeStringTable->UnicodeString;\r
         return EFI_SUCCESS;\r
       }\r
+\r
       LanguageString += Index;\r
-      for (Index = 0 ;LanguageString[Index] != 0 && LanguageString[Index] == ';'; Index++);\r
+      for (Index = 0; LanguageString[Index] != 0 && LanguageString[Index] == ';'; Index++) {\r
+      }\r
     }\r
+\r
     UnicodeStringTable++;\r
   }\r
 \r
   return EFI_UNSUPPORTED;\r
 }\r
 \r
-\r
 /**\r
   This function adds a Unicode string to UnicodeStringTable.\r
 \r
@@ -921,7 +924,7 @@ AddUnicodeString (
   //\r
   // Make sure the parameter are valid\r
   //\r
-  if (Language == NULL || UnicodeString == NULL || UnicodeStringTable == NULL) {\r
+  if ((Language == NULL) || (UnicodeString == NULL) || (UnicodeStringTable == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -944,7 +947,6 @@ AddUnicodeString (
   //\r
   while (*SupportedLanguages != 0) {\r
     if (CompareIso639LanguageCode (Language, SupportedLanguages)) {\r
-\r
       //\r
       // Determine the size of the Unicode String Table by looking for a NULL Language entry\r
       //\r
@@ -977,10 +979,10 @@ AddUnicodeString (
       //\r
       if (*UnicodeStringTable != NULL) {\r
         CopyMem (\r
-           NewUnicodeStringTable,\r
-           *UnicodeStringTable,\r
-           NumberOfEntries * sizeof (EFI_UNICODE_STRING_TABLE)\r
-           );\r
+          NewUnicodeStringTable,\r
+          *UnicodeStringTable,\r
+          NumberOfEntries * sizeof (EFI_UNICODE_STRING_TABLE)\r
+          );\r
       }\r
 \r
       //\r
@@ -995,16 +997,16 @@ AddUnicodeString (
       //\r
       // Compute the length of the Unicode String\r
       //\r
-      for (UnicodeStringLength = 0; UnicodeString[UnicodeStringLength] != 0; UnicodeStringLength++)\r
-        ;\r
+      for (UnicodeStringLength = 0; UnicodeString[UnicodeStringLength] != 0; UnicodeStringLength++) {\r
+      }\r
 \r
       //\r
       // Allocate space for a copy of the Unicode String\r
       //\r
       NewUnicodeStringTable[NumberOfEntries].UnicodeString = AllocateCopyPool (\r
-                                                              (UnicodeStringLength + 1) * sizeof (CHAR16),\r
-                                                              UnicodeString\r
-                                                              );\r
+                                                               (UnicodeStringLength + 1) * sizeof (CHAR16),\r
+                                                               UnicodeString\r
+                                                               );\r
       if (NewUnicodeStringTable[NumberOfEntries].UnicodeString == NULL) {\r
         FreePool (NewUnicodeStringTable[NumberOfEntries].Language);\r
         FreePool (NewUnicodeStringTable);\r
@@ -1014,8 +1016,8 @@ AddUnicodeString (
       //\r
       // Mark the end of the Unicode String Table\r
       //\r
-      NewUnicodeStringTable[NumberOfEntries + 1].Language       = NULL;\r
-      NewUnicodeStringTable[NumberOfEntries + 1].UnicodeString  = NULL;\r
+      NewUnicodeStringTable[NumberOfEntries + 1].Language      = NULL;\r
+      NewUnicodeStringTable[NumberOfEntries + 1].UnicodeString = NULL;\r
 \r
       //\r
       // Free the old Unicode String Table\r
@@ -1038,7 +1040,6 @@ AddUnicodeString (
   return EFI_UNSUPPORTED;\r
 }\r
 \r
-\r
 /**\r
   This function adds the Null-terminated Unicode string specified by UnicodeString\r
   to UnicodeStringTable.\r
@@ -1102,7 +1103,7 @@ AddUnicodeString2 (
   //\r
   // Make sure the parameter are valid\r
   //\r
-  if (Language == NULL || UnicodeString == NULL || UnicodeStringTable == NULL) {\r
+  if ((Language == NULL) || (UnicodeString == NULL) || (UnicodeStringTable == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1130,11 +1131,13 @@ AddUnicodeString2 (
         Found = TRUE;\r
         break;\r
       }\r
+\r
       SupportedLanguages += 3;\r
     }\r
   } else {\r
-    Found = !IsLanguageSupported(SupportedLanguages, Language);\r
+    Found = !IsLanguageSupported (SupportedLanguages, Language);\r
   }\r
+\r
   //\r
   // If Language is not a member of SupportedLanguages, then return EFI_UNSUPPORTED\r
   //\r
@@ -1152,14 +1155,18 @@ AddUnicodeString2 (
       LanguageString = OldUnicodeStringTable->Language;\r
 \r
       while (*LanguageString != 0) {\r
-        for (Index = 0; LanguageString[Index] != 0 && LanguageString[Index] != ';'; Index++);\r
+        for (Index = 0; LanguageString[Index] != 0 && LanguageString[Index] != ';'; Index++) {\r
+        }\r
 \r
         if (AsciiStrnCmp (Language, LanguageString, Index) == 0) {\r
           return EFI_ALREADY_STARTED;\r
         }\r
+\r
         LanguageString += Index;\r
-        for (; *LanguageString != 0 && *LanguageString == ';'; LanguageString++);\r
+        for ( ; *LanguageString != 0 && *LanguageString == ';'; LanguageString++) {\r
+        }\r
       }\r
+\r
       OldUnicodeStringTable++;\r
       NumberOfEntries++;\r
     }\r
@@ -1190,7 +1197,7 @@ AddUnicodeString2 (
   //\r
   // Allocate space for a copy of the Language specifier\r
   //\r
-  NewUnicodeStringTable[NumberOfEntries].Language = AllocateCopyPool (AsciiStrSize(Language), Language);\r
+  NewUnicodeStringTable[NumberOfEntries].Language = AllocateCopyPool (AsciiStrSize (Language), Language);\r
   if (NewUnicodeStringTable[NumberOfEntries].Language == NULL) {\r
     FreePool (NewUnicodeStringTable);\r
     return EFI_OUT_OF_RESOURCES;\r
@@ -1199,7 +1206,8 @@ AddUnicodeString2 (
   //\r
   // Compute the length of the Unicode String\r
   //\r
-  for (UnicodeStringLength = 0; UnicodeString[UnicodeStringLength] != 0; UnicodeStringLength++);\r
+  for (UnicodeStringLength = 0; UnicodeString[UnicodeStringLength] != 0; UnicodeStringLength++) {\r
+  }\r
 \r
   //\r
   // Allocate space for a copy of the Unicode String\r
@@ -1214,8 +1222,8 @@ AddUnicodeString2 (
   //\r
   // Mark the end of the Unicode String Table\r
   //\r
-  NewUnicodeStringTable[NumberOfEntries + 1].Language       = NULL;\r
-  NewUnicodeStringTable[NumberOfEntries + 1].UnicodeString  = NULL;\r
+  NewUnicodeStringTable[NumberOfEntries + 1].Language      = NULL;\r
+  NewUnicodeStringTable[NumberOfEntries + 1].UnicodeString = NULL;\r
 \r
   //\r
   // Free the old Unicode String Table\r
@@ -1250,7 +1258,7 @@ FreeUnicodeStringTable (
   IN EFI_UNICODE_STRING_TABLE  *UnicodeStringTable\r
   )\r
 {\r
-  UINTN Index;\r
+  UINTN  Index;\r
 \r
   //\r
   // If the Unicode String Table is NULL, then it is already freed\r
@@ -1263,7 +1271,6 @@ FreeUnicodeStringTable (
   // Loop through the Unicode String Table until we reach the end of table marker\r
   //\r
   for (Index = 0; UnicodeStringTable[Index].Language != NULL; Index++) {\r
-\r
     //\r
     // Free the Language string from the Unicode String Table\r
     //\r
@@ -1285,7 +1292,6 @@ FreeUnicodeStringTable (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Returns the status whether get the variable success. The function retrieves\r
   variable  through the UEFI Runtime Service GetVariable().  The\r
@@ -1326,10 +1332,10 @@ GetVariable2 (
   BufferSize = 0;\r
   *Value     = NULL;\r
   if (Size != NULL) {\r
-    *Size  = 0;\r
+    *Size = 0;\r
   }\r
 \r
-  Status = gRT->GetVariable ((CHAR16 *) Name, (EFI_GUID *) Guid, NULL, &BufferSize, *Value);\r
+  Status = gRT->GetVariable ((CHAR16 *)Name, (EFI_GUID *)Guid, NULL, &BufferSize, *Value);\r
   if (Status != EFI_BUFFER_TOO_SMALL) {\r
     return Status;\r
   }\r
@@ -1346,9 +1352,9 @@ GetVariable2 (
   //\r
   // Get the variable data.\r
   //\r
-  Status = gRT->GetVariable ((CHAR16 *) Name, (EFI_GUID *) Guid, NULL, &BufferSize, *Value);\r
+  Status = gRT->GetVariable ((CHAR16 *)Name, (EFI_GUID *)Guid, NULL, &BufferSize, *Value);\r
   if (EFI_ERROR (Status)) {\r
-    FreePool(*Value);\r
+    FreePool (*Value);\r
     *Value = NULL;\r
   }\r
 \r
@@ -1384,24 +1390,24 @@ GetVariable2 (
 **/\r
 EFI_STATUS\r
 EFIAPI\r
-GetVariable3(\r
-  IN CONST CHAR16       *Name,\r
-  IN CONST EFI_GUID     *Guid,\r
-     OUT VOID           **Value,\r
-     OUT UINTN          *Size OPTIONAL,\r
-     OUT UINT32         *Attr OPTIONAL\r
+GetVariable3 (\r
+  IN CONST CHAR16    *Name,\r
+  IN CONST EFI_GUID  *Guid,\r
+  OUT VOID           **Value,\r
+  OUT UINTN          *Size OPTIONAL,\r
+  OUT UINT32         *Attr OPTIONAL\r
   )\r
 {\r
   EFI_STATUS  Status;\r
   UINTN       BufferSize;\r
 \r
-  ASSERT(Name != NULL && Guid != NULL && Value != NULL);\r
+  ASSERT (Name != NULL && Guid != NULL && Value != NULL);\r
 \r
   //\r
   // Try to get the variable size.\r
   //\r
   BufferSize = 0;\r
-  *Value = NULL;\r
+  *Value     = NULL;\r
   if (Size != NULL) {\r
     *Size = 0;\r
   }\r
@@ -1410,7 +1416,7 @@ GetVariable3(
     *Attr = 0;\r
   }\r
 \r
-  Status = gRT->GetVariable((CHAR16 *)Name, (EFI_GUID *)Guid, Attr, &BufferSize, *Value);\r
+  Status = gRT->GetVariable ((CHAR16 *)Name, (EFI_GUID *)Guid, Attr, &BufferSize, *Value);\r
   if (Status != EFI_BUFFER_TOO_SMALL) {\r
     return Status;\r
   }\r
@@ -1418,8 +1424,8 @@ GetVariable3(
   //\r
   // Allocate buffer to get the variable.\r
   //\r
-  *Value = AllocatePool(BufferSize);\r
-  ASSERT(*Value != NULL);\r
+  *Value = AllocatePool (BufferSize);\r
+  ASSERT (*Value != NULL);\r
   if (*Value == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -1427,9 +1433,9 @@ GetVariable3(
   //\r
   // Get the variable data.\r
   //\r
-  Status = gRT->GetVariable((CHAR16 *)Name, (EFI_GUID *)Guid, Attr, &BufferSize, *Value);\r
-  if (EFI_ERROR(Status)) {\r
-    FreePool(*Value);\r
+  Status = gRT->GetVariable ((CHAR16 *)Name, (EFI_GUID *)Guid, Attr, &BufferSize, *Value);\r
+  if (EFI_ERROR (Status)) {\r
+    FreePool (*Value);\r
     *Value = NULL;\r
   }\r
 \r
@@ -1462,9 +1468,9 @@ GetVariable3(
 EFI_STATUS\r
 EFIAPI\r
 GetEfiGlobalVariable2 (\r
-  IN CONST CHAR16    *Name,\r
-  OUT VOID           **Value,\r
-  OUT UINTN          *Size OPTIONAL\r
+  IN CONST CHAR16  *Name,\r
+  OUT VOID         **Value,\r
+  OUT UINTN        *Size OPTIONAL\r
   )\r
 {\r
   return GetVariable2 (Name, &gEfiGlobalVariableGuid, Value, Size);\r
@@ -1540,7 +1546,8 @@ GetBestLanguage (
     // If in RFC 4646 mode, then determine the length of the first RFC 4646 language code in Language\r
     //\r
     if (Iso639Language == 0) {\r
-      for (LanguageLength = 0; Language[LanguageLength] != 0 && Language[LanguageLength] != ';'; LanguageLength++);\r
+      for (LanguageLength = 0; Language[LanguageLength] != 0 && Language[LanguageLength] != ';'; LanguageLength++) {\r
+      }\r
     }\r
 \r
     //\r
@@ -1558,11 +1565,15 @@ GetBestLanguage (
           //\r
           // Skip ';' characters in Supported\r
           //\r
-          for (; *Supported != '\0' && *Supported == ';'; Supported++);\r
+          for ( ; *Supported != '\0' && *Supported == ';'; Supported++) {\r
+          }\r
+\r
           //\r
           // Determine the length of the next language code in Supported\r
           //\r
-          for (CompareLength = 0; Supported[CompareLength] != 0 && Supported[CompareLength] != ';'; CompareLength++);\r
+          for (CompareLength = 0; Supported[CompareLength] != 0 && Supported[CompareLength] != ';'; CompareLength++) {\r
+          }\r
+\r
           //\r
           // If Language is longer than the Supported, then skip to the next language\r
           //\r
@@ -1570,6 +1581,7 @@ GetBestLanguage (
             continue;\r
           }\r
         }\r
+\r
         //\r
         // See if the first LanguageLength characters in Supported match Language\r
         //\r
@@ -1582,6 +1594,7 @@ GetBestLanguage (
           if (BestLanguage == NULL) {\r
             return NULL;\r
           }\r
+\r
           return CopyMem (BestLanguage, Supported, CompareLength);\r
         }\r
       }\r
@@ -1595,10 +1608,12 @@ GetBestLanguage (
         //\r
         // If RFC 4646 mode, then trim Language from the right to the next '-' character\r
         //\r
-        for (LanguageLength--; LanguageLength > 0 && Language[LanguageLength] != '-'; LanguageLength--);\r
+        for (LanguageLength--; LanguageLength > 0 && Language[LanguageLength] != '-'; LanguageLength--) {\r
+        }\r
       }\r
     }\r
   }\r
+\r
   VA_END (Args);\r
 \r
   //\r
@@ -1646,7 +1661,7 @@ EfiLocateProtocolBuffer (
   //\r
   // Check input parameters\r
   //\r
-  if (Protocol == NULL || NoProtocols == NULL || Buffer == NULL) {\r
+  if ((Protocol == NULL) || (NoProtocols == NULL) || (Buffer == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1654,7 +1669,7 @@ EfiLocateProtocolBuffer (
   // Initialze output parameters\r
   //\r
   *NoProtocols = 0;\r
-  *Buffer = NULL;\r
+  *Buffer      = NULL;\r
 \r
   //\r
   // Retrieve the array of handles that support Protocol\r
@@ -1685,6 +1700,7 @@ EfiLocateProtocolBuffer (
     gBS->FreePool (HandleBuffer);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
+\r
   ZeroMem (*Buffer, NoHandles * sizeof (VOID *));\r
 \r
   //\r
@@ -1794,18 +1810,19 @@ EfiOpenFileByDevicePath (
   IN     UINT64                    Attributes\r
   )\r
 {\r
-  EFI_STATUS                      Status;\r
-  EFI_HANDLE                      FileSystemHandle;\r
-  EFI_SIMPLE_FILE_SYSTEM_PROTOCOL *FileSystem;\r
-  EFI_FILE_PROTOCOL               *LastFile;\r
-  FILEPATH_DEVICE_PATH            *FilePathNode;\r
-  CHAR16                          *AlignedPathName;\r
-  CHAR16                          *PathName;\r
-  EFI_FILE_PROTOCOL               *NextFile;\r
+  EFI_STATUS                       Status;\r
+  EFI_HANDLE                       FileSystemHandle;\r
+  EFI_SIMPLE_FILE_SYSTEM_PROTOCOL  *FileSystem;\r
+  EFI_FILE_PROTOCOL                *LastFile;\r
+  FILEPATH_DEVICE_PATH             *FilePathNode;\r
+  CHAR16                           *AlignedPathName;\r
+  CHAR16                           *PathName;\r
+  EFI_FILE_PROTOCOL                *NextFile;\r
 \r
   if (File == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
+\r
   *File = NULL;\r
 \r
   if (FilePath == NULL) {\r
@@ -1823,6 +1840,7 @@ EfiOpenFileByDevicePath (
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
+\r
   Status = gBS->OpenProtocol (\r
                   FileSystemHandle,\r
                   &gEfiSimpleFileSystemProtocolGuid,\r
@@ -1848,11 +1866,13 @@ EfiOpenFileByDevicePath (
   // Traverse the device path nodes relative to the filesystem.\r
   //\r
   while (!IsDevicePathEnd (*FilePath)) {\r
-    if (DevicePathType (*FilePath) != MEDIA_DEVICE_PATH ||\r
-        DevicePathSubType (*FilePath) != MEDIA_FILEPATH_DP) {\r
+    if ((DevicePathType (*FilePath) != MEDIA_DEVICE_PATH) ||\r
+        (DevicePathSubType (*FilePath) != MEDIA_FILEPATH_DP))\r
+    {\r
       Status = EFI_INVALID_PARAMETER;\r
       goto CloseLastFile;\r
     }\r
+\r
     FilePathNode = (FILEPATH_DEVICE_PATH *)*FilePath;\r
 \r
     //\r
@@ -1862,7 +1882,7 @@ EfiOpenFileByDevicePath (
     //\r
     if ((UINTN)FilePathNode->PathName % sizeof *FilePathNode->PathName == 0) {\r
       AlignedPathName = NULL;\r
-      PathName = FilePathNode->PathName;\r
+      PathName        = FilePathNode->PathName;\r
     } else {\r
       AlignedPathName = AllocateCopyPool (\r
                           (DevicePathNodeLength (FilePathNode) -\r
@@ -1873,6 +1893,7 @@ EfiOpenFileByDevicePath (
         Status = EFI_OUT_OF_RESOURCES;\r
         goto CloseLastFile;\r
       }\r
+\r
       PathName = AlignedPathName;\r
     }\r
 \r
@@ -1894,6 +1915,7 @@ EfiOpenFileByDevicePath (
     if (AlignedPathName != NULL) {\r
       FreePool (AlignedPathName);\r
     }\r
+\r
     if (EFI_ERROR (Status)) {\r
       goto CloseLastFile;\r
     }\r
@@ -1902,7 +1924,7 @@ EfiOpenFileByDevicePath (
     // Advance to the next device path node.\r
     //\r
     LastFile->Close (LastFile);\r
-    LastFile = NextFile;\r
+    LastFile  = NextFile;\r
     *FilePath = NextDevicePathNode (FilePathNode);\r
   }\r
 \r