]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Aest/AestParser.c
ShellPkg: Apply uncrustify changes
[mirror_edk2.git] / ShellPkg / Library / UefiShellAcpiViewCommandLib / Parsers / Aest / AestParser.c
index 7aa8c8fd2167822f4f5e0f6e4067195aac510e5c..48f71484fb82d55f29e6f55cbebe4f41abaca768 100644 (file)
 #include "AcpiViewConfig.h"\r
 \r
 // Local variables\r
-STATIC ACPI_DESCRIPTION_HEADER_INFO AcpiHdrInfo;\r
-STATIC UINT8*                       AestNodeType;\r
-STATIC UINT16*                      AestNodeLength;\r
-STATIC UINT32*                      NodeDataOffset;\r
-STATIC UINT32*                      NodeInterfaceOffset;\r
-STATIC UINT32*                      NodeInterruptArrayOffset;\r
-STATIC UINT32*                      NodeInterruptCount;\r
-STATIC UINT32*                      ProcessorId;\r
-STATIC UINT8*                       ProcessorFlags;\r
-STATIC UINT8*                       ProcessorResourceType;\r
+STATIC ACPI_DESCRIPTION_HEADER_INFO  AcpiHdrInfo;\r
+STATIC UINT8                         *AestNodeType;\r
+STATIC UINT16                        *AestNodeLength;\r
+STATIC UINT32                        *NodeDataOffset;\r
+STATIC UINT32                        *NodeInterfaceOffset;\r
+STATIC UINT32                        *NodeInterruptArrayOffset;\r
+STATIC UINT32                        *NodeInterruptCount;\r
+STATIC UINT32                        *ProcessorId;\r
+STATIC UINT8                         *ProcessorFlags;\r
+STATIC UINT8                         *ProcessorResourceType;\r
 \r
 /**\r
   Validate Processor Flags.\r
@@ -40,16 +40,18 @@ STATIC
 VOID\r
 EFIAPI\r
 ValidateProcessorFlags (\r
-  IN UINT8Ptr,\r
-  IN VOID*  Context\r
+  IN UINT8  *Ptr,\r
+  IN VOID   *Context\r
   )\r
 {\r
   // If the global or shared node flag is set then the ACPI Processor ID\r
   // field must be set to 0 and ignored.\r
   if (((*Ptr & 0x3) != 0) && (*ProcessorId != 0)) {\r
     IncrementErrorCount ();\r
-    Print (L"\nERROR: 'ACPI Processor ID' field must be set to 0 for global"\r
-           L" or shared nodes.");\r
+    Print (\r
+      L"\nERROR: 'ACPI Processor ID' field must be set to 0 for global"\r
+      L" or shared nodes."\r
+      );\r
   }\r
 }\r
 \r
@@ -64,13 +66,13 @@ STATIC
 VOID\r
 EFIAPI\r
 ValidateGicInterfaceType (\r
-  IN UINT8Ptr,\r
-  IN VOID*  Context\r
+  IN UINT8  *Ptr,\r
+  IN VOID   *Context\r
   )\r
 {\r
-  UINT32 GicInterfaceType;\r
+  UINT32  GicInterfaceType;\r
 \r
-  GicInterfaceType = *(UINT32*)Ptr;\r
+  GicInterfaceType = *(UINT32 *)Ptr;\r
   if (GicInterfaceType > 3) {\r
     IncrementErrorCount ();\r
     Print (L"\nError: Invalid GIC Interface type %d", GicInterfaceType);\r
@@ -88,8 +90,8 @@ STATIC
 VOID\r
 EFIAPI\r
 ValidateInterfaceType (\r
-  IN UINT8Ptr,\r
-  IN VOID*  Context\r
+  IN UINT8  *Ptr,\r
+  IN VOID   *Context\r
   )\r
 {\r
   if (*Ptr > 1) {\r
@@ -109,8 +111,8 @@ STATIC
 VOID\r
 EFIAPI\r
 ValidateInterruptType (\r
-  IN UINT8Ptr,\r
-  IN VOID*  Context\r
+  IN UINT8  *Ptr,\r
+  IN VOID   *Context\r
   )\r
 {\r
   if (*Ptr > 1) {\r
@@ -130,8 +132,8 @@ STATIC
 VOID\r
 EFIAPI\r
 ValidateInterruptFlags (\r
-  IN UINT8Ptr,\r
-  IN VOID*  Context\r
+  IN UINT8  *Ptr,\r
+  IN VOID   *Context\r
   )\r
 {\r
   if ((*Ptr & 0xfe) != 0) {\r
@@ -149,8 +151,8 @@ ValidateInterruptFlags (
 VOID\r
 EFIAPI\r
 DumpVendorSpecificData (\r
-  IN CONST CHAR16Format OPTIONAL,\r
-  IN UINT8*        Ptr\r
+  IN CONST CHAR16  *Format OPTIONAL,\r
+  IN UINT8         *Ptr\r
   )\r
 {\r
   Print (\r
@@ -183,27 +185,27 @@ DumpVendorSpecificData (
 /**\r
   An ACPI_PARSER array describing the ACPI AEST Table.\r
 **/\r
