]> git.proxmox.com Git - mirror_edk2.git/blobdiff - SecurityPkg/Tcg/TcgDxe/TcgDxe.c
SecurityPkg: Apply uncrustify changes
[mirror_edk2.git] / SecurityPkg / Tcg / TcgDxe / TcgDxe.c
index 7019c666b1b0b5dd55449d3c0430ad97a707329a..ee6c6273033bde5120d02641ca8da2da382c58b6 100644 (file)
@@ -52,29 +52,27 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
   BASE_CR (this, TCG_DXE_DATA, TcgProtocol)\r
 \r
 typedef struct _TCG_DXE_DATA {\r
-  EFI_TCG_PROTOCOL                  TcgProtocol;\r
-  TCG_EFI_BOOT_SERVICE_CAPABILITY   BsCap;\r
-  EFI_TCG_CLIENT_ACPI_TABLE         *TcgClientAcpiTable;\r
-  EFI_TCG_SERVER_ACPI_TABLE         *TcgServerAcpiTable;\r
-  UINTN                             EventLogSize;\r
-  UINT8                             *LastEvent;\r
+  EFI_TCG_PROTOCOL                   TcgProtocol;\r
+  TCG_EFI_BOOT_SERVICE_CAPABILITY    BsCap;\r
+  EFI_TCG_CLIENT_ACPI_TABLE          *TcgClientAcpiTable;\r
+  EFI_TCG_SERVER_ACPI_TABLE          *TcgServerAcpiTable;\r
+  UINTN                              EventLogSize;\r
+  UINT8                              *LastEvent;\r
 } TCG_DXE_DATA;\r
 \r
