]> git.proxmox.com Git - mirror_edk2.git/blobdiff - DynamicTablesPkg/Library/Common/AmlLib/Utils/AmlUtility.c
DynamicTablesPkg: Apply uncrustify changes
[mirror_edk2.git] / DynamicTablesPkg / Library / Common / AmlLib / Utils / AmlUtility.c
index 8b5153f916a4a26c113ebc74bb6188afcbda81f8..811b7d22022853df68345531a2fa48dab408c415 100644 (file)
 EFI_STATUS\r
 EFIAPI\r
 AcpiPlatformChecksum (\r
-  IN  EFI_ACPI_DESCRIPTION_HEADER  * AcpiTable\r
+  IN  EFI_ACPI_DESCRIPTION_HEADER  *AcpiTable\r
   )\r
 {\r
-  UINT8   * Ptr;\r
-  UINT8     Sum;\r
-  UINT32    Size;\r
+  UINT8   *Ptr;\r
+  UINT8   Sum;\r
+  UINT32  Size;\r
 \r
   if (AcpiTable == NULL) {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  Ptr = (UINT8*)AcpiTable;\r
+  Ptr  = (UINT8 *)AcpiTable;\r
   Size = AcpiTable->Length;\r
-  Sum = 0;\r
+  Sum  = 0;\r
 \r
   // Set the checksum field to 0 first.\r
   AcpiTable->Checksum = 0;\r
@@ -76,70 +76,78 @@ STATIC
 BOOLEAN\r
 EFIAPI\r
 AmlComputeSizeCallback (\r
-  IN      AML_NODE_HEADER  * Node,\r
-  IN  OUT VOID             * Context,\r
-  IN  OUT EFI_STATUS       * Status   OPTIONAL\r
+  IN      AML_NODE_HEADER  *Node,\r
+  IN  OUT VOID             *Context,\r
+  IN  OUT EFI_STATUS       *Status   OPTIONAL\r
   )\r
 {\r
-  UINT32                    Size;\r
-  EAML_PARSE_INDEX          IndexPtr;\r
-  CONST AML_OBJECT_NODE   * ParentNode;\r
+  UINT32                 Size;\r
+  EAML_PARSE_INDEX       IndexPtr;\r
+  CONST AML_OBJECT_NODE  *ParentNode;\r
 \r
   if (!IS_AML_NODE_VALID (Node) ||\r
-      (Context == NULL)) {\r
+      (Context == NULL))\r
+  {\r
     ASSERT (0);\r
     if (Status != NULL) {\r
       *Status = EFI_INVALID_PARAMETER;\r
     }\r
+\r
     return FALSE;\r
   }\r
 \r
   // Ignore the second fixed argument of method invocation nodes\r
   // as the information stored there (the argument count) is not in the\r
   // ACPI specification.\r
-  ParentNode = (CONST AML_OBJECT_NODE*)AmlGetParent (Node);\r
+  ParentNode = (CONST AML_OBJECT_NODE *)AmlGetParent (Node);\r
   if (IS_AML_OBJECT_NODE (ParentNode)                             &&\r
       AmlNodeCompareOpCode (ParentNode, AML_METHOD_INVOC_OP, 0)   &&\r
-      AmlIsNodeFixedArgument (Node, &IndexPtr)) {\r
+      AmlIsNodeFixedArgument (Node, &IndexPtr))\r
+  {\r
     if (IndexPtr == EAmlParseIndexTerm1) {\r
       if (Status != NULL) {\r
         *Status = EFI_SUCCESS;\r
       }\r
+\r
       return TRUE;\r
     }\r
   }\r
 \r
-  Size = *((UINT32*)Context);\r
+  Size = *((UINT32 *)Context);\r
 \r
   if (IS_AML_DATA_NODE (Node)) {\r
-    Size += ((AML_DATA_NODE*)Node)->Size;\r
+    Size += ((AML_DATA_NODE *)Node)->Size;\r
   } else if (IS_AML_OBJECT_NODE (Node)  &&\r
              !AmlNodeHasAttribute (\r
-                (CONST AML_OBJECT_NODE*)Node,\r
-                AML_IS_PSEUDO_OPCODE)) {\r
+                (CONST AML_OBJECT_NODE *)Node,\r
+                AML_IS_PSEUDO_OPCODE\r
+                ))\r
+  {\r
     // Ignore pseudo-opcodes as they are not part of the\r
     // ACPI specification.\r
 \r
-    Size += (((AML_OBJECT_NODE*)Node)->AmlByteEncoding->OpCode ==\r
-               AML_EXT_OP) ? 2 : 1;\r
+    Size += (((AML_OBJECT_NODE *)Node)->AmlByteEncoding->OpCode ==\r
+             AML_EXT_OP) ? 2 : 1;\r
 \r
     // Add the size of the PkgLen.\r
     if (AmlNodeHasAttribute (\r
-          (AML_OBJECT_NODE*)Node,\r
-          AML_HAS_PKG_LENGTH)) {\r
-      Size += AmlComputePkgLengthWidth (((AML_OBJECT_NODE*)Node)->PkgLen);\r
+          (AML_OBJECT_NODE *)Node,\r
+          AML_HAS_PKG_LENGTH\r
+          ))\r
+    {\r
+      Size += AmlComputePkgLengthWidth (((AML_OBJECT_NODE *)Node)->PkgLen);\r
     }\r
   }\r
 \r
   // Check for overflow.\r
   // The root node has a null size, thus the strict comparison.\r
-  if (*((UINT32*)Context) > Size) {\r
+  if (*((UINT32 *)Context) > Size) {\r
     ASSERT (0);\r
     *Status = EFI_INVALID_PARAMETER;\r
     return FALSE;\r
   }\r
 \r
-  *((UINT32*)Context) = Size;\r
+  *((UINT32 *)Context) = Size;\r
 \r
   if (Status != NULL) {\r
     *Status = EFI_SUCCESS;\r
@@ -159,14 +167,15 @@ AmlComputeSizeCallback (
 EFI_STATUS\r
 EFIAPI\r
 AmlComputeSize (\r
-  IN      CONST AML_NODE_HEADER   * Node,\r
-  IN  OUT       UINT32            * Size\r
+  IN      CONST AML_NODE_HEADER  *Node,\r
+  IN  OUT       UINT32           *Size\r
   )\r
 {\r
   EFI_STATUS  Status;\r
 \r
   if (!IS_AML_NODE_VALID (Node) ||\r
-      (Size == NULL)) {\r
+      (Size == NULL))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -174,9 +183,9 @@ AmlComputeSize (
   *Size = 0;\r
 \r
   AmlEnumTree (\r
-    (AML_NODE_HEADER*)Node,\r
+    (AML_NODE_HEADER *)Node,\r
     AmlComputeSizeCallback,\r
-    (VOID*)Size,\r
+    (VOID *)Size,\r
     &Status\r
     );\r
 \r
@@ -195,15 +204,16 @@ AmlComputeSize (
 EFI_STATUS\r
 EFIAPI\r
 AmlNodeGetIntegerValue (\r
-  IN  AML_OBJECT_NODE   * Node,\r
-  OUT UINT64            * Value\r
+  IN  AML_OBJECT_NODE  *Node,\r
+  OUT UINT64           *Value\r
   )\r
 {\r
-  AML_DATA_NODE  * DataNode;\r
+  AML_DATA_NODE  *DataNode;\r
 \r
   if ((!IsIntegerNode (Node)            &&\r
        !IsSpecialIntegerNode (Node))    ||\r
-      (Value == NULL)) {\r
+      (Value == NULL))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -219,13 +229,15 @@ AmlNodeGetIntegerValue (
       ASSERT (0);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
+\r
     return EFI_SUCCESS;\r
   }\r
 \r
   // For integer nodes, the value is in the first fixed argument.\r
-  DataNode = (AML_DATA_NODE*)Node->FixedArgs[EAmlParseIndexTerm0];\r
+  DataNode = (AML_DATA_NODE *)Node->FixedArgs[EAmlParseIndexTerm0];\r
   if (!IS_AML_DATA_NODE (DataNode) ||\r
-      (DataNode->DataType != EAmlNodeDataTypeUInt)) {\r
+      (DataNode->DataType != EAmlNodeDataTypeUInt))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -233,22 +245,22 @@ AmlNodeGetIntegerValue (
   switch (DataNode->Size) {\r
     case 1:\r
     {\r
-      *Value = *((UINT8*)(DataNode->Buffer));\r
+      *Value = *((UINT8 *)(DataNode->Buffer));\r
       break;\r
     }\r
     case 2:\r
     {\r
-      *Value = *((UINT16*)(DataNode->Buffer));\r
+      *Value = *((UINT16 *)(DataNode->Buffer));\r
       break;\r
     }\r
     case 4:\r
     {\r
-      *Value = *((UINT32*)(DataNode->Buffer));\r
+      *Value = *((UINT32 *)(DataNode->Buffer));\r
       break;\r
     }\r
     case 8:\r
     {\r
-      *Value = *((UINT64*)(DataNode->Buffer));\r
+      *Value = *((UINT64 *)(DataNode->Buffer));\r
       break;\r
     }\r
     default:\r
@@ -274,14 +286,14 @@ STATIC
 EFI_STATUS\r
 EFIAPI\r
 AmlUnwindSpecialInteger (\r
-  IN  AML_OBJECT_NODE   * Node\r
+  IN  AML_OBJECT_NODE  *Node\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
+  EFI_STATUS  Status;\r
 \r
-  AML_DATA_NODE             * NewDataNode;\r
-  UINT8                       Value;\r
-  CONST AML_BYTE_ENCODING   * ByteEncoding;\r
+  AML_DATA_NODE            *NewDataNode;\r
+  UINT8                    Value;\r
+  CONST AML_BYTE_ENCODING  *ByteEncoding;\r
 \r
   if (!IsSpecialIntegerNode (Node)) {\r
     ASSERT (0);\r
@@ -300,11 +312,11 @@ AmlUnwindSpecialInteger (
   }\r
 \r
   Status = AmlCreateDataNode (\r
-              EAmlNodeDataTypeUInt,\r
-              &Value,\r
-              sizeof (UINT8),\r
-              (AML_DATA_NODE**)&NewDataNode\r
-              );\r
+             EAmlNodeDataTypeUInt,\r
+             &Value,\r
+             sizeof (UINT8),\r
+             (AML_DATA_NODE **)&NewDataNode\r
+             );\r
   if (EFI_ERROR (Status)) {\r
     ASSERT (0);\r
     return Status;\r
@@ -323,10 +335,10 @@ AmlUnwindSpecialInteger (
 \r
   // Add the data node as the first fixed argument of the ByteOp object.\r
   Status = AmlSetFixedArgument (\r
-              (AML_OBJECT_NODE*)Node,\r
-              EAmlParseIndexTerm0,\r
-              (AML_NODE_HEADER*)NewDataNode\r
-              );\r
+             (AML_OBJECT_NODE *)Node,\r
+             EAmlParseIndexTerm0,\r
+             (AML_NODE_HEADER *)NewDataNode\r
+             );\r
   if (EFI_ERROR (Status)) {\r
     ASSERT (0);\r
     goto error_handler;\r
@@ -335,7 +347,7 @@ AmlUnwindSpecialInteger (
   return Status;\r
 \r
 error_handler:\r
-  AmlDeleteTree ((AML_NODE_HEADER*)NewDataNode);\r
+  AmlDeleteTree ((AML_NODE_HEADER *)NewDataNode);\r
   return Status;\r
 }\r
 \r
@@ -360,20 +372,21 @@ error_handler:
 EFI_STATUS\r
 EFIAPI\r
 AmlNodeSetIntegerValue (\r
-  IN  AML_OBJECT_NODE   * Node,\r
-  IN  UINT64              NewValue,\r
-  OUT INT8              * ValueWidthDiff\r
+  IN  AML_OBJECT_NODE  *Node,\r
+  IN  UINT64           NewValue,\r
+  OUT INT8             *ValueWidthDiff\r
   )\r
 {\r
-  EFI_STATUS        Status;\r
-  AML_DATA_NODE   * DataNode;\r
+  EFI_STATUS     Status;\r
+  AML_DATA_NODE  *DataNode;\r
 \r
-  UINT8             NewOpCode;\r
-  UINT8             NumberOfBytes;\r
+  UINT8  NewOpCode;\r
+  UINT8  NumberOfBytes;\r
 \r
   if ((!IsIntegerNode (Node)            &&\r
        !IsSpecialIntegerNode (Node))    ||\r
-      (ValueWidthDiff == NULL)) {\r
+      (ValueWidthDiff == NULL))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -396,6 +409,7 @@ AmlNodeSetIntegerValue (
           ASSERT (0);\r
           return Status;\r
         }\r
+\r
         // The AmlUnwindSpecialInteger functions converts a special integer\r
         // node to a UInt8/Byte data node. Thus, the size increments by one:\r
         // special integer are encoded as one byte (the opcode only) while byte\r
@@ -406,9 +420,10 @@ AmlNodeSetIntegerValue (
   } // IsSpecialIntegerNode (Node)\r
 \r
   // For integer nodes, the value is in the first fixed argument.\r
-  DataNode = (AML_DATA_NODE*)Node->FixedArgs[EAmlParseIndexTerm0];\r
+  DataNode = (AML_DATA_NODE *)Node->FixedArgs[EAmlParseIndexTerm0];\r
   if (!IS_AML_DATA_NODE (DataNode) ||\r
-      (DataNode->DataType != EAmlNodeDataTypeUInt)) {\r
+      (DataNode->DataType != EAmlNodeDataTypeUInt))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -416,7 +431,7 @@ AmlNodeSetIntegerValue (
   // The value can be encoded with a special 0 or 1 OpCode.\r
   // The AML_ONES_OP is not handled.\r
   if (NewValue <= 1) {\r
-    NewOpCode = (NewValue == 0) ? AML_ZERO_OP : AML_ONE_OP;\r
+    NewOpCode             = (NewValue == 0) ? AML_ZERO_OP : AML_ONE_OP;\r
     Node->AmlByteEncoding = AmlGetByteEncodingByOpCode (NewOpCode, 0);\r
 \r
     // The value is encoded with a AML_ZERO_OP or AML_ONE_OP.\r
@@ -425,9 +440,9 @@ AmlNodeSetIntegerValue (
     *ValueWidthDiff = -((INT8)DataNode->Size);\r
 \r
     // Detach and free the DataNode containing the integer value.\r
-    DataNode->NodeHeader.Parent = NULL;\r
+    DataNode->NodeHeader.Parent          = NULL;\r
     Node->FixedArgs[EAmlParseIndexTerm0] = NULL;\r
-    Status = AmlDeleteNode ((AML_NODE_HEADER*)DataNode);\r
+    Status                               = AmlDeleteNode ((AML_NODE_HEADER *)DataNode);\r
     if (EFI_ERROR (Status)) {\r
       ASSERT (0);\r
       return Status;\r
@@ -439,19 +454,19 @@ AmlNodeSetIntegerValue (
   // Check the number of bits needed to represent the value.\r
   if (NewValue > MAX_UINT32) {\r
     // Value is 64 bits.\r
-    NewOpCode = AML_QWORD_PREFIX;\r
+    NewOpCode     = AML_QWORD_PREFIX;\r
     NumberOfBytes = 8;\r
   } else if (NewValue > MAX_UINT16) {\r
     // Value is 32 bits.\r
-    NewOpCode = AML_DWORD_PREFIX;\r
+    NewOpCode     = AML_DWORD_PREFIX;\r
     NumberOfBytes = 4;\r
   } else if (NewValue > MAX_UINT8) {\r
     // Value is 16 bits.\r
-    NewOpCode = AML_WORD_PREFIX;\r
+    NewOpCode     = AML_WORD_PREFIX;\r
     NumberOfBytes = 2;\r
   } else {\r
     // Value is 8 bits.\r
-    NewOpCode = AML_BYTE_PREFIX;\r
+    NewOpCode     = AML_BYTE_PREFIX;\r
     NumberOfBytes = 1;\r
   }\r
 \r
@@ -473,6 +488,7 @@ AmlNodeSetIntegerValue (
       ASSERT (0);\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
+\r
     DataNode->Size = NumberOfBytes;\r
   }\r
 \r
@@ -504,14 +520,14 @@ STATIC
 EFI_STATUS\r
 EFIAPI\r
 AmlNodeUpdateIntegerValue (\r
-  IN  AML_OBJECT_NODE       * IntegerNode,\r
-  IN  BOOLEAN                 IsIncrement,\r
-  IN  UINT64                  Diff,\r
-  OUT INT8                  * ValueWidthDiff\r
+  IN  AML_OBJECT_NODE  *IntegerNode,\r
+  IN  BOOLEAN          IsIncrement,\r
+  IN  UINT64           Diff,\r
+  OUT INT8             *ValueWidthDiff\r
   )\r
 {\r
-  EFI_STATUS       Status;\r
-  UINT64           Value;\r
+  EFI_STATUS  Status;\r
+  UINT64      Value;\r
 \r
   if (ValueWidthDiff == NULL) {\r
     ASSERT (0);\r
@@ -528,7 +544,8 @@ AmlNodeUpdateIntegerValue (
 \r
   // Check for UINT64 over/underflow.\r
   if ((IsIncrement && (Value > (MAX_UINT64 - Diff))) ||\r
-      (!IsIncrement && (Value < Diff))) {\r
+      (!IsIncrement && (Value < Diff)))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -570,29 +587,30 @@ STATIC
 EFI_STATUS\r
 EFIAPI\r
 AmlPropagateSize (\r
-  IN  AML_NODE_HEADER   * Node,\r
-  IN  BOOLEAN             IsIncrement,\r
-  IN  UINT32            * Diff\r
+  IN  AML_NODE_HEADER  *Node,\r
+  IN  BOOLEAN          IsIncrement,\r
+  IN  UINT32           *Diff\r
   )\r
 {\r
-  EFI_STATUS         Status;\r
-  AML_OBJECT_NODE  * ObjectNode;\r
-  AML_NODE_HEADER  * ParentNode;\r
+  EFI_STATUS       Status;\r
+  AML_OBJECT_NODE  *ObjectNode;\r
+  AML_NODE_HEADER  *ParentNode;\r
 \r
-  UINT32             Value;\r
-  UINT32             InitialPkgLenWidth;\r
-  UINT32             NewPkgLenWidth;\r
-  UINT32             ReComputedPkgLenWidth;\r
-  INT8               FieldWidthChange;\r
+  UINT32  Value;\r
+  UINT32  InitialPkgLenWidth;\r
+  UINT32  NewPkgLenWidth;\r
+  UINT32  ReComputedPkgLenWidth;\r
+  INT8    FieldWidthChange;\r
 \r
   if (!IS_AML_OBJECT_NODE (Node) &&\r
-      !IS_AML_ROOT_NODE (Node)) {\r
+      !IS_AML_ROOT_NODE (Node))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   if (IS_AML_OBJECT_NODE (Node)) {\r
-    ObjectNode = (AML_OBJECT_NODE*)Node;\r
+    ObjectNode = (AML_OBJECT_NODE *)Node;\r
 \r
     // For BufferOp, the buffer size is stored in BufferSize. Therefore,\r
     // BufferOp needs special handling to update the BufferSize.\r
@@ -606,10 +624,10 @@ AmlPropagateSize (
       // (can be a BYTE, WORD, DWORD or QWORD).\r
       // BufferSize is an object node.\r
       Status = AmlNodeUpdateIntegerValue (\r
-                 (AML_OBJECT_NODE*)AmlGetFixedArgument (\r
-                                     ObjectNode,\r
-                                     EAmlParseIndexTerm0\r
-                                     ),\r
+                 (AML_OBJECT_NODE *)AmlGetFixedArgument (\r
+                                      ObjectNode,\r
+                                      EAmlParseIndexTerm0\r
+                                      ),\r
                  IsIncrement,\r
                  (UINT64)(*Diff),\r
                  &FieldWidthChange\r
@@ -624,7 +642,8 @@ AmlPropagateSize (
       if ((IsIncrement              &&\r
            (FieldWidthChange < 0))  ||\r
           (!IsIncrement             &&\r
-           (FieldWidthChange > 0))) {\r
+           (FieldWidthChange > 0)))\r
+      {\r
         ASSERT (0);\r
         return EFI_INVALID_PARAMETER;\r
       }\r
@@ -647,13 +666,14 @@ AmlPropagateSize (
       // Subtract the size of the PkgLen encoding. The size of the PkgLen\r
       // encoding must be computed after having updated Value.\r
       InitialPkgLenWidth = AmlComputePkgLengthWidth (Value);\r
-      Value -= InitialPkgLenWidth;\r
+      Value             -= InitialPkgLenWidth;\r
 \r
       // Check for an over/underflows.\r
       // PkgLen is a 28 bit value, cf 20.2.4 Package Length Encoding\r
       // i.e. the maximum value is (2^28 - 1) = ((BIT0 << 28) - 1).\r
       if ((IsIncrement && ((((BIT0 << 28) - 1) - Value) < *Diff))   ||\r
-          (!IsIncrement && (Value < *Diff))) {\r
+          (!IsIncrement && (Value < *Diff)))\r
+      {\r
         ASSERT (0);\r
         return EFI_INVALID_PARAMETER;\r
       }\r
@@ -682,7 +702,8 @@ AmlPropagateSize (
       ReComputedPkgLenWidth = AmlComputePkgLengthWidth (Value);\r
       if (ReComputedPkgLenWidth != NewPkgLenWidth) {\r
         if ((ReComputedPkgLenWidth != 0)   &&\r
-            (ReComputedPkgLenWidth < 4)) {\r
+            (ReComputedPkgLenWidth < 4))\r
+        {\r
           // No need to recompute the PkgLen since a new threshold cannot\r
           // be reached by incrementing the value by one.\r
           Value += 1;\r
@@ -693,8 +714,8 @@ AmlPropagateSize (
       }\r
 \r
       *Diff += (InitialPkgLenWidth > ReComputedPkgLenWidth) ?\r
-                 (InitialPkgLenWidth - ReComputedPkgLenWidth) :\r
-                 (ReComputedPkgLenWidth - InitialPkgLenWidth);\r
+               (InitialPkgLenWidth - ReComputedPkgLenWidth) :\r
+               (ReComputedPkgLenWidth - InitialPkgLenWidth);\r
       ObjectNode->PkgLen = Value;\r
     } // PkgLen update.\r
 \r
@@ -702,7 +723,7 @@ AmlPropagateSize (
     // there is no root node at the top of the tree. Stop\r
     // propagating the new size when finding a root node\r
     // OR when a NULL parent is found.\r
-    ParentNode = AmlGetParent ((AML_NODE_HEADER*)Node);\r
+    ParentNode = AmlGetParent ((AML_NODE_HEADER *)Node);\r
     if (ParentNode != NULL) {\r
       // Propagate the size up the tree.\r
       Status = AmlPropagateSize (\r
@@ -715,14 +736,14 @@ AmlPropagateSize (
         return Status;\r
       }\r
     }\r
-\r
   } else if (IS_AML_ROOT_NODE (Node)) {\r
     // Update the length field in the SDT header.\r
-    Value = ((AML_ROOT_NODE*)Node)->SdtHeader->Length;\r
+    Value = ((AML_ROOT_NODE *)Node)->SdtHeader->Length;\r
 \r
     // Check for an over/underflows.\r
     if ((IsIncrement && (Value > (MAX_UINT32 - *Diff))) ||\r
-        (!IsIncrement && (Value < *Diff))) {\r
+        (!IsIncrement && (Value < *Diff)))\r
+    {\r
       ASSERT (0);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
@@ -734,7 +755,7 @@ AmlPropagateSize (
       Value -= *Diff;\r
     }\r
 \r
-    ((AML_ROOT_NODE*)Node)->SdtHeader->Length = Value;\r
+    ((AML_ROOT_NODE *)Node)->SdtHeader->Length = Value;\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -762,21 +783,22 @@ STATIC
 EFI_STATUS\r
 EFIAPI\r
 AmlPropagateNodeCount (\r
-  IN  AML_OBJECT_NODE   * ObjectNode,\r
-  IN  BOOLEAN             IsIncrement,\r
-  IN  UINT8               NodeCount,\r
-  OUT INT8              * FieldWidthChange\r
+  IN  AML_OBJECT_NODE  *ObjectNode,\r
+  IN  BOOLEAN          IsIncrement,\r
+  IN  UINT8            NodeCount,\r
+  OUT INT8             *FieldWidthChange\r
   )\r
 {\r
-  EFI_STATUS         Status;\r
+  EFI_STATUS  Status;\r
 \r
-  AML_NODE_HEADER  * NodeCountArg;\r
-  UINT8              CurrNodeCount;\r
+  AML_NODE_HEADER  *NodeCountArg;\r
+  UINT8            CurrNodeCount;\r
 \r
   // Currently there is no use case where (NodeCount > 1).\r
   if (!IS_AML_OBJECT_NODE (ObjectNode)  ||\r
       (FieldWidthChange == NULL)        ||\r
-      (NodeCount > 1)) {\r
+      (NodeCount > 1))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -797,25 +819,26 @@ AmlPropagateNodeCount (
     // in the package. It is an UINT8.\r
 \r
     // Check for over/underflow.\r
-    CurrNodeCount = *(((AML_DATA_NODE*)NodeCountArg)->Buffer);\r
+    CurrNodeCount = *(((AML_DATA_NODE *)NodeCountArg)->Buffer);\r
     if ((IsIncrement && (CurrNodeCount == MAX_UINT8)) ||\r
-        (!IsIncrement && (CurrNodeCount == 0))) {\r
+        (!IsIncrement && (CurrNodeCount == 0)))\r
+    {\r
       ASSERT (0);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
     // Update the node count in the DataNode.\r
-    CurrNodeCount = IsIncrement ? (CurrNodeCount + 1) : (CurrNodeCount - 1);\r
-    *(((AML_DATA_NODE*)NodeCountArg)->Buffer) =  CurrNodeCount;\r
+    CurrNodeCount                              = IsIncrement ? (CurrNodeCount + 1) : (CurrNodeCount - 1);\r
+    *(((AML_DATA_NODE *)NodeCountArg)->Buffer) =  CurrNodeCount;\r
   } else if (AmlNodeCompareOpCode (ObjectNode, AML_VAR_PACKAGE_OP, 0)) {\r
     // First fixed argument of PackageOp stores the number of elements\r
     // in the package. It is an integer (can be a BYTE, WORD, DWORD, QWORD).\r
     Status = AmlNodeUpdateIntegerValue (\r
-                (AML_OBJECT_NODE*)NodeCountArg,\r
-                IsIncrement,\r
-                NodeCount,\r
-                FieldWidthChange\r
-                );\r
+               (AML_OBJECT_NODE *)NodeCountArg,\r
+               IsIncrement,\r
+               NodeCount,\r
+               FieldWidthChange\r
+               );\r
     if (EFI_ERROR (Status)) {\r
       ASSERT (0);\r
       return Status;\r
@@ -845,10 +868,10 @@ AmlPropagateNodeCount (
 EFI_STATUS\r
 EFIAPI\r
 AmlPropagateInformation (\r
-  IN  AML_NODE_HEADER   * Node,\r
-  IN  BOOLEAN             IsIncrement,\r
-  IN  UINT32              Diff,\r
-  IN  UINT8               NodeCount\r
+  IN  AML_NODE_HEADER  *Node,\r
+  IN  BOOLEAN          IsIncrement,\r
+  IN  UINT32           Diff,\r
+  IN  UINT8            NodeCount\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -857,7 +880,8 @@ AmlPropagateInformation (
   // Currently there is no use case where (NodeCount > 1).\r
   if ((!IS_AML_ROOT_NODE (Node)     &&\r
        !IS_AML_OBJECT_NODE (Node))  ||\r
-      (NodeCount > 1)) {\r
+      (NodeCount > 1))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -865,9 +889,10 @@ AmlPropagateInformation (
   // Propagate the node count first as it may change the number of bytes\r
   // needed to store the node count, and then impact FieldWidthChange.\r
   if ((NodeCount != 0) &&\r
-      IS_AML_OBJECT_NODE (Node)) {\r
+      IS_AML_OBJECT_NODE (Node))\r
+  {\r
     Status = AmlPropagateNodeCount (\r
-               (AML_OBJECT_NODE*)Node,\r
+               (AML_OBJECT_NODE *)Node,\r
                IsIncrement,\r
                NodeCount,\r
                &FieldWidthChange\r
@@ -881,14 +906,16 @@ AmlPropagateInformation (
     // Maximum change is between UINT8/UINT64: 8 bytes.\r
     if ((ABS (FieldWidthChange) > 8)                          ||\r
         (IsIncrement                                          &&\r
-          ((FieldWidthChange < 0)                             ||\r
-           ((Diff + (UINT8)FieldWidthChange) > MAX_UINT32)))  ||\r
+         ((FieldWidthChange < 0)                             ||\r
+          ((Diff + (UINT8)FieldWidthChange) > MAX_UINT32)))  ||\r
         (!IsIncrement                                         &&\r
-          ((FieldWidthChange > 0)                             ||\r
-           (Diff < (UINT32)ABS (FieldWidthChange))))) {\r
+         ((FieldWidthChange > 0)                             ||\r
+          (Diff < (UINT32)ABS (FieldWidthChange)))))\r
+    {\r
       ASSERT (0);\r
       return EFI_INVALID_PARAMETER;\r
     }\r
+\r
     Diff = (UINT32)(Diff + (UINT8)ABS (FieldWidthChange));\r
   }\r
 \r
@@ -932,13 +959,13 @@ AmlPropagateInformation (
 EFI_STATUS\r
 EFIAPI\r
 AmlSetRdListCheckSum (\r
-  IN  AML_OBJECT_NODE   * BufferOpNode,\r
-  IN  UINT8               CheckSum\r
+  IN  AML_OBJECT_NODE  *BufferOpNode,\r
+  IN  UINT8            CheckSum\r
   )\r
 {\r
-  EFI_STATUS        Status;\r
-  AML_DATA_NODE   * LastRdNode;\r
-  AML_RD_HEADER     RdDataType;\r
+  EFI_STATUS     Status;\r
+  AML_DATA_NODE  *LastRdNode;\r
+  AML_RD_HEADER  RdDataType;\r
 \r
   if (!AmlNodeCompareOpCode (BufferOpNode, AML_BUFFER_OP, 0)) {\r
     ASSERT (0);\r
@@ -947,13 +974,14 @@ AmlSetRdListCheckSum (
 \r
   // Get the last Resource data node in the variable list of\r
   // argument of the BufferOp node.\r
-  LastRdNode = (AML_DATA_NODE*)AmlGetPreviousVariableArgument (\r
-                                 (AML_NODE_HEADER*)BufferOpNode,\r
-                                 NULL\r
-                                 );\r
+  LastRdNode = (AML_DATA_NODE *)AmlGetPreviousVariableArgument (\r
+                                  (AML_NODE_HEADER *)BufferOpNode,\r
+                                  NULL\r
+                                  );\r
   if ((LastRdNode == NULL)             ||\r
       !IS_AML_DATA_NODE (LastRdNode)   ||\r
-      (LastRdNode->DataType != EAmlNodeDataTypeResourceData)) {\r
+      (LastRdNode->DataType != EAmlNodeDataTypeResourceData))\r
+  {\r
     ASSERT (0);\r
     return EFI_INVALID_PARAMETER;\r
   }\r
@@ -968,8 +996,10 @@ AmlSetRdListCheckSum (
   // It is possible to have only one Resource Data in a BufferOp with\r
   // no EndTag. Return EFI_NOT_FOUND is such case.\r
   if (!AmlRdCompareDescId (\r
-        &RdDataType,\r
-        AML_RD_BUILD_SMALL_DESC_ID (ACPI_SMALL_END_TAG_DESCRIPTOR_NAME))) {\r
+         &RdDataType,\r
+         AML_RD_BUILD_SMALL_DESC_ID (ACPI_SMALL_END_TAG_DESCRIPTOR_NAME)\r
+         ))\r
+  {\r
     ASSERT (0);\r
     return EFI_NOT_FOUND;\r
   }\r