-STATIC CONST ACPI_PARSER AestParser[] = {\r
+STATIC CONST ACPI_PARSER  AestParser[] = {\r
   PARSE_ACPI_HEADER (&AcpiHdrInfo)\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the AEST Node Header.\r
 **/\r
-STATIC CONST ACPI_PARSER AestNodeHeaderParser[] = {\r
-  {L"Type", 1, 0, L"%d", NULL, (VOID**)&AestNodeType, NULL, NULL},\r
-  {L"Length", 2, 1, L"%d", NULL, (VOID**)&AestNodeLength, NULL, NULL},\r
-  {L"Reserved", 1, 3, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Node Data Offset", 4, 4, L"%d", NULL, (VOID**)&NodeDataOffset, NULL, NULL},\r
-  {L"Node Interface Offset", 4, 8, L"%d", NULL,\r
-    (VOID**)&NodeInterfaceOffset, NULL, NULL},\r
-  {L"Node Interrupt Array Offset", 4, 12, L"%d", NULL,\r
-    (VOID**)&NodeInterruptArrayOffset, NULL, NULL},\r
-  {L"Node Interrupt Count", 4, 16, L"%d", NULL,\r
-    (VOID**)&NodeInterruptCount, NULL, NULL},\r
-  {L"Timestamp Rate", 8, 20, L"%ld", NULL, NULL, NULL, NULL},\r
-  {L"Reserved1", 8, 28, L"0x%lx", NULL, NULL, NULL, NULL},\r
-  {L"Error Injection Countdown Rate", 8, 36, L"%ld", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestNodeHeaderParser[] = {\r
+  { L"Type",                           1, 0,  L"%d",    NULL, (VOID **)&AestNodeType,   NULL, NULL },\r
+  { L"Length",                         2, 1,  L"%d",    NULL, (VOID **)&AestNodeLength, NULL, NULL },\r
+  { L"Reserved",                       1, 3,  L"0x%x",  NULL, NULL,                     NULL, NULL },\r
+  { L"Node Data Offset",               4, 4,  L"%d",    NULL, (VOID **)&NodeDataOffset, NULL, NULL },\r
+  { L"Node Interface Offset",          4, 8,  L"%d",    NULL,\r
+    (VOID **)&NodeInterfaceOffset, NULL, NULL },\r
+  { L"Node Interrupt Array Offset",    4, 12, L"%d",    NULL,\r
+    (VOID **)&NodeInterruptArrayOffset, NULL, NULL },\r
+  { L"Node Interrupt Count",           4, 16, L"%d",    NULL,\r
+    (VOID **)&NodeInterruptCount, NULL, NULL },\r
+  { L"Timestamp Rate",                 8, 20, L"%ld",   NULL, NULL,                     NULL, NULL },\r
+  { L"Reserved1",                      8, 28, L"0x%lx", NULL, NULL,                     NULL, NULL },\r
+  { L"Error Injection Countdown Rate", 8, 36, L"%ld",   NULL, NULL,                     NULL, NULL }\r
   // Node specific data...\r
   // Node interface...\r
   // Node interrupt array...\r
@@ -212,100 +214,100 @@ STATIC CONST ACPI_PARSER AestNodeHeaderParser[] = {
 /**\r
   An ACPI_PARSER array describing the Processor error node specific data.\r
 **/\r
-STATIC CONST ACPI_PARSER AestProcessorStructure[] = {\r
-  {L"ACPI Processor ID", 4, 0, L"0x%x", NULL, (VOID**)&ProcessorId, NULL, NULL},\r
-  {L"Resource Type", 1, 4, L"%d", NULL, (VOID**)&ProcessorResourceType, NULL,\r
-    NULL},\r
-  {L"Reserved", 1, 5, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Flags", 1, 6, L"0x%x", NULL, (VOID**)&ProcessorFlags,\r
-    ValidateProcessorFlags, NULL},\r
-  {L"Revision", 1, 7, L"%d", NULL, NULL, NULL, NULL},\r
-  {L"Processor Affinity Level Indicator", 8, 8, L"0x%lx", NULL, NULL, NULL,\r
-    NULL},\r
+STATIC CONST ACPI_PARSER  AestProcessorStructure[] = {\r
+  { L"ACPI Processor ID",                  4, 0, L"0x%x",  NULL, (VOID **)&ProcessorId,           NULL, NULL },\r
+  { L"Resource Type",                      1, 4, L"%d",    NULL, (VOID **)&ProcessorResourceType, NULL,\r
+    NULL },\r
+  { L"Reserved",                           1, 5, L"0x%x",  NULL, NULL,                            NULL, NULL },\r
+  { L"Flags",                              1, 6, L"0x%x",  NULL, (VOID **)&ProcessorFlags,\r
+    ValidateProcessorFlags, NULL },\r
+  { L"Revision",                           1, 7, L"%d",    NULL, NULL,                            NULL, NULL },\r
+  { L"Processor Affinity Level Indicator", 8, 8, L"0x%lx", NULL, NULL,                            NULL,\r
+    NULL },\r
   // Resource specific data...\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the processor cache resource substructure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestProcessorCacheResourceSubstructure[] = {\r
-  {L"Cache reference ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Reserved", 4, 4, L"%d", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestProcessorCacheResourceSubstructure[] = {\r
+  { L"Cache reference ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL },\r
+  { L"Reserved",           4, 4, L"%d",   NULL, NULL, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the processor TLB resource substructure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestProcessorTlbResourceSubstructure[] = {\r
-  {L"TLB reference ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Reserved", 4, 4, L"%d", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestProcessorTlbResourceSubstructure[] = {\r
+  { L"TLB reference ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL },\r
+  { L"Reserved",         4, 4, L"%d",   NULL, NULL, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the processor generic resource substructure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestProcessorGenericResourceSubstructure[] = {\r
-  {L"Vendor-defined data", 4, 0, L"%x", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestProcessorGenericResourceSubstructure[] = {\r
+  { L"Vendor-defined data", 4, 0, L"%x", NULL, NULL, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the memory controller structure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestMemoryControllerStructure[] = {\r
-  {L"Proximity Domain", 4, 0, L"0x%x", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestMemoryControllerStructure[] = {\r
+  { L"Proximity Domain", 4, 0, L"0x%x", NULL, NULL, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the SMMU structure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestSmmuStructure[] = {\r
-  {L"IORT Node reference ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"SubComponent reference ID", 4, 4, L"0x%x", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestSmmuStructure[] = {\r
+  { L"IORT Node reference ID",    4, 0, L"0x%x", NULL, NULL, NULL, NULL },\r
+  { L"SubComponent reference ID", 4, 4, L"0x%x", NULL, NULL, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the vendor-defined structure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestVendorDefinedStructure[] = {\r
-  {L"Hardware ID", 4, 0, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Unique ID", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Vendor-specific data", 16, 8, NULL, DumpVendorSpecificData, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestVendorDefinedStructure[] = {\r
+  { L"Hardware ID",          4,  0, L"0x%x", NULL,                   NULL, NULL, NULL },\r
+  { L"Unique ID",            4,  4, L"0x%x", NULL,                   NULL, NULL, NULL },\r
+  { L"Vendor-specific data", 16, 8, NULL,    DumpVendorSpecificData, NULL, NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the GIC structure.\r
 **/\r
-STATIC CONST ACPI_PARSER AestGicStructure[] = {\r
-  {L"GIC Interface Type", 4, 0, L"0x%x", NULL, NULL, ValidateGicInterfaceType,\r
-    NULL},\r
-  {L"GIC Interface reference ID", 4, 4, L"0x%x", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestGicStructure[] = {\r
+  { L"GIC Interface Type",         4, 0, L"0x%x", NULL, NULL, ValidateGicInterfaceType,\r
+    NULL },\r
+  { L"GIC Interface reference ID", 4, 4, L"0x%x", NULL, NULL, NULL,                    NULL}\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the node interface.\r
 **/\r
-STATIC CONST ACPI_PARSER AestNodeInterface[] = {\r
-  {L"Interface Type", 1, 0, L"%d", NULL, NULL, ValidateInterfaceType, NULL},\r
-  {L"Reserved", 3, 1, L"%x %x %x", Dump3Chars, NULL, NULL, NULL},\r
-  {L"Flags", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Base Address", 8, 8, L"0x%lx", NULL, NULL, NULL, NULL},\r
-  {L"Start Error Record Index", 4, 16, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Number of Error Records", 4, 20, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Error Records Implemented", 8, 24, L"0x%lx", NULL, NULL, NULL, NULL},\r
-  {L"Error Records Support", 8, 32, L"0x%lx", NULL, NULL, NULL, NULL},\r
-  {L"Addressing mode", 8, 40, L"0x%lx", NULL, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestNodeInterface[] = {\r
+  { L"Interface Type",            1, 0,  L"%d",       NULL,       NULL, ValidateInterfaceType, NULL },\r
+  { L"Reserved",                  3, 1,  L"%x %x %x", Dump3Chars, NULL, NULL,                  NULL },\r
+  { L"Flags",                     4, 4,  L"0x%x",     NULL,       NULL, NULL,                  NULL },\r
+  { L"Base Address",              8, 8,  L"0x%lx",    NULL,       NULL, NULL,                  NULL },\r
+  { L"Start Error Record Index",  4, 16, L"0x%x",     NULL,       NULL, NULL,                  NULL },\r
+  { L"Number of Error Records",   4, 20, L"0x%x",     NULL,       NULL, NULL,                  NULL },\r
+  { L"Error Records Implemented", 8, 24, L"0x%lx",    NULL,       NULL, NULL,                  NULL },\r
+  { L"Error Records Support",     8, 32, L"0x%lx",    NULL,       NULL, NULL,                  NULL },\r
+  { L"Addressing mode",           8, 40, L"0x%lx",    NULL,       NULL, NULL,                  NULL }\r
 };\r
 \r
 /**\r
   An ACPI_PARSER array describing the node interrupts.\r
 **/\r
-STATIC CONST ACPI_PARSER AestNodeInterrupt[] = {\r
-  {L"Interrupt Type", 1, 0, L"%d", NULL, NULL, ValidateInterruptType, NULL},\r
-  {L"Reserved", 2, 1, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Interrupt Flags", 1, 3, L"0x%x", NULL, NULL, ValidateInterruptFlags, NULL},\r
-  {L"Interrupt GSIV", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"ID", 1, 8, L"0x%x", NULL, NULL, NULL, NULL},\r
-  {L"Reserved1", 3, 9, L"%x %x %x", Dump3Chars, NULL, NULL, NULL}\r
+STATIC CONST ACPI_PARSER  AestNodeInterrupt[] = {\r
+  { L"Interrupt Type",  1, 0, L"%d",       NULL,       NULL, ValidateInterruptType,  NULL },\r
+  { L"Reserved",        2, 1, L"0x%x",     NULL,       NULL, NULL,                   NULL },\r
+  { L"Interrupt Flags", 1, 3, L"0x%x",     NULL,       NULL, ValidateInterruptFlags, NULL },\r
+  { L"Interrupt GSIV",  4, 4, L"0x%x",     NULL,       NULL, NULL,                   NULL },\r
+  { L"ID",              1, 8, L"0x%x",     NULL,       NULL, NULL,                   NULL },\r
+  { L"Reserved1",       3, 9, L"%x %x %x", Dump3Chars, NULL, NULL,                   NULL }\r
 };\r
 \r
 /**\r
@@ -318,11 +320,11 @@ STATIC CONST ACPI_PARSER AestNodeInterrupt[] = {
 STATIC\r
 VOID\r
 DumpProcessorNode (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
-  UINT32 Offset;\r
+  UINT32  Offset;\r
 \r
   Offset = ParseAcpi (\r
              TRUE,\r
@@ -337,7 +339,8 @@ DumpProcessorNode (
   // successfully read.\r
   if ((ProcessorId == NULL)           ||\r
       (ProcessorResourceType == NULL) ||\r
-      (ProcessorFlags == NULL)) {\r
+      (ProcessorFlags == NULL))\r
+  {\r
     IncrementErrorCount ();\r
     Print (\r
       L"ERROR: Insufficient Processor Error Node length. Length = %d.\n",\r
@@ -393,8 +396,8 @@ DumpProcessorNode (
 STATIC\r
 VOID\r
 DumpMemoryControllerNode (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
   ParseAcpi (\r
@@ -416,8 +419,8 @@ DumpMemoryControllerNode (
 STATIC\r
 VOID\r
 DumpSmmuNode (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
   ParseAcpi (\r
@@ -439,8 +442,8 @@ DumpSmmuNode (
 STATIC\r
 VOID\r
 DumpVendorDefinedNode (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
   ParseAcpi (\r
@@ -462,8 +465,8 @@ DumpVendorDefinedNode (
 STATIC\r
 VOID\r
 DumpGicNode (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
   ParseAcpi (\r
@@ -485,8 +488,8 @@ DumpGicNode (
 STATIC\r
 VOID\r
 DumpNodeInterface (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length\r
   )\r
 {\r
   ParseAcpi (\r
@@ -509,19 +512,19 @@ DumpNodeInterface (
 STATIC\r
 VOID\r
 DumpNodeInterrupts (\r
-  IN UINT8Ptr,\r
-  IN UINT32 Length,\r
-  IN UINT32 InterruptCount\r
+  IN UINT8   *Ptr,\r
+  IN UINT32  Length,\r
+  IN UINT32  InterruptCount\r
   )\r
 {\r
-  UINT32 Offset;\r
-  UINT32 Index;\r
-  CHAR8  Buffer[64];\r
+  UINT32  Offset;\r
+  UINT32  Index;\r
+  CHAR8   Buffer[64];\r
 \r
   if (Length < (InterruptCount * sizeof (EFI_ACPI_AEST_INTERRUPT_STRUCT))) {\r
     IncrementErrorCount ();\r
     Print (\r
-      L"ERROR: Node not long enough for Interrupt Array.\n"\\r
+      L"ERROR: Node not long enough for Interrupt Array.\n" \\r
       L"       Length left = %d, Required = %d, Interrupt Count = %d\n",\r
       Length,\r
       (InterruptCount * sizeof (EFI_ACPI_AEST_INTERRUPT_STRUCT)),\r
@@ -547,7 +550,7 @@ DumpNodeInterrupts (
                 Length - Offset,\r
                 PARSER_PARAMS (AestNodeInterrupt)\r
                 );\r
-  } //for\r
+  } // for\r
 }\r
 \r
 /**\r
@@ -564,7 +567,7 @@ DumpNodeInterrupts (
 STATIC\r
 VOID\r
 DumpAestNodeStructure (\r
-  IN UINT8*  Ptr,\r
+  IN UINT8   *Ptr,\r
   IN UINT32  Length,\r
   IN UINT8   NodeType,\r
   IN UINT32  DataOffset,\r
@@ -573,9 +576,9 @@ DumpAestNodeStructure (
   IN UINT32  InterruptCount\r
   )\r
 {\r
-  UINT32 Offset;\r
-  UINT32 RemainingLength;\r
-  UINT8NodeDataPtr;\r
+  UINT32  Offset;\r
+  UINT32  RemainingLength;\r
+  UINT8   *NodeDataPtr;\r
 \r
   Offset = ParseAcpi (\r
              TRUE,\r
@@ -589,7 +592,7 @@ DumpAestNodeStructure (
   if ((Offset > DataOffset) || (DataOffset > Length)) {\r
     IncrementErrorCount ();\r
     Print (\r
-      L"ERROR: Invalid Node Data Offset: %d.\n"\\r
+      L"ERROR: Invalid Node Data Offset: %d.\n" \\r
       L"       It should be between %d and %d.\n",\r
       DataOffset,\r
       Offset,\r
@@ -600,7 +603,7 @@ DumpAestNodeStructure (
   if ((Offset > InterfaceOffset) || (InterfaceOffset > Length)) {\r
     IncrementErrorCount ();\r
     Print (\r
-      L"ERROR: Invalid Node Interface Offset: %d.\n"\\r
+      L"ERROR: Invalid Node Interface Offset: %d.\n" \\r
       L"       It should be between %d and %d.\n",\r
       InterfaceOffset,\r
       Offset,\r
@@ -611,7 +614,7 @@ DumpAestNodeStructure (
   if ((Offset > InterruptArrayOffset) || (InterruptArrayOffset > Length)) {\r
     IncrementErrorCount ();\r
     Print (\r
-      L"ERROR: Invalid Node Interrupt Array Offset: %d.\n"\\r
+      L"ERROR: Invalid Node Interrupt Array Offset: %d.\n" \\r
       L"       It should be between %d and %d.\n",\r
       InterruptArrayOffset,\r
       Offset,\r
@@ -620,7 +623,7 @@ DumpAestNodeStructure (
   }\r
 \r
   // Parse Node Data Field.\r
-  NodeDataPtr = Ptr + DataOffset;\r
+  NodeDataPtr     = Ptr + DataOffset;\r
   RemainingLength = Length - DataOffset;\r
   switch (NodeType) {\r
     case EFI_ACPI_AEST_NODE_TYPE_PROCESSOR:\r
@@ -675,14 +678,14 @@ DumpAestNodeStructure (
 VOID\r
 EFIAPI\r
 ParseAcpiAest (\r
-  IN BOOLEAN Trace,\r
-  IN UINT8*  Ptr,\r
-  IN UINT32  AcpiTableLength,\r
-  IN UINT8   AcpiTableRevision\r
+  IN BOOLEAN  Trace,\r
+  IN UINT8    *Ptr,\r
+  IN UINT32   AcpiTableLength,\r
+  IN UINT8    AcpiTableRevision\r
   )\r
 {\r
   UINT32  Offset;\r
-  UINT8*  NodePtr;\r
+  UINT8   *NodePtr;\r
 \r
   if (!Trace) {\r
     return;\r
@@ -716,10 +719,11 @@ ParseAcpiAest (
         (NodeDataOffset == NULL)           ||\r
         (NodeInterfaceOffset == NULL)      ||\r
         (NodeInterruptArrayOffset == NULL) ||\r
-        (NodeInterruptCount == NULL)) {\r
+        (NodeInterruptCount == NULL))\r
+    {\r
       IncrementErrorCount ();\r
       Print (\r
-        L"ERROR: Insufficient length left for Node Structure.\n"\\r
+        L"ERROR: Insufficient length left for Node Structure.\n" \\r
         L"       Length left = %d.\n",\r
         AcpiTableLength - Offset\r
         );\r
@@ -728,11 +732,12 @@ ParseAcpiAest (
 \r
     // Validate AEST Node length\r
     if ((*AestNodeLength == 0) ||\r
-        ((Offset + (*AestNodeLength)) > AcpiTableLength)) {\r
+        ((Offset + (*AestNodeLength)) > AcpiTableLength))\r
+    {\r
       IncrementErrorCount ();\r
       Print (\r
         L"ERROR: Invalid AEST Node length. " \\r
-          L"Length = %d. Offset = %d. AcpiTableLength = %d.\n",\r
+        L"Length = %d. Offset = %d. AcpiTableLength = %d.\n",\r
         *AestNodeLength,\r
         Offset,\r
         AcpiTableLength\r