-\r
-\r
-EFI_TCG_CLIENT_ACPI_TABLE           mTcgClientAcpiTemplate = {\r
+EFI_TCG_CLIENT_ACPI_TABLE  mTcgClientAcpiTemplate = {\r
   {\r
     EFI_ACPI_3_0_TRUSTED_COMPUTING_PLATFORM_ALLIANCE_CAPABILITIES_TABLE_SIGNATURE,\r
     sizeof (mTcgClientAcpiTemplate),\r
-    0x02                      //Revision\r
+    0x02                      // Revision\r
     //\r
     // Compiler initializes the remaining bytes to 0\r
     // These fields should be filled in in production\r
     //\r
   },\r
-  0,                          // 0 for PC Client Platform Class\r
-  0,                          // Log Area Max Length\r
-  (EFI_PHYSICAL_ADDRESS) (SIZE_4GB - 1)  // Log Area Start Address\r
+  0,                                   // 0 for PC Client Platform Class\r
+  0,                                   // Log Area Max Length\r
+  (EFI_PHYSICAL_ADDRESS)(SIZE_4GB - 1) // Log Area Start Address\r
 };\r
 \r
 //\r
@@ -83,26 +81,26 @@ EFI_TCG_CLIENT_ACPI_TABLE           mTcgClientAcpiTemplate = {
 // this _UID can be changed and should match with the _UID setting of the TPM\r
 // ACPI device object\r
 //\r
-EFI_TCG_SERVER_ACPI_TABLE           mTcgServerAcpiTemplate = {\r
+EFI_TCG_SERVER_ACPI_TABLE  mTcgServerAcpiTemplate = {\r
   {\r
     EFI_ACPI_3_0_TRUSTED_COMPUTING_PLATFORM_ALLIANCE_CAPABILITIES_TABLE_SIGNATURE,\r
     sizeof (mTcgServerAcpiTemplate),\r
-    0x02                      //Revision\r
+    0x02                      // Revision\r
     //\r
     // Compiler initializes the remaining bytes to 0\r
     // These fields should be filled in in production\r
     //\r
   },\r
-  1,                          // 1 for Server Platform Class\r
-  0,                          // Reserved\r
-  0,                          // Log Area Max Length\r
-  (EFI_PHYSICAL_ADDRESS) (SIZE_4GB - 1), // Log Area Start Address\r
-  0x0120,                     // TCG Specification revision 1.2\r
-  0,                          // Device Flags\r
-  0,                          // Interrupt Flags\r
-  0,                          // GPE\r
-  {0},                        // Reserved 3 bytes\r
-  0,                          // Global System Interrupt\r
+  1,                                    // 1 for Server Platform Class\r
+  0,                                    // Reserved\r
+  0,                                    // Log Area Max Length\r
+  (EFI_PHYSICAL_ADDRESS)(SIZE_4GB - 1), // Log Area Start Address\r
+  0x0120,                               // TCG Specification revision 1.2\r
+  0,                                    // Device Flags\r
+  0,                                    // Interrupt Flags\r
+  0,                                    // GPE\r
+  { 0 },                                // Reserved 3 bytes\r
+  0,                                    // Global System Interrupt\r
   {\r
     EFI_ACPI_3_0_SYSTEM_MEMORY,\r
     0,\r
@@ -111,15 +109,15 @@ EFI_TCG_SERVER_ACPI_TABLE           mTcgServerAcpiTemplate = {
     0                         // Base Address\r
   },\r
   0,                          // Reserved\r
-  {0},                        // Configuration Address\r
+  { 0 },                      // Configuration Address\r
   0xFF,                       // ACPI _UID value of the device, can be changed for different platforms\r
   0,                          // ACPI _UID value of the device, can be changed for different platforms\r
   0,                          // ACPI _UID value of the device, can be changed for different platforms\r
   0                           // ACPI _UID value of the device, can be changed for different platforms\r
 };\r
 \r
-UINTN  mBootAttempts  = 0;\r
-CHAR16 mBootVarName[] = L"BootOrder";\r
+UINTN   mBootAttempts  = 0;\r
+CHAR16  mBootVarName[] = L"BootOrder";\r
 \r
 /**\r
   Get All processors EFI_CPU_LOCATION in system. LocationBuf is allocated inside the function\r
@@ -134,19 +132,19 @@ CHAR16 mBootVarName[] = L"BootOrder";
 **/\r
 EFI_STATUS\r
 GetProcessorsCpuLocation (\r
-    OUT  EFI_CPU_PHYSICAL_LOCATION   **LocationBuf,\r
-    OUT  UINTN                       *Num\r
+  OUT  EFI_CPU_PHYSICAL_LOCATION  **LocationBuf,\r
+  OUT  UINTN                      *Num\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  EFI_MP_SERVICES_PROTOCOL          *MpProtocol;\r
-  UINTN                             ProcessorNum;\r
-  UINTN                             EnabledProcessorNum;\r
-  EFI_PROCESSOR_INFORMATION         ProcessorInfo;\r
-  EFI_CPU_PHYSICAL_LOCATION         *ProcessorLocBuf;\r
-  UINTN                             Index;\r
-\r
-  Status = gBS->LocateProtocol (&gEfiMpServiceProtocolGuid, NULL, (VOID **) &MpProtocol);\r
+  EFI_STATUS                 Status;\r
+  EFI_MP_SERVICES_PROTOCOL   *MpProtocol;\r
+  UINTN                      ProcessorNum;\r
+  UINTN                      EnabledProcessorNum;\r
+  EFI_PROCESSOR_INFORMATION  ProcessorInfo;\r
+  EFI_CPU_PHYSICAL_LOCATION  *ProcessorLocBuf;\r
+  UINTN                      Index;\r
+\r
+  Status = gBS->LocateProtocol (&gEfiMpServiceProtocolGuid, NULL, (VOID **)&MpProtocol);\r
   if (EFI_ERROR (Status)) {\r
     //\r
     // MP protocol is not installed\r
@@ -154,21 +152,21 @@ GetProcessorsCpuLocation (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  Status = MpProtocol->GetNumberOfProcessors(\r
+  Status = MpProtocol->GetNumberOfProcessors (\r
                          MpProtocol,\r
                          &ProcessorNum,\r
                          &EnabledProcessorNum\r
                          );\r
-  if (EFI_ERROR(Status)){\r
+  if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
 \r
-  Status = gBS->AllocatePool(\r
+  Status = gBS->AllocatePool (\r
                   EfiBootServicesData,\r
-                  sizeof(EFI_CPU_PHYSICAL_LOCATION) * ProcessorNum,\r
-                  (VOID **) &ProcessorLocBuf\r
+                  sizeof (EFI_CPU_PHYSICAL_LOCATION) * ProcessorNum,\r
+                  (VOID **)&ProcessorLocBuf\r
                   );\r
-  if (EFI_ERROR(Status)){\r
+  if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
 \r
@@ -176,28 +174,28 @@ GetProcessorsCpuLocation (
   // Get each processor Location info\r
   //\r
   for (Index = 0; Index < ProcessorNum; Index++) {\r
-    Status = MpProtocol->GetProcessorInfo(\r
+    Status = MpProtocol->GetProcessorInfo (\r
                            MpProtocol,\r
                            Index,\r
                            &ProcessorInfo\r
                            );\r
-    if (EFI_ERROR(Status)){\r
-      FreePool(ProcessorLocBuf);\r
+    if (EFI_ERROR (Status)) {\r
+      FreePool (ProcessorLocBuf);\r
       return Status;\r
     }\r
 \r
     //\r
     // Get all Processor Location info & measure\r
     //\r
-    CopyMem(\r
+    CopyMem (\r
       &ProcessorLocBuf[Index],\r
       &ProcessorInfo.Location,\r
-      sizeof(EFI_CPU_PHYSICAL_LOCATION)\r
+      sizeof (EFI_CPU_PHYSICAL_LOCATION)\r
       );\r
   }\r
 \r
   *LocationBuf = ProcessorLocBuf;\r
-  *Num = ProcessorNum;\r
+  *Num         = ProcessorNum;\r
 \r
   return Status;\r
 }\r
@@ -234,7 +232,7 @@ TcgDxeStatusCheck (
   OUT     EFI_PHYSICAL_ADDRESS             *EventLogLastEntry\r
   )\r
 {\r
-  TCG_DXE_DATA                      *TcgData;\r
+  TCG_DXE_DATA  *TcgData;\r
 \r
   TcgData = TCG_DXE_DATA_FROM_THIS (This);\r
 \r
@@ -320,15 +318,15 @@ TpmCommHashAll (
 EFI_STATUS\r
 EFIAPI\r
 TcgDxeHashAll (\r
-  IN      EFI_TCG_PROTOCOL          *This,\r
-  IN      UINT8                     *HashData,\r
-  IN      UINT64                    HashDataLen,\r
-  IN      TCG_ALGORITHM_ID          AlgorithmId,\r
-  IN OUT  UINT64                    *HashedDataLen,\r
-  IN OUT  UINT8                     **HashedDataResult\r
+  IN      EFI_TCG_PROTOCOL  *This,\r
+  IN      UINT8             *HashData,\r
+  IN      UINT64            HashDataLen,\r
+  IN      TCG_ALGORITHM_ID  AlgorithmId,\r
+  IN OUT  UINT64            *HashedDataLen,\r
+  IN OUT  UINT8             **HashedDataResult\r
   )\r
 {\r
-  if (HashedDataLen == NULL || HashedDataResult == NULL) {\r
+  if ((HashedDataLen == NULL) || (HashedDataResult == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -336,7 +334,7 @@ TcgDxeHashAll (
     case TPM_ALG_SHA:\r
       if (*HashedDataLen == 0) {\r
         *HashedDataLen    = sizeof (TPM_DIGEST);\r
-        *HashedDataResult = AllocatePool ((UINTN) *HashedDataLen);\r
+        *HashedDataResult = AllocatePool ((UINTN)*HashedDataLen);\r
         if (*HashedDataResult == NULL) {\r
           return EFI_OUT_OF_RESOURCES;\r
         }\r
@@ -346,16 +344,17 @@ TcgDxeHashAll (
         *HashedDataLen = sizeof (TPM_DIGEST);\r
         return EFI_BUFFER_TOO_SMALL;\r
       }\r
+\r
       *HashedDataLen = sizeof (TPM_DIGEST);\r
 \r
       if (*HashedDataResult == NULL) {\r
-        *HashedDataResult = AllocatePool ((UINTN) *HashedDataLen);\r
+        *HashedDataResult = AllocatePool ((UINTN)*HashedDataLen);\r
       }\r
 \r
       return TpmCommHashAll (\r
                HashData,\r
-               (UINTN) HashDataLen,\r
-               (TPM_DIGEST*)*HashedDataResult\r
+               (UINTN)HashDataLen,\r
+               (TPM_DIGEST *)*HashedDataResult\r
                );\r
     default:\r
       return EFI_UNSUPPORTED;\r
@@ -377,19 +376,19 @@ Add a new entry to the Event Log.
 **/\r
 EFI_STATUS\r
 TpmCommLogEvent (\r
-  IN OUT  UINT8                     **EventLogPtr,\r
-  IN OUT  UINTN                     *LogSize,\r
-  IN      UINTN                     MaxSize,\r
-  IN      TCG_PCR_EVENT_HDR         *NewEventHdr,\r
-  IN      UINT8                     *NewEventData\r
+  IN OUT  UINT8              **EventLogPtr,\r
+  IN OUT  UINTN              *LogSize,\r
+  IN      UINTN              MaxSize,\r
+  IN      TCG_PCR_EVENT_HDR  *NewEventHdr,\r
+  IN      UINT8              *NewEventData\r
   )\r
 {\r
-  UINTN                            NewLogSize;\r
+  UINTN  NewLogSize;\r
 \r
   //\r
   // Prevent Event Overflow\r
   //\r
-  if ((UINTN) NewEventHdr->EventSize > MAX_UINTN - sizeof (*NewEventHdr)) {\r
+  if ((UINTN)NewEventHdr->EventSize > MAX_UINTN - sizeof (*NewEventHdr)) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
@@ -399,7 +398,7 @@ TpmCommLogEvent (
   }\r
 \r
   *EventLogPtr += *LogSize;\r
-  *LogSize += NewLogSize;\r
+  *LogSize     += NewLogSize;\r
   CopyMem (*EventLogPtr, NewEventHdr, sizeof (*NewEventHdr));\r
   CopyMem (\r
     *EventLogPtr + sizeof (*NewEventHdr),\r
@@ -423,13 +422,13 @@ TpmCommLogEvent (
 EFI_STATUS\r
 EFIAPI\r
 TcgDxeLogEventI (\r
-  IN      TCG_DXE_DATA              *TcgData,\r
-  IN      TCG_PCR_EVENT_HDR         *NewEventHdr,\r
-  IN      UINT8                     *NewEventData\r
+  IN      TCG_DXE_DATA       *TcgData,\r
+  IN      TCG_PCR_EVENT_HDR  *NewEventHdr,\r
+  IN      UINT8              *NewEventData\r
   )\r
 {\r
   if (PcdGet8 (PcdTpmPlatformClass) == TCG_PLATFORM_TYPE_CLIENT) {\r
-    TcgData->LastEvent = (UINT8*)(UINTN)TcgData->TcgClientAcpiTable->Lasa;\r
+    TcgData->LastEvent = (UINT8 *)(UINTN)TcgData->TcgClientAcpiTable->Lasa;\r
     return TpmCommLogEvent (\r
              &TcgData->LastEvent,\r
              &TcgData->EventLogSize,\r
@@ -438,7 +437,7 @@ TcgDxeLogEventI (
              NewEventData\r
              );\r
   } else {\r
-    TcgData->LastEvent = (UINT8*)(UINTN)TcgData->TcgServerAcpiTable->Lasa;\r
+    TcgData->LastEvent = (UINT8 *)(UINTN)TcgData->TcgServerAcpiTable->Lasa;\r
     return TpmCommLogEvent (\r
              &TcgData->LastEvent,\r
              &TcgData->EventLogSize,\r
@@ -469,15 +468,15 @@ TcgDxeLogEventI (
 EFI_STATUS\r
 EFIAPI\r
 TcgDxeLogEvent (\r
-  IN      EFI_TCG_PROTOCOL          *This,\r
-  IN      TCG_PCR_EVENT             *TCGLogData,\r
-  IN OUT  UINT32                    *EventNumber,\r
-  IN      UINT32                    Flags\r
+  IN      EFI_TCG_PROTOCOL  *This,\r
+  IN      TCG_PCR_EVENT     *TCGLogData,\r
+  IN OUT  UINT32            *EventNumber,\r
+  IN      UINT32            Flags\r
   )\r
 {\r
   TCG_DXE_DATA  *TcgData;\r
 \r
-  if (TCGLogData == NULL){\r
+  if (TCGLogData == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -486,9 +485,10 @@ TcgDxeLogEvent (
   if (TcgData->BsCap.TPMDeactivatedFlag || (!TcgData->BsCap.TPMPresentFlag)) {\r
     return EFI_DEVICE_ERROR;\r
   }\r
+\r
   return TcgDxeLogEventI (\r
            TcgData,\r
-           (TCG_PCR_EVENT_HDR*)TCGLogData,\r
+           (TCG_PCR_EVENT_HDR *)TCGLogData,\r
            TCGLogData->Event\r
            );\r
 }\r
@@ -511,17 +511,18 @@ TcgDxeLogEvent (
 EFI_STATUS\r
 EFIAPI\r
 TcgDxePassThroughToTpm (\r
-  IN      EFI_TCG_PROTOCOL          *This,\r
-  IN      UINT32                    TpmInputParameterBlockSize,\r
-  IN      UINT8                     *TpmInputParameterBlock,\r
-  IN      UINT32                    TpmOutputParameterBlockSize,\r
-  IN      UINT8                     *TpmOutputParameterBlock\r
+  IN      EFI_TCG_PROTOCOL  *This,\r
+  IN      UINT32            TpmInputParameterBlockSize,\r
+  IN      UINT8             *TpmInputParameterBlock,\r
+  IN      UINT32            TpmOutputParameterBlockSize,\r
+  IN      UINT8             *TpmOutputParameterBlock\r
   )\r
 {\r
-  if (TpmInputParameterBlock == NULL ||\r
-      TpmOutputParameterBlock == NULL ||\r
-      TpmInputParameterBlockSize == 0 ||\r
-      TpmOutputParameterBlockSize == 0) {\r
+  if ((TpmInputParameterBlock == NULL) ||\r
+      (TpmOutputParameterBlock == NULL) ||\r
+      (TpmInputParameterBlockSize == 0) ||\r
+      (TpmOutputParameterBlockSize == 0))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -552,26 +553,26 @@ TcgDxePassThroughToTpm (
 EFI_STATUS\r
 EFIAPI\r
 TcgDxeHashLogExtendEventI (\r
-  IN      TCG_DXE_DATA              *TcgData,\r
-  IN      UINT8                     *HashData,\r
-  IN      UINT64                    HashDataLen,\r
-  IN OUT  TCG_PCR_EVENT_HDR         *NewEventHdr,\r
-  IN      UINT8                     *NewEventData\r
+  IN      TCG_DXE_DATA       *TcgData,\r
+  IN      UINT8              *HashData,\r
+  IN      UINT64             HashDataLen,\r
+  IN OUT  TCG_PCR_EVENT_HDR  *NewEventHdr,\r
+  IN      UINT8              *NewEventData\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
+  EFI_STATUS  Status;\r
 \r
   if (!TcgData->BsCap.TPMPresentFlag) {\r
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
-  if (HashDataLen > 0 || HashData != NULL) {\r
+  if ((HashDataLen > 0) || (HashData != NULL)) {\r
     Status = TpmCommHashAll (\r
                HashData,\r
-               (UINTN) HashDataLen,\r
+               (UINTN)HashDataLen,\r
                &NewEventHdr->Digest\r
                );\r
-    if (EFI_ERROR(Status)) {\r
+    if (EFI_ERROR (Status)) {\r
       DEBUG ((DEBUG_ERROR, "TpmCommHashAll Failed. %x\n", Status));\r
       goto Done;\r
     }\r
@@ -627,19 +628,19 @@ Done:
 EFI_STATUS\r
 EFIAPI\r
 TcgDxeHashLogExtendEvent (\r
-  IN      EFI_TCG_PROTOCOL          *This,\r
-  IN      EFI_PHYSICAL_ADDRESS      HashData,\r
-  IN      UINT64                    HashDataLen,\r
-  IN      TPM_ALGORITHM_ID          AlgorithmId,\r
-  IN OUT  TCG_PCR_EVENT             *TCGLogData,\r
-  IN OUT  UINT32                    *EventNumber,\r
-     OUT  EFI_PHYSICAL_ADDRESS      *EventLogLastEntry\r
+  IN      EFI_TCG_PROTOCOL      *This,\r
+  IN      EFI_PHYSICAL_ADDRESS  HashData,\r
+  IN      UINT64                HashDataLen,\r
+  IN      TPM_ALGORITHM_ID      AlgorithmId,\r
+  IN OUT  TCG_PCR_EVENT         *TCGLogData,\r
+  IN OUT  UINT32                *EventNumber,\r
+  OUT  EFI_PHYSICAL_ADDRESS     *EventLogLastEntry\r
   )\r
 {\r
   TCG_DXE_DATA  *TcgData;\r
   EFI_STATUS    Status;\r
 \r
-  if (TCGLogData == NULL || EventLogLastEntry == NULL){\r
+  if ((TCGLogData == NULL) || (EventLogLastEntry == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -653,26 +654,26 @@ TcgDxeHashLogExtendEvent (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  if (HashData == 0 && HashDataLen > 0) {\r
+  if ((HashData == 0) && (HashDataLen > 0)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   Status = TcgDxeHashLogExtendEventI (\r
              TcgData,\r
-             (UINT8 *) (UINTN) HashData,\r
+             (UINT8 *)(UINTN)HashData,\r
              HashDataLen,\r
-             (TCG_PCR_EVENT_HDR*)TCGLogData,\r
+             (TCG_PCR_EVENT_HDR *)TCGLogData,\r
              TCGLogData->Event\r
              );\r
 \r
-  if (!EFI_ERROR(Status)){\r
-    *EventLogLastEntry = (EFI_PHYSICAL_ADDRESS)(UINTN) TcgData->LastEvent;\r
+  if (!EFI_ERROR (Status)) {\r
+    *EventLogLastEntry = (EFI_PHYSICAL_ADDRESS)(UINTN)TcgData->LastEvent;\r
   }\r
 \r
   return Status;\r
 }\r
 \r
-TCG_DXE_DATA                 mTcgDxeData = {\r
+TCG_DXE_DATA  mTcgDxeData = {\r
   {\r
     TcgDxeStatusCheck,\r
     TcgDxeHashAll,\r
@@ -724,6 +725,7 @@ SetupEventLog (
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
+\r
     mTcgClientAcpiTemplate.Lasa = Lasa;\r
     //\r
     // To initialize them as 0xFF is recommended\r
@@ -731,7 +733,6 @@ SetupEventLog (
     //\r
     SetMem ((VOID *)(UINTN)mTcgClientAcpiTemplate.Lasa, PcdGet32 (PcdTcgLogAreaMinLen), 0xFF);\r
     mTcgClientAcpiTemplate.Laml = PcdGet32 (PcdTcgLogAreaMinLen);\r
-\r
   } else {\r
     Lasa = mTcgServerAcpiTemplate.Lasa;\r
 \r
@@ -744,6 +745,7 @@ SetupEventLog (
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
+\r
     mTcgServerAcpiTemplate.Lasa = Lasa;\r
     //\r
     // To initialize them as 0xFF is recommended\r
@@ -755,14 +757,15 @@ SetupEventLog (
 \r
   GuidHob.Raw = GetHobList ();\r
   while (!EFI_ERROR (Status) &&\r
-         (GuidHob.Raw = GetNextGuidHob (&gTcgEventEntryHobGuid, GuidHob.Raw)) != NULL) {\r
+         (GuidHob.Raw = GetNextGuidHob (&gTcgEventEntryHobGuid, GuidHob.Raw)) != NULL)\r
+  {\r
     TcgEvent    = GET_GUID_HOB_DATA (GuidHob.Guid);\r
     GuidHob.Raw = GET_NEXT_HOB (GuidHob);\r
-    Status = TcgDxeLogEventI (\r
-               &mTcgDxeData,\r
-               (TCG_PCR_EVENT_HDR*)TcgEvent,\r
-               TcgEvent->Event\r
-               );\r
+    Status      = TcgDxeLogEventI (\r
+                    &mTcgDxeData,\r
+                    (TCG_PCR_EVENT_HDR *)TcgEvent,\r
+                    TcgEvent->Event\r
+                    );\r
   }\r
 \r
   return Status;\r
@@ -780,20 +783,20 @@ SetupEventLog (
 EFI_STATUS\r
 EFIAPI\r
 TcgMeasureAction (\r
-  IN      CHAR8                     *String\r
+  IN      CHAR8  *String\r
   )\r
 {\r
-  TCG_PCR_EVENT_HDR                 TcgEvent;\r
+  TCG_PCR_EVENT_HDR  TcgEvent;\r
 \r
   TcgEvent.PCRIndex  = 5;\r
   TcgEvent.EventType = EV_EFI_ACTION;\r
   TcgEvent.EventSize = (UINT32)AsciiStrLen (String);\r
   return TcgDxeHashLogExtendEventI (\r
            &mTcgDxeData,\r
-           (UINT8*)String,\r
+           (UINT8 *)String,\r
            TcgEvent.EventSize,\r
            &TcgEvent,\r
-           (UINT8 *) String\r
+           (UINT8 *)String\r
            );\r
 }\r
 \r
@@ -810,40 +813,40 @@ MeasureHandoffTables (
   VOID\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  TCG_PCR_EVENT_HDR                 TcgEvent;\r
-  EFI_HANDOFF_TABLE_POINTERS        HandoffTables;\r
-  UINTN                             ProcessorNum;\r
-  EFI_CPU_PHYSICAL_LOCATION         *ProcessorLocBuf;\r
+  EFI_STATUS                  Status;\r
+  TCG_PCR_EVENT_HDR           TcgEvent;\r
+  EFI_HANDOFF_TABLE_POINTERS  HandoffTables;\r
+  UINTN                       ProcessorNum;\r
+  EFI_CPU_PHYSICAL_LOCATION   *ProcessorLocBuf;\r
 \r
   ProcessorLocBuf = NULL;\r
-  Status = EFI_SUCCESS;\r
+  Status          = EFI_SUCCESS;\r
 \r
   if (PcdGet8 (PcdTpmPlatformClass) == TCG_PLATFORM_TYPE_SERVER) {\r
     //\r
     // Tcg Server spec.\r
     // Measure each processor EFI_CPU_PHYSICAL_LOCATION with EV_TABLE_OF_DEVICES to PCR[1]\r
     //\r
-    Status = GetProcessorsCpuLocation(&ProcessorLocBuf, &ProcessorNum);\r
+    Status = GetProcessorsCpuLocation (&ProcessorLocBuf, &ProcessorNum);\r
 \r
-    if (!EFI_ERROR(Status)){\r
+    if (!EFI_ERROR (Status)) {\r
       TcgEvent.PCRIndex  = 1;\r
       TcgEvent.EventType = EV_TABLE_OF_DEVICES;\r
       TcgEvent.EventSize = sizeof (HandoffTables);\r
 \r
-      HandoffTables.NumberOfTables = 1;\r
+      HandoffTables.NumberOfTables            = 1;\r
       HandoffTables.TableEntry[0].VendorGuid  = gEfiMpServiceProtocolGuid;\r
       HandoffTables.TableEntry[0].VendorTable = ProcessorLocBuf;\r
 \r
       Status = TcgDxeHashLogExtendEventI (\r
                  &mTcgDxeData,\r
-                 (UINT8*)(UINTN)ProcessorLocBuf,\r
-                 sizeof(EFI_CPU_PHYSICAL_LOCATION) * ProcessorNum,\r
+                 (UINT8 *)(UINTN)ProcessorLocBuf,\r
+                 sizeof (EFI_CPU_PHYSICAL_LOCATION) * ProcessorNum,\r
                  &TcgEvent,\r
-                 (UINT8*)&HandoffTables\r
+                 (UINT8 *)&HandoffTables\r
                  );\r
 \r
-      FreePool(ProcessorLocBuf);\r
+      FreePool (ProcessorLocBuf);\r
     }\r
   }\r
 \r
@@ -862,13 +865,13 @@ MeasureHandoffTables (
 EFI_STATUS\r
 EFIAPI\r
 MeasureSeparatorEvent (\r
-  IN      TPM_PCRINDEX              PCRIndex\r
+  IN      TPM_PCRINDEX  PCRIndex\r
   )\r
 {\r
-  TCG_PCR_EVENT_HDR                 TcgEvent;\r
-  UINT32                            EventData;\r
+  TCG_PCR_EVENT_HDR  TcgEvent;\r
+  UINT32             EventData;\r
 \r
-  EventData = 0;\r
+  EventData          = 0;\r
   TcgEvent.PCRIndex  = PCRIndex;\r
   TcgEvent.EventType = EV_SEPARATOR;\r
   TcgEvent.EventSize = (UINT32)sizeof (EventData);\r
@@ -897,22 +900,22 @@ MeasureSeparatorEvent (
 VOID *\r
 EFIAPI\r
 ReadVariable (\r
-  IN      CHAR16                    *VarName,\r
-  IN      EFI_GUID                  *VendorGuid,\r
-  OUT     UINTN                     *VarSize\r
+  IN      CHAR16    *VarName,\r
+  IN      EFI_GUID  *VendorGuid,\r
+  OUT     UINTN     *VarSize\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  VOID                              *VarData;\r
+  EFI_STATUS  Status;\r
+  VOID        *VarData;\r
 \r
   *VarSize = 0;\r
-  Status = gRT->GetVariable (\r
-                  VarName,\r
-                  VendorGuid,\r
-                  NULL,\r
-                  VarSize,\r
-                  NULL\r
-                  );\r
+  Status   = gRT->GetVariable (\r
+                    VarName,\r
+                    VendorGuid,\r
+                    NULL,\r
+                    VarSize,\r
+                    NULL\r
+                    );\r
   if (Status != EFI_BUFFER_TOO_SMALL) {\r
     return NULL;\r
   }\r
@@ -928,10 +931,11 @@ ReadVariable (
                     );\r
     if (EFI_ERROR (Status)) {\r
       FreePool (VarData);\r
-      VarData = NULL;\r
+      VarData  = NULL;\r
       *VarSize = 0;\r
     }\r
   }\r
+\r
   return VarData;\r
 }\r
 \r
@@ -953,26 +957,26 @@ ReadVariable (
 EFI_STATUS\r
 EFIAPI\r
 MeasureVariable (\r
-  IN      TPM_PCRINDEX              PCRIndex,\r
-  IN      TCG_EVENTTYPE             EventType,\r
-  IN      CHAR16                    *VarName,\r
-  IN      EFI_GUID                  *VendorGuid,\r
-  IN      VOID                      *VarData,\r
-  IN      UINTN                     VarSize\r
+  IN      TPM_PCRINDEX   PCRIndex,\r
+  IN      TCG_EVENTTYPE  EventType,\r
+  IN      CHAR16         *VarName,\r
+  IN      EFI_GUID       *VendorGuid,\r
+  IN      VOID           *VarData,\r
+  IN      UINTN          VarSize\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  TCG_PCR_EVENT_HDR                 TcgEvent;\r
-  UINTN                             VarNameLength;\r
-  EFI_VARIABLE_DATA                 *VarLog;\r
+  EFI_STATUS         Status;\r
+  TCG_PCR_EVENT_HDR  TcgEvent;\r
+  UINTN              VarNameLength;\r
+  EFI_VARIABLE_DATA  *VarLog;\r
 \r
   VarNameLength      = StrLen (VarName);\r
   TcgEvent.PCRIndex  = PCRIndex;\r
   TcgEvent.EventType = EventType;\r
   TcgEvent.EventSize = (UINT32)(sizeof (*VarLog) + VarNameLength * sizeof (*VarName) + VarSize\r
-                        - sizeof (VarLog->UnicodeName) - sizeof (VarLog->VariableData));\r
+                                - sizeof (VarLog->UnicodeName) - sizeof (VarLog->VariableData));\r
 \r
-  VarLog = (EFI_VARIABLE_DATA*)AllocatePool (TcgEvent.EventSize);\r
+  VarLog = (EFI_VARIABLE_DATA *)AllocatePool (TcgEvent.EventSize);\r
   if (VarLog == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -981,22 +985,22 @@ MeasureVariable (
   VarLog->UnicodeNameLength  = VarNameLength;\r
   VarLog->VariableDataLength = VarSize;\r
   CopyMem (\r
-     VarLog->UnicodeName,\r
-     VarName,\r
-     VarNameLength * sizeof (*VarName)\r
-     );\r
+    VarLog->UnicodeName,\r
+    VarName,\r
+    VarNameLength * sizeof (*VarName)\r
+    );\r
   CopyMem (\r
-     (CHAR16 *)VarLog->UnicodeName + VarNameLength,\r
-     VarData,\r
-     VarSize\r
-     );\r
+    (CHAR16 *)VarLog->UnicodeName + VarNameLength,\r
+    VarData,\r
+    VarSize\r
+    );\r
 \r
   Status = TcgDxeHashLogExtendEventI (\r
              &mTcgDxeData,\r
-             (UINT8*)VarLog,\r
+             (UINT8 *)VarLog,\r
              TcgEvent.EventSize,\r
              &TcgEvent,\r
-             (UINT8*)VarLog\r
+             (UINT8 *)VarLog\r
              );\r
   FreePool (VarLog);\r
   return Status;\r
@@ -1018,13 +1022,13 @@ MeasureVariable (
 EFI_STATUS\r
 EFIAPI\r
 ReadAndMeasureBootVariable (\r
-  IN      CHAR16                    *VarName,\r
-  IN      EFI_GUID                  *VendorGuid,\r
-  OUT     UINTN                     *VarSize,\r
-  OUT     VOID                      **VarData\r
+  IN      CHAR16    *VarName,\r
+  IN      EFI_GUID  *VendorGuid,\r
+  OUT     UINTN     *VarSize,\r
+  OUT     VOID      **VarData\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
+  EFI_STATUS  Status;\r
 \r
   *VarData = ReadVariable (VarName, VendorGuid, VarSize);\r
   if (*VarData == NULL) {\r
@@ -1058,20 +1062,20 @@ MeasureAllBootVariables (
   VOID\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  UINT16                            *BootOrder;\r
-  UINTN                             BootCount;\r
-  UINTN                             Index;\r
-  VOID                              *BootVarData;\r
-  UINTN                             Size;\r
+  EFI_STATUS  Status;\r
+  UINT16      *BootOrder;\r
+  UINTN       BootCount;\r
+  UINTN       Index;\r
+  VOID        *BootVarData;\r
+  UINTN       Size;\r
 \r
   Status = ReadAndMeasureBootVariable (\r
              mBootVarName,\r
              &gEfiGlobalVariableGuid,\r
              &BootCount,\r
-             (VOID **) &BootOrder\r
+             (VOID **)&BootOrder\r
              );\r
-  if (Status == EFI_NOT_FOUND || BootOrder == NULL) {\r
+  if ((Status == EFI_NOT_FOUND) || (BootOrder == NULL)) {\r
     return EFI_SUCCESS;\r
   }\r
 \r
@@ -1113,15 +1117,14 @@ MeasureAllBootVariables (
 VOID\r
 EFIAPI\r
 OnReadyToBoot (\r
-  IN      EFI_EVENT                 Event,\r
-  IN      VOID                      *Context\r
+  IN      EFI_EVENT  Event,\r
+  IN      VOID       *Context\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  TPM_PCRINDEX                      PcrIndex;\r
+  EFI_STATUS    Status;\r
+  TPM_PCRINDEX  PcrIndex;\r
 \r
   if (mBootAttempts == 0) {\r
-\r
     //\r
     // Measure handoff tables.\r
     //\r
@@ -1201,15 +1204,15 @@ OnReadyToBoot (
 VOID\r
 EFIAPI\r
 InstallAcpiTable (\r
-  IN EFI_EVENT                      Event,\r
-  IN VOID*                          Context\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
   )\r
 {\r
-  UINTN                             TableKey;\r
-  EFI_STATUS                        Status;\r
-  EFI_ACPI_TABLE_PROTOCOL           *AcpiTable;\r
-  UINT8                             Checksum;\r
-  UINT64                            OemTableId;\r
+  UINTN                    TableKey;\r
+  EFI_STATUS               Status;\r
+  EFI_ACPI_TABLE_PROTOCOL  *AcpiTable;\r
+  UINT8                    Checksum;\r
+  UINT64                   OemTableId;\r
 \r
   Status = gBS->LocateProtocol (&gEfiAcpiTableProtocolGuid, NULL, (VOID **)&AcpiTable);\r
   if (EFI_ERROR (Status)) {\r
@@ -1220,47 +1223,47 @@ InstallAcpiTable (
     CopyMem (mTcgClientAcpiTemplate.Header.OemId, PcdGetPtr (PcdAcpiDefaultOemId), sizeof (mTcgClientAcpiTemplate.Header.OemId));\r
     OemTableId = PcdGet64 (PcdAcpiDefaultOemTableId);\r
     CopyMem (&mTcgClientAcpiTemplate.Header.OemTableId, &OemTableId, sizeof (UINT64));\r
-    mTcgClientAcpiTemplate.Header.OemRevision      = PcdGet32 (PcdAcpiDefaultOemRevision);\r
-    mTcgClientAcpiTemplate.Header.CreatorId        = PcdGet32 (PcdAcpiDefaultCreatorId);\r
-    mTcgClientAcpiTemplate.Header.CreatorRevision  = PcdGet32 (PcdAcpiDefaultCreatorRevision);\r
+    mTcgClientAcpiTemplate.Header.OemRevision     = PcdGet32 (PcdAcpiDefaultOemRevision);\r
+    mTcgClientAcpiTemplate.Header.CreatorId       = PcdGet32 (PcdAcpiDefaultCreatorId);\r
+    mTcgClientAcpiTemplate.Header.CreatorRevision = PcdGet32 (PcdAcpiDefaultCreatorRevision);\r
     //\r
     // The ACPI table must be checksummed before calling the InstallAcpiTable()\r
     // service of the ACPI table protocol to install it.\r
     //\r
-    Checksum = CalculateCheckSum8 ((UINT8 *)&mTcgClientAcpiTemplate, sizeof (mTcgClientAcpiTemplate));\r
+    Checksum                               = CalculateCheckSum8 ((UINT8 *)&mTcgClientAcpiTemplate, sizeof (mTcgClientAcpiTemplate));\r
     mTcgClientAcpiTemplate.Header.Checksum = Checksum;\r
 \r
     Status = AcpiTable->InstallAcpiTable (\r
-                            AcpiTable,\r
-                            &mTcgClientAcpiTemplate,\r
-                            sizeof (mTcgClientAcpiTemplate),\r
-                            &TableKey\r
-                            );\r
+                          AcpiTable,\r
+                          &mTcgClientAcpiTemplate,\r
+                          sizeof (mTcgClientAcpiTemplate),\r
+                          &TableKey\r
+                          );\r
   } else {\r
     CopyMem (mTcgServerAcpiTemplate.Header.OemId, PcdGetPtr (PcdAcpiDefaultOemId), sizeof (mTcgServerAcpiTemplate.Header.OemId));\r
     OemTableId = PcdGet64 (PcdAcpiDefaultOemTableId);\r
     CopyMem (&mTcgServerAcpiTemplate.Header.OemTableId, &OemTableId, sizeof (UINT64));\r
-    mTcgServerAcpiTemplate.Header.OemRevision      = PcdGet32 (PcdAcpiDefaultOemRevision);\r
-    mTcgServerAcpiTemplate.Header.CreatorId        = PcdGet32 (PcdAcpiDefaultCreatorId);\r
-    mTcgServerAcpiTemplate.Header.CreatorRevision  = PcdGet32 (PcdAcpiDefaultCreatorRevision);\r
+    mTcgServerAcpiTemplate.Header.OemRevision     = PcdGet32 (PcdAcpiDefaultOemRevision);\r
+    mTcgServerAcpiTemplate.Header.CreatorId       = PcdGet32 (PcdAcpiDefaultCreatorId);\r
+    mTcgServerAcpiTemplate.Header.CreatorRevision = PcdGet32 (PcdAcpiDefaultCreatorRevision);\r
     //\r
     // The ACPI table must be checksummed before calling the InstallAcpiTable()\r
     // service of the ACPI table protocol to install it.\r
     //\r
-    Checksum = CalculateCheckSum8 ((UINT8 *)&mTcgServerAcpiTemplate, sizeof (mTcgServerAcpiTemplate));\r
+    Checksum                               = CalculateCheckSum8 ((UINT8 *)&mTcgServerAcpiTemplate, sizeof (mTcgServerAcpiTemplate));\r
     mTcgServerAcpiTemplate.Header.Checksum = Checksum;\r
 \r
     mTcgServerAcpiTemplate.BaseAddress.Address = PcdGet64 (PcdTpmBaseAddress);\r
-    Status = AcpiTable->InstallAcpiTable (\r
-                            AcpiTable,\r
-                            &mTcgServerAcpiTemplate,\r
-                            sizeof (mTcgServerAcpiTemplate),\r
-                            &TableKey\r
-                            );\r
+    Status                                     = AcpiTable->InstallAcpiTable (\r
+                                                              AcpiTable,\r
+                                                              &mTcgServerAcpiTemplate,\r
+                                                              sizeof (mTcgServerAcpiTemplate),\r
+                                                              &TableKey\r
+                                                              );\r
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG((DEBUG_ERROR, "Tcg Acpi Table installation failure"));\r
+    DEBUG ((DEBUG_ERROR, "Tcg Acpi Table installation failure"));\r
   }\r
 }\r
 \r
@@ -1276,11 +1279,11 @@ InstallAcpiTable (
 VOID\r
 EFIAPI\r
 OnExitBootServices (\r
-  IN      EFI_EVENT                 Event,\r
-  IN      VOID                      *Context\r
+  IN      EFI_EVENT  Event,\r
+  IN      VOID       *Context\r
   )\r
 {\r
-  EFI_STATUS    Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // Measure invocation of ExitBootServices,\r
@@ -1298,7 +1301,7 @@ OnExitBootServices (
   Status = TcgMeasureAction (\r
              EFI_EXIT_BOOT_SERVICES_SUCCEEDED\r
              );\r
-  if (EFI_ERROR (Status)){\r
+  if (EFI_ERROR (Status)) {\r
     DEBUG ((DEBUG_ERROR, "%a not Measured. Error!\n", EFI_EXIT_BOOT_SERVICES_SUCCEEDED));\r
   }\r
 }\r
@@ -1315,11 +1318,11 @@ OnExitBootServices (
 VOID\r
 EFIAPI\r
 OnExitBootServicesFailed (\r
-  IN      EFI_EVENT                 Event,\r
-  IN      VOID                      *Context\r
+  IN      EFI_EVENT  Event,\r
+  IN      VOID       *Context\r
   )\r
 {\r
-  EFI_STATUS    Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // Measure Failure of ExitBootServices,\r
@@ -1327,7 +1330,7 @@ OnExitBootServicesFailed (
   Status = TcgMeasureAction (\r
              EFI_EXIT_BOOT_SERVICES_FAILED\r
              );\r
-  if (EFI_ERROR (Status)){\r
+  if (EFI_ERROR (Status)) {\r
     DEBUG ((DEBUG_ERROR, "%a not Measured. Error!\n", EFI_EXIT_BOOT_SERVICES_FAILED));\r
   }\r
 }\r
@@ -1372,15 +1375,15 @@ GetTpmStatus (
 EFI_STATUS\r
 EFIAPI\r
 DriverEntry (\r
-  IN    EFI_HANDLE                  ImageHandle,\r
-  IN    EFI_SYSTEM_TABLE            *SystemTable\r
+  IN    EFI_HANDLE        ImageHandle,\r
+  IN    EFI_SYSTEM_TABLE  *SystemTable\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  EFI_EVENT                         Event;\r
-  VOID                              *Registration;\r
+  EFI_STATUS  Status;\r
+  EFI_EVENT   Event;\r
+  VOID        *Registration;\r
 \r
-  if (!CompareGuid (PcdGetPtr(PcdTpmInstanceGuid), &gEfiTpmDeviceInstanceTpm12Guid)){\r
+  if (!CompareGuid (PcdGetPtr (PcdTpmInstanceGuid), &gEfiTpmDeviceInstanceTpm12Guid)) {\r
     DEBUG ((DEBUG_ERROR, "No TPM12 instance required!\n"));\r
     return EFI_UNSUPPORTED;\r
   }\r