]> git.proxmox.com Git - mirror_edk2.git/blobdiff - SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c
SecurityPkg: Apply uncrustify changes
[mirror_edk2.git] / SecurityPkg / Library / Tpm2DeviceLibDTpm / Tpm2Ptp.c
index 5ceb6c8466d59c24db64e2b61fbd0d7a7c609c71..40ab998004e255bcec27bf701d8c6e394d263f72 100644 (file)
@@ -26,12 +26,12 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 // Execution of the command may take from several seconds to minutes for certain\r
 // commands, such as key generation.\r
 //\r
-#define PTP_TIMEOUT_MAX             (90000 * 1000)  // 90s\r
+#define PTP_TIMEOUT_MAX  (90000 * 1000)             // 90s\r
 \r
 //\r
 // Max TPM command/response length\r
 //\r
-#define TPMCMDBUFLENGTH             0x500\r
+#define TPMCMDBUFLENGTH  0x500\r
 \r
 /**\r
   Check whether TPM PTP register exist.\r
@@ -43,10 +43,10 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 **/\r
 BOOLEAN\r
 Tpm2IsPtpPresence (\r
-  IN VOID *Reg\r
+  IN VOID  *Reg\r
   )\r
 {\r
-  UINT8                             RegRead;\r
+  UINT8  RegRead;\r
 \r
   RegRead = MmioRead8 ((UINTN)Reg);\r
   if (RegRead == 0xFF) {\r
@@ -55,6 +55,7 @@ Tpm2IsPtpPresence (
     //\r
     return FALSE;\r
   }\r
+\r
   return TRUE;\r
 }\r
 \r
@@ -71,22 +72,24 @@ Tpm2IsPtpPresence (
 **/\r
 EFI_STATUS\r
 PtpCrbWaitRegisterBits (\r
-  IN      UINT32                    *Register,\r
-  IN      UINT32                    BitSet,\r
-  IN      UINT32                    BitClear,\r
-  IN      UINT32                    TimeOut\r
+  IN      UINT32  *Register,\r
+  IN      UINT32  BitSet,\r
+  IN      UINT32  BitClear,\r
+  IN      UINT32  TimeOut\r
   )\r
 {\r
-  UINT32                            RegRead;\r
-  UINT32                            WaitTime;\r
+  UINT32  RegRead;\r
+  UINT32  WaitTime;\r
 \r
-  for (WaitTime = 0; WaitTime < TimeOut; WaitTime += 30){\r
+  for (WaitTime = 0; WaitTime < TimeOut; WaitTime += 30) {\r
     RegRead = MmioRead32 ((UINTN)Register);\r
-    if ((RegRead & BitSet) == BitSet && (RegRead & BitClear) == 0) {\r
+    if (((RegRead & BitSet) == BitSet) && ((RegRead & BitClear) == 0)) {\r
       return EFI_SUCCESS;\r
     }\r
+\r
     MicroSecondDelay (30);\r
   }\r
+\r
   return EFI_TIMEOUT;\r
 }\r
 \r
@@ -102,16 +105,16 @@ PtpCrbWaitRegisterBits (
 **/\r
 EFI_STATUS\r
 PtpCrbRequestUseTpm (\r
-  IN      PTP_CRB_REGISTERS_PTR      CrbReg\r
+  IN      PTP_CRB_REGISTERS_PTR  CrbReg\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
+  EFI_STATUS  Status;\r
 \r
   if (!Tpm2IsPtpPresence (CrbReg)) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  MmioWrite32((UINTN)&CrbReg->LocalityControl, PTP_CRB_LOCALITY_CONTROL_REQUEST_ACCESS);\r
+  MmioWrite32 ((UINTN)&CrbReg->LocalityControl, PTP_CRB_LOCALITY_CONTROL_REQUEST_ACCESS);\r
   Status = PtpCrbWaitRegisterBits (\r
              &CrbReg->LocalityStatus,\r
              PTP_CRB_LOCALITY_STATUS_GRANTED,\r
@@ -138,52 +141,55 @@ PtpCrbRequestUseTpm (
 **/\r
 EFI_STATUS\r
 PtpCrbTpmCommand (\r
-  IN     PTP_CRB_REGISTERS_PTR      CrbReg,\r
-  IN     UINT8                      *BufferIn,\r
-  IN     UINT32                     SizeIn,\r
-  IN OUT UINT8                      *BufferOut,\r
-  IN OUT UINT32                     *SizeOut\r
+  IN     PTP_CRB_REGISTERS_PTR  CrbReg,\r
+  IN     UINT8                  *BufferIn,\r
+  IN     UINT32                 SizeIn,\r
+  IN OUT UINT8                  *BufferOut,\r
+  IN OUT UINT32                 *SizeOut\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  UINT32                            Index;\r
-  UINT32                            TpmOutSize;\r
-  UINT16                            Data16;\r
-  UINT32                            Data32;\r
+  EFI_STATUS  Status;\r
+  UINT32      Index;\r
+  UINT32      TpmOutSize;\r
+  UINT16      Data16;\r
+  UINT32      Data32;\r
 \r
   DEBUG_CODE_BEGIN ();\r
-    UINTN  DebugSize;\r
+  UINTN  DebugSize;\r
 \r
-    DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand Send - "));\r
-    if (SizeIn > 0x100) {\r
-      DebugSize = 0x40;\r
-    } else {\r
-      DebugSize = SizeIn;\r
-    }\r
-    for (Index = 0; Index < DebugSize; Index++) {\r
+  DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand Send - "));\r
+  if (SizeIn > 0x100) {\r
+    DebugSize = 0x40;\r
+  } else {\r
+    DebugSize = SizeIn;\r
+  }\r
+\r
+  for (Index = 0; Index < DebugSize; Index++) {\r
+    DEBUG ((DEBUG_VERBOSE, "%02x ", BufferIn[Index]));\r
+  }\r
+\r
+  if (DebugSize != SizeIn) {\r
+    DEBUG ((DEBUG_VERBOSE, "...... "));\r
+    for (Index = SizeIn - 0x20; Index < SizeIn; Index++) {\r
       DEBUG ((DEBUG_VERBOSE, "%02x ", BufferIn[Index]));\r
     }\r
-    if (DebugSize != SizeIn) {\r
-      DEBUG ((DEBUG_VERBOSE, "...... "));\r
-      for (Index = SizeIn - 0x20; Index < SizeIn; Index++) {\r
-        DEBUG ((DEBUG_VERBOSE, "%02x ", BufferIn[Index]));\r
-      }\r
-    }\r
-    DEBUG ((DEBUG_VERBOSE, "\n"));\r
+  }\r
+\r
+  DEBUG ((DEBUG_VERBOSE, "\n"));\r
   DEBUG_CODE_END ();\r
-  TpmOutSize         = 0;\r
+  TpmOutSize = 0;\r
 \r
   //\r
   // STEP 0:\r
   // if CapCRbIdelByPass == 0, enforce Idle state before sending command\r
   //\r
-  if (GetCachedIdleByPass () == 0 && (MmioRead32((UINTN)&CrbReg->CrbControlStatus) & PTP_CRB_CONTROL_AREA_STATUS_TPM_IDLE) == 0){\r
+  if ((GetCachedIdleByPass () == 0) && ((MmioRead32 ((UINTN)&CrbReg->CrbControlStatus) & PTP_CRB_CONTROL_AREA_STATUS_TPM_IDLE) == 0)) {\r
     Status = PtpCrbWaitRegisterBits (\r
-              &CrbReg->CrbControlStatus,\r
-              PTP_CRB_CONTROL_AREA_STATUS_TPM_IDLE,\r
-              0,\r
-              PTP_TIMEOUT_C\r
-              );\r
+               &CrbReg->CrbControlStatus,\r
+               PTP_CRB_CONTROL_AREA_STATUS_TPM_IDLE,\r
+               0,\r
+               PTP_TIMEOUT_C\r
+               );\r
     if (EFI_ERROR (Status)) {\r
       //\r
       // Try to goIdle to recover TPM\r
@@ -199,7 +205,7 @@ PtpCrbTpmCommand (
   // of 1 by software to Request.cmdReady, as indicated by the Status field\r
   // being cleared to 0.\r
   //\r
-  MmioWrite32((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_COMMAND_READY);\r
+  MmioWrite32 ((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_COMMAND_READY);\r
   Status = PtpCrbWaitRegisterBits (\r
              &CrbReg->CrbControlRequest,\r
              0,\r
@@ -210,6 +216,7 @@ PtpCrbTpmCommand (
     Status = EFI_DEVICE_ERROR;\r
     goto GoIdle_Exit;\r
   }\r
+\r
   Status = PtpCrbWaitRegisterBits (\r
              &CrbReg->CrbControlStatus,\r
              0,\r
@@ -230,19 +237,20 @@ PtpCrbTpmCommand (
   for (Index = 0; Index < SizeIn; Index++) {\r
     MmioWrite8 ((UINTN)&CrbReg->CrbDataBuffer[Index], BufferIn[Index]);\r
   }\r
+\r
   MmioWrite32 ((UINTN)&CrbReg->CrbControlCommandAddressHigh, (UINT32)RShiftU64 ((UINTN)CrbReg->CrbDataBuffer, 32));\r
   MmioWrite32 ((UINTN)&CrbReg->CrbControlCommandAddressLow, (UINT32)(UINTN)CrbReg->CrbDataBuffer);\r
-  MmioWrite32 ((UINTN)&CrbReg->CrbControlCommandSize, sizeof(CrbReg->CrbDataBuffer));\r
+  MmioWrite32 ((UINTN)&CrbReg->CrbControlCommandSize, sizeof (CrbReg->CrbDataBuffer));\r
 \r
   MmioWrite64 ((UINTN)&CrbReg->CrbControlResponseAddrss, (UINT32)(UINTN)CrbReg->CrbDataBuffer);\r
-  MmioWrite32 ((UINTN)&CrbReg->CrbControlResponseSize, sizeof(CrbReg->CrbDataBuffer));\r
+  MmioWrite32 ((UINTN)&CrbReg->CrbControlResponseSize, sizeof (CrbReg->CrbDataBuffer));\r
 \r
   //\r
   // STEP 3:\r
   // Command Execution occurs after receipt of a 1 to Start and the TPM\r
   // clearing Start to 0.\r
   //\r
-  MmioWrite32((UINTN)&CrbReg->CrbControlStart, PTP_CRB_CONTROL_START);\r
+  MmioWrite32 ((UINTN)&CrbReg->CrbControlStart, PTP_CRB_CONTROL_START);\r
   Status = PtpCrbWaitRegisterBits (\r
              &CrbReg->CrbControlStart,\r
              0,\r
@@ -254,16 +262,16 @@ PtpCrbTpmCommand (
     // Command Completion check timeout. Cancel the currently executing command by writing TPM_CRB_CTRL_CANCEL,\r
     // Expect TPM_RC_CANCELLED or successfully completed response.\r
     //\r
-    MmioWrite32((UINTN)&CrbReg->CrbControlCancel, PTP_CRB_CONTROL_CANCEL);\r
+    MmioWrite32 ((UINTN)&CrbReg->CrbControlCancel, PTP_CRB_CONTROL_CANCEL);\r
     Status = PtpCrbWaitRegisterBits (\r
                &CrbReg->CrbControlStart,\r
                0,\r
                PTP_CRB_CONTROL_START,\r
                PTP_TIMEOUT_B\r
                );\r
-    MmioWrite32((UINTN)&CrbReg->CrbControlCancel, 0);\r
+    MmioWrite32 ((UINTN)&CrbReg->CrbControlCancel, 0);\r
 \r
-    if (EFI_ERROR(Status)) {\r
+    if (EFI_ERROR (Status)) {\r
       //\r
       // Still in Command Execution state. Try to goIdle, the behavior is agnostic.\r
       //\r
@@ -285,12 +293,14 @@ PtpCrbTpmCommand (
   for (Index = 0; Index < sizeof (TPM2_RESPONSE_HEADER); Index++) {\r
     BufferOut[Index] = MmioRead8 ((UINTN)&CrbReg->CrbDataBuffer[Index]);\r
   }\r
+\r
   DEBUG_CODE_BEGIN ();\r
-    DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand ReceiveHeader - "));\r
-    for (Index = 0; Index < sizeof (TPM2_RESPONSE_HEADER); Index++) {\r
-      DEBUG ((DEBUG_VERBOSE, "%02x ", BufferOut[Index]));\r
-    }\r
-    DEBUG ((DEBUG_VERBOSE, "\n"));\r
+  DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand ReceiveHeader - "));\r
+  for (Index = 0; Index < sizeof (TPM2_RESPONSE_HEADER); Index++) {\r
+    DEBUG ((DEBUG_VERBOSE, "%02x ", BufferOut[Index]));\r
+  }\r
+\r
+  DEBUG ((DEBUG_VERBOSE, "\n"));\r
   DEBUG_CODE_END ();\r
   //\r
   // Check the response data header (tag, parasize and returncode)\r
@@ -304,7 +314,7 @@ PtpCrbTpmCommand (
   }\r
 \r
   CopyMem (&Data32, (BufferOut + 2), sizeof (UINT32));\r
-  TpmOutSize  = SwapBytes32 (Data32);\r
+  TpmOutSize = SwapBytes32 (Data32);\r
   if (*SizeOut < TpmOutSize) {\r
     //\r
     // Command completed, but buffer is not enough\r
@@ -312,6 +322,7 @@ PtpCrbTpmCommand (
     Status = EFI_BUFFER_TOO_SMALL;\r
     goto GoReady_Exit;\r
   }\r
+\r
   *SizeOut = TpmOutSize;\r
   //\r
   // Continue reading the remaining data\r
@@ -321,11 +332,12 @@ PtpCrbTpmCommand (
   }\r
 \r
   DEBUG_CODE_BEGIN ();\r
-    DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand Receive - "));\r
-    for (Index = 0; Index < TpmOutSize; Index++) {\r
-      DEBUG ((DEBUG_VERBOSE, "%02x ", BufferOut[Index]));\r
-    }\r
-    DEBUG ((DEBUG_VERBOSE, "\n"));\r
+  DEBUG ((DEBUG_VERBOSE, "PtpCrbTpmCommand Receive - "));\r
+  for (Index = 0; Index < TpmOutSize; Index++) {\r
+    DEBUG ((DEBUG_VERBOSE, "%02x ", BufferOut[Index]));\r
+  }\r
+\r
+  DEBUG ((DEBUG_VERBOSE, "\n"));\r
   DEBUG_CODE_END ();\r
 \r
 GoReady_Exit:\r
@@ -334,7 +346,7 @@ GoReady_Exit:
   // If not supported. flow down to GoIdle\r
   //\r
   if (GetCachedIdleByPass () == 1) {\r
-    MmioWrite32((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_COMMAND_READY);\r
+    MmioWrite32 ((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_COMMAND_READY);\r
     return Status;\r
   }\r
 \r
@@ -347,13 +359,13 @@ GoIdle_Exit:
   //\r
   //  Return to Idle state by setting TPM_CRB_CTRL_STS_x.Status.goIdle to 1.\r
   //\r
-  MmioWrite32((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_GO_IDLE);\r
+  MmioWrite32 ((UINTN)&CrbReg->CrbControlRequest, PTP_CRB_CONTROL_AREA_REQUEST_GO_IDLE);\r
 \r
   //\r
   // Only enforce Idle state transition if execution fails when CRBIdleBypass==1\r
   // Leave regular Idle delay at the beginning of next command execution\r
   //\r
-  if (GetCachedIdleByPass () == 1){\r
+  if (GetCachedIdleByPass () == 1) {\r
     Status = PtpCrbWaitRegisterBits (\r
                &CrbReg->CrbControlStatus,\r
                PTP_CRB_CONTROL_AREA_STATUS_TPM_IDLE,\r
@@ -382,11 +394,11 @@ GoIdle_Exit:
 **/\r
 EFI_STATUS\r
 Tpm2TisTpmCommand (\r
-  IN     TIS_PC_REGISTERS_PTR       TisReg,\r
-  IN     UINT8                      *BufferIn,\r
-  IN     UINT32                     SizeIn,\r
-  IN OUT UINT8                      *BufferOut,\r
-  IN OUT UINT32                     *SizeOut\r
+  IN     TIS_PC_REGISTERS_PTR  TisReg,\r
+  IN     UINT8                 *BufferIn,\r
+  IN     UINT32                SizeIn,\r
+  IN OUT UINT8                 *BufferOut,\r
+  IN OUT UINT32                *SizeOut\r
   );\r
 \r
 /**\r
@@ -402,7 +414,7 @@ Tpm2TisTpmCommand (
 **/\r
 EFI_STATUS\r
 TisPcRequestUseTpm (\r
-  IN     TIS_PC_REGISTERS_PTR       TisReg\r
+  IN     TIS_PC_REGISTERS_PTR  TisReg\r
   );\r
 \r
 /**\r
@@ -414,32 +426,37 @@ TisPcRequestUseTpm (
 **/\r
 TPM2_PTP_INTERFACE_TYPE\r
 Tpm2GetPtpInterface (\r
-  IN VOID *Register\r
+  IN VOID  *Register\r
   )\r
 {\r
-  PTP_CRB_INTERFACE_IDENTIFIER  InterfaceId;\r
-  PTP_FIFO_INTERFACE_CAPABILITY InterfaceCapability;\r
+  PTP_CRB_INTERFACE_IDENTIFIER   InterfaceId;\r
+  PTP_FIFO_INTERFACE_CAPABILITY  InterfaceCapability;\r
 \r
   if (!Tpm2IsPtpPresence (Register)) {\r
     return Tpm2PtpInterfaceMax;\r
   }\r
+\r
   //\r
   // Check interface id\r
   //\r
-  InterfaceId.Uint32 = MmioRead32 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->InterfaceId);\r
+  InterfaceId.Uint32         = MmioRead32 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->InterfaceId);\r
   InterfaceCapability.Uint32 = MmioRead32 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->InterfaceCapability);\r
 \r
   if ((InterfaceId.Bits.InterfaceType == PTP_INTERFACE_IDENTIFIER_INTERFACE_TYPE_CRB) &&\r
       (InterfaceId.Bits.InterfaceVersion == PTP_INTERFACE_IDENTIFIER_INTERFACE_VERSION_CRB) &&\r
-      (InterfaceId.Bits.CapCRB != 0)) {\r
+      (InterfaceId.Bits.CapCRB != 0))\r
+  {\r
     return Tpm2PtpInterfaceCrb;\r
   }\r
+\r
   if ((InterfaceId.Bits.InterfaceType == PTP_INTERFACE_IDENTIFIER_INTERFACE_TYPE_FIFO) &&\r
       (InterfaceId.Bits.InterfaceVersion == PTP_INTERFACE_IDENTIFIER_INTERFACE_VERSION_FIFO) &&\r
       (InterfaceId.Bits.CapFIFO != 0) &&\r
-      (InterfaceCapability.Bits.InterfaceVersion == INTERFACE_CAPABILITY_INTERFACE_VERSION_PTP)) {\r
+      (InterfaceCapability.Bits.InterfaceVersion == INTERFACE_CAPABILITY_INTERFACE_VERSION_PTP))\r
+  {\r
     return Tpm2PtpInterfaceFifo;\r
   }\r
+\r
   return Tpm2PtpInterfaceTis;\r
 }\r
 \r
@@ -452,7 +469,7 @@ Tpm2GetPtpInterface (
 **/\r
 UINT8\r
 Tpm2GetIdleByPass (\r
-  IN VOID *Register\r
+  IN VOID  *Register\r
   )\r
 {\r
   PTP_CRB_INTERFACE_IDENTIFIER  InterfaceId;\r
@@ -472,24 +489,24 @@ Tpm2GetIdleByPass (
 **/\r
 VOID\r
 DumpPtpInfo (\r
-  IN VOID *Register\r
+  IN VOID  *Register\r
   )\r
 {\r
-  PTP_CRB_INTERFACE_IDENTIFIER  InterfaceId;\r
-  PTP_FIFO_INTERFACE_CAPABILITY InterfaceCapability;\r
-  UINT8                         StatusEx;\r
-  UINT16                        Vid;\r
-  UINT16                        Did;\r
-  UINT8                         Rid;\r
-  TPM2_PTP_INTERFACE_TYPE       PtpInterface;\r
+  PTP_CRB_INTERFACE_IDENTIFIER   InterfaceId;\r
+  PTP_FIFO_INTERFACE_CAPABILITY  InterfaceCapability;\r
+  UINT8                          StatusEx;\r
+  UINT16                         Vid;\r
+  UINT16                         Did;\r
+  UINT8                          Rid;\r
+  TPM2_PTP_INTERFACE_TYPE        PtpInterface;\r
 \r
   if (!Tpm2IsPtpPresence (Register)) {\r
-    return ;\r
+    return;\r
   }\r
 \r
-  InterfaceId.Uint32 = MmioRead32 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->InterfaceId);\r
+  InterfaceId.Uint32         = MmioRead32 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->InterfaceId);\r
   InterfaceCapability.Uint32 = MmioRead32 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->InterfaceCapability);\r
-  StatusEx = MmioRead8 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->StatusEx);\r
+  StatusEx                   = MmioRead8 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->StatusEx);\r
 \r
   //\r
   // Dump InterfaceId Register for PTP\r
@@ -507,7 +524,8 @@ DumpPtpInfo (
   //\r
   DEBUG ((DEBUG_INFO, "InterfaceCapability - 0x%08x\n", InterfaceCapability.Uint32));\r
   if ((InterfaceId.Bits.InterfaceType == PTP_INTERFACE_IDENTIFIER_INTERFACE_TYPE_TIS) ||\r
-      (InterfaceId.Bits.InterfaceType == PTP_INTERFACE_IDENTIFIER_INTERFACE_TYPE_FIFO)) {\r
+      (InterfaceId.Bits.InterfaceType == PTP_INTERFACE_IDENTIFIER_INTERFACE_TYPE_FIFO))\r
+  {\r
     DEBUG ((DEBUG_INFO, "  InterfaceVersion - 0x%x\n", InterfaceCapability.Bits.InterfaceVersion));\r
   }\r
 \r
@@ -519,26 +537,27 @@ DumpPtpInfo (
     DEBUG ((DEBUG_INFO, "  TpmFamily - 0x%x\n", (StatusEx & PTP_FIFO_STS_EX_TPM_FAMILY) >> PTP_FIFO_STS_EX_TPM_FAMILY_OFFSET));\r
   }\r
 \r
-  Vid = 0xFFFF;\r
-  Did = 0xFFFF;\r
-  Rid = 0xFF;\r
+  Vid          = 0xFFFF;\r
+  Did          = 0xFFFF;\r
+  Rid          = 0xFF;\r
   PtpInterface = GetCachedPtpInterface ();\r
   DEBUG ((DEBUG_INFO, "PtpInterface - %x\n", PtpInterface));\r
   switch (PtpInterface) {\r
-  case Tpm2PtpInterfaceCrb:\r
-    Vid = MmioRead16 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->Vid);\r
-    Did = MmioRead16 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->Did);\r
-    Rid = (UINT8)InterfaceId.Bits.Rid;\r
-    break;\r
-  case Tpm2PtpInterfaceFifo:\r
-  case Tpm2PtpInterfaceTis:\r
-    Vid = MmioRead16 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Vid);\r
-    Did = MmioRead16 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Did);\r
-    Rid = MmioRead8 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Rid);\r
-    break;\r
-  default:\r
-    break;\r
+    case Tpm2PtpInterfaceCrb:\r
+      Vid = MmioRead16 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->Vid);\r
+      Did = MmioRead16 ((UINTN)&((PTP_CRB_REGISTERS *)Register)->Did);\r
+      Rid = (UINT8)InterfaceId.Bits.Rid;\r
+      break;\r
+    case Tpm2PtpInterfaceFifo:\r
+    case Tpm2PtpInterfaceTis:\r
+      Vid = MmioRead16 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Vid);\r
+      Did = MmioRead16 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Did);\r
+      Rid = MmioRead8 ((UINTN)&((PTP_FIFO_REGISTERS *)Register)->Rid);\r
+      break;\r
+    default:\r
+      break;\r
   }\r
+\r
   DEBUG ((DEBUG_INFO, "VID - 0x%04x\n", Vid));\r
   DEBUG ((DEBUG_INFO, "DID - 0x%04x\n", Did));\r
   DEBUG ((DEBUG_INFO, "RID - 0x%02x\n", Rid));\r
@@ -559,35 +578,35 @@ DumpPtpInfo (
 EFI_STATUS\r
 EFIAPI\r
 DTpm2SubmitCommand (\r
-  IN UINT32            InputParameterBlockSize,\r
-  IN UINT8             *InputParameterBlock,\r
-  IN OUT UINT32        *OutputParameterBlockSize,\r
-  IN UINT8             *OutputParameterBlock\r
+  IN UINT32      InputParameterBlockSize,\r
+  IN UINT8       *InputParameterBlock,\r
+  IN OUT UINT32  *OutputParameterBlockSize,\r
+  IN UINT8       *OutputParameterBlock\r
   )\r
 {\r
   TPM2_PTP_INTERFACE_TYPE  PtpInterface;\r
 \r
   PtpInterface = GetCachedPtpInterface ();\r
   switch (PtpInterface) {\r
-  case Tpm2PtpInterfaceCrb:\r
-    return PtpCrbTpmCommand (\r
-           (PTP_CRB_REGISTERS_PTR) (UINTN) PcdGet64 (PcdTpmBaseAddress),\r
-           InputParameterBlock,\r
-           InputParameterBlockSize,\r
-           OutputParameterBlock,\r
-           OutputParameterBlockSize\r
-           );\r
-  case Tpm2PtpInterfaceFifo:\r
-  case Tpm2PtpInterfaceTis:\r
-    return Tpm2TisTpmCommand (\r
-           (TIS_PC_REGISTERS_PTR) (UINTN) PcdGet64 (PcdTpmBaseAddress),\r
-           InputParameterBlock,\r
-           InputParameterBlockSize,\r
-           OutputParameterBlock,\r
-           OutputParameterBlockSize\r
-           );\r
-  default:\r
-    return EFI_NOT_FOUND;\r
+    case Tpm2PtpInterfaceCrb:\r
+      return PtpCrbTpmCommand (\r
+               (PTP_CRB_REGISTERS_PTR)(UINTN)PcdGet64 (PcdTpmBaseAddress),\r
+               InputParameterBlock,\r
+               InputParameterBlockSize,\r
+               OutputParameterBlock,\r
+               OutputParameterBlockSize\r
+               );\r
+    case Tpm2PtpInterfaceFifo:\r
+    case Tpm2PtpInterfaceTis:\r
+      return Tpm2TisTpmCommand (\r
+               (TIS_PC_REGISTERS_PTR)(UINTN)PcdGet64 (PcdTpmBaseAddress),\r
+               InputParameterBlock,\r
+               InputParameterBlockSize,\r
+               OutputParameterBlock,\r
+               OutputParameterBlockSize\r
+               );\r
+    default:\r
+      return EFI_NOT_FOUND;\r
   }\r
 }\r
 \r
@@ -608,12 +627,12 @@ DTpm2RequestUseTpm (
 \r
   PtpInterface = GetCachedPtpInterface ();\r
   switch (PtpInterface) {\r
-  case Tpm2PtpInterfaceCrb:\r
-    return PtpCrbRequestUseTpm ((PTP_CRB_REGISTERS_PTR) (UINTN) PcdGet64 (PcdTpmBaseAddress));\r
-  case Tpm2PtpInterfaceFifo:\r
-  case Tpm2PtpInterfaceTis:\r
-    return TisPcRequestUseTpm ((TIS_PC_REGISTERS_PTR) (UINTN) PcdGet64 (PcdTpmBaseAddress));\r
-  default:\r
-    return EFI_NOT_FOUND;\r
+    case Tpm2PtpInterfaceCrb:\r
+      return PtpCrbRequestUseTpm ((PTP_CRB_REGISTERS_PTR)(UINTN)PcdGet64 (PcdTpmBaseAddress));\r
+    case Tpm2PtpInterfaceFifo:\r
+    case Tpm2PtpInterfaceTis:\r
+      return TisPcRequestUseTpm ((TIS_PC_REGISTERS_PTR)(UINTN)PcdGet64 (PcdTpmBaseAddress));\r
+    default:\r
+      return EFI_NOT_FOUND;\r
   }\r
 }\r