IntelFsp2Pkg: Apply uncrustify changes

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3737

Apply uncrustify changes to .c/.h files in the IntelFsp2Pkg package

Cc: Andrew Fish <afish@apple.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>

5 files changed, 134 insertions, 135 deletions

diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.c b/IntelFsp2Pkg/FspSecCore/SecFsp.c
index 216f7bb6c5..ae03fa228e 100644
--- a/IntelFsp2Pkg/FspSecCore/SecFsp.c
+++ b/IntelFsp2Pkg/FspSecCore/SecFsp.c
@@ -17,20 +17,20 @@
 

 **/

 UINT64

-FspGetExceptionHandler(

+FspGetExceptionHandler (

   IN  UINT64  IdtEntryTemplate

   )

 {

   UINT32                    Entry;

   UINT64                    ExceptionHandler;

-  IA32_IDT_GATE_DESCRIPTOR *IdtGateDescriptor;

-  FSP_INFO_HEADER          *FspInfoHeader;

-

-  FspInfoHeader     = (FSP_INFO_HEADER *)AsmGetFspInfoHeader();

-  ExceptionHandler  = IdtEntryTemplate;

-  IdtGateDescriptor = (IA32_IDT_GATE_DESCRIPTOR *)&ExceptionHandler;

-  Entry = (IdtGateDescriptor->Bits.OffsetHigh << 16) | IdtGateDescriptor->Bits.OffsetLow;

-  Entry = FspInfoHeader->ImageBase + FspInfoHeader->ImageSize - (~Entry + 1);

+  IA32_IDT_GATE_DESCRIPTOR  *IdtGateDescriptor;

+  FSP_INFO_HEADER           *FspInfoHeader;

+

+  FspInfoHeader                      = (FSP_INFO_HEADER *)AsmGetFspInfoHeader ();

+  ExceptionHandler                   = IdtEntryTemplate;

+  IdtGateDescriptor                  = (IA32_IDT_GATE_DESCRIPTOR *)&ExceptionHandler;

+  Entry                              = (IdtGateDescriptor->Bits.OffsetHigh << 16) | IdtGateDescriptor->Bits.OffsetLow;

+  Entry                              = FspInfoHeader->ImageBase + FspInfoHeader->ImageSize - (~Entry + 1);

   IdtGateDescriptor->Bits.OffsetHigh = (UINT16)(Entry >> 16);

   IdtGateDescriptor->Bits.OffsetLow  = (UINT16)Entry;

 

@@ -46,13 +46,13 @@ FspGetExceptionHandler(
 VOID

 EFIAPI

 SecGetPlatformData (

-  IN OUT  FSP_GLOBAL_DATA    *FspData

+  IN OUT  FSP_GLOBAL_DATA  *FspData

   )

 {

-  FSP_PLAT_DATA    *FspPlatformData;

-  UINT32            TopOfCar;

-  UINT32           *StackPtr;

-  UINT32            DwordSize;

+  FSP_PLAT_DATA  *FspPlatformData;

+  UINT32         TopOfCar;

+  UINT32         *StackPtr;

+  UINT32         DwordSize;

 

   FspPlatformData = &FspData->PlatformData;

 

@@ -61,7 +61,7 @@ SecGetPlatformData (
   // reside in the bottom of stack, left untouched by normal stack operation.

   //

 

-  FspPlatformData->DataPtr   = NULL;

+  FspPlatformData->DataPtr             = NULL;

   FspPlatformData->MicrocodeRegionBase = 0;

   FspPlatformData->MicrocodeRegionSize = 0;

   FspPlatformData->CodeRegionBase      = 0;

@@ -70,7 +70,7 @@ SecGetPlatformData (
   //

   // Pointer to the size field

   //

-  TopOfCar = PcdGet32(PcdTemporaryRamBase) + PcdGet32(PcdTemporaryRamSize);

+  TopOfCar = PcdGet32 (PcdTemporaryRamBase) + PcdGet32 (PcdTemporaryRamSize);

   StackPtr = (UINT32 *)(TopOfCar - sizeof (UINT32));

 

   if (*(StackPtr - 1) == FSP_MCUD_SIGNATURE) {

@@ -114,26 +114,26 @@ SecGetPlatformData (
 **/

 VOID

 FspGlobalDataInit (

-  IN OUT  FSP_GLOBAL_DATA    *PeiFspData,

-  IN UINT32                   BootLoaderStack,

-  IN UINT8                    ApiIdx

+  IN OUT  FSP_GLOBAL_DATA  *PeiFspData,

+  IN UINT32                BootLoaderStack,

+  IN UINT8                 ApiIdx

   )

 {

-  VOID              *FspmUpdDataPtr;

-  CHAR8              ImageId[9];

-  UINTN              Idx;

+  VOID   *FspmUpdDataPtr;

+  CHAR8  ImageId[9];

+  UINTN  Idx;

 

   //

   // Set FSP Global Data pointer

   //

-  SetFspGlobalDataPointer    (PeiFspData);

-  ZeroMem  ((VOID *)PeiFspData, sizeof(FSP_GLOBAL_DATA));

+  SetFspGlobalDataPointer (PeiFspData);

+  ZeroMem ((VOID *)PeiFspData, sizeof (FSP_GLOBAL_DATA));

 

-  PeiFspData->Signature            = FSP_GLOBAL_DATA_SIGNATURE;

-  PeiFspData->Version              = 0;

-  PeiFspData->CoreStack            = BootLoaderStack;

-  PeiFspData->PerfIdx              = 2;

-  PeiFspData->PerfSig              = FSP_PERFORMANCE_DATA_SIGNATURE;

+  PeiFspData->Signature = FSP_GLOBAL_DATA_SIGNATURE;

+  PeiFspData->Version   = 0;

+  PeiFspData->CoreStack = BootLoaderStack;

+  PeiFspData->PerfIdx   = 2;

+  PeiFspData->PerfSig   = FSP_PERFORMANCE_DATA_SIGNATURE;

 

   SetFspMeasurePoint (FSP_PERF_ID_API_FSP_MEMORY_INIT_ENTRY);

 

@@ -141,7 +141,7 @@ FspGlobalDataInit (
   // Get FSP Header offset

   // It may have multiple FVs, so look into the last one for FSP header

   //

-  PeiFspData->FspInfoHeader      = (FSP_INFO_HEADER *)AsmGetFspInfoHeader();

+  PeiFspData->FspInfoHeader = (FSP_INFO_HEADER *)AsmGetFspInfoHeader ();

   SecGetPlatformData (PeiFspData);

 

   //

@@ -152,10 +152,11 @@ FspGlobalDataInit (
   //

   // Set UPD pointer

   //

-  FspmUpdDataPtr = (VOID *) GetFspApiParameter ();

+  FspmUpdDataPtr = (VOID *)GetFspApiParameter ();

   if (FspmUpdDataPtr == NULL) {

     FspmUpdDataPtr = (VOID *)(PeiFspData->FspInfoHeader->ImageBase + PeiFspData->FspInfoHeader->CfgRegionOffset);

   }

+

   SetFspUpdDataPointer (FspmUpdDataPtr);

   SetFspMemoryInitUpdDataPointer (FspmUpdDataPtr);

   SetFspSiliconInitUpdDataPointer (NULL);

@@ -186,17 +187,21 @@ FspGlobalDataInit (
   for (Idx = 0; Idx < 8; Idx++) {

     ImageId[Idx] = PeiFspData->FspInfoHeader->ImageId[Idx];

   }

+

   ImageId[Idx] = 0;

 

-  DEBUG ((DEBUG_INFO | DEBUG_INIT, "\n============= FSP Spec v%d.%d Header Revision v%x (%a v%x.%x.%x.%x) =============\n", \

-         (PeiFspData->FspInfoHeader->SpecVersion >> 4) & 0xF, \

-         PeiFspData->FspInfoHeader->SpecVersion & 0xF, \

-         PeiFspData->FspInfoHeader->HeaderRevision, \

-         ImageId, \

-         (PeiFspData->FspInfoHeader->ImageRevision >> 24) & 0xFF, \

-         (PeiFspData->FspInfoHeader->ImageRevision >> 16) & 0xFF, \

-         (PeiFspData->FspInfoHeader->ImageRevision >> 8) & 0xFF, \

-         PeiFspData->FspInfoHeader->ImageRevision & 0xFF));

+  DEBUG ((

+    DEBUG_INFO | DEBUG_INIT,

+    "\n============= FSP Spec v%d.%d Header Revision v%x (%a v%x.%x.%x.%x) =============\n", \

+    (PeiFspData->FspInfoHeader->SpecVersion >> 4) & 0xF, \

+    PeiFspData->FspInfoHeader->SpecVersion & 0xF, \

+    PeiFspData->FspInfoHeader->HeaderRevision, \

+    ImageId, \

+    (PeiFspData->FspInfoHeader->ImageRevision >> 24) & 0xFF, \

+    (PeiFspData->FspInfoHeader->ImageRevision >> 16) & 0xFF, \

+    (PeiFspData->FspInfoHeader->ImageRevision >> 8) & 0xFF, \

+    PeiFspData->FspInfoHeader->ImageRevision & 0xFF

+    ));

 }

 

 /**

@@ -208,11 +213,11 @@ FspGlobalDataInit (
 **/

 VOID

 FspDataPointerFixUp (

-  IN UINT32   OffsetGap

+  IN UINT32  OffsetGap

   )

 {

   FSP_GLOBAL_DATA  *NewFspData;

 

-  NewFspData = (FSP_GLOBAL_DATA *)((UINTN)GetFspGlobalDataPointer() + (UINTN)OffsetGap);

+  NewFspData = (FSP_GLOBAL_DATA *)((UINTN)GetFspGlobalDataPointer () + (UINTN)OffsetGap);

   SetFspGlobalDataPointer (NewFspData);

 }

diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h b/IntelFsp2Pkg/FspSecCore/SecFsp.h
index 75f3ef463c..aacd32f7f7 100644
--- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
+++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
@@ -31,7 +31,7 @@
 

 **/

 UINT64

-FspGetExceptionHandler(

+FspGetExceptionHandler (

   IN  UINT64  IdtEntryTemplate

   );

 

@@ -47,12 +47,11 @@ FspGetExceptionHandler(
 **/

 VOID

 FspGlobalDataInit (

-  IN OUT  FSP_GLOBAL_DATA    *PeiFspData,

-  IN UINT32                   BootLoaderStack,

-  IN UINT8                    ApiIdx

+  IN OUT  FSP_GLOBAL_DATA  *PeiFspData,

+  IN UINT32                BootLoaderStack,

+  IN UINT8                 ApiIdx

   );

 

-

 /**

 

   Adjust the FSP data pointers after the stack is migrated to memory.

@@ -62,10 +61,9 @@ FspGlobalDataInit (
 **/

 VOID

 FspDataPointerFixUp (

-  IN UINT32   OffsetGap

+  IN UINT32  OffsetGap

   );

 

-

 /**

   This interface returns the base address of FSP binary.

 

diff --git a/IntelFsp2Pkg/FspSecCore/SecFspApiChk.c b/IntelFsp2Pkg/FspSecCore/SecFspApiChk.c
index 1334959005..7d6ef11fe7 100644
--- a/IntelFsp2Pkg/FspSecCore/SecFspApiChk.c
+++ b/IntelFsp2Pkg/FspSecCore/SecFspApiChk.c
@@ -7,7 +7,6 @@
 

 #include "SecFsp.h"

 

-

 /**

   This function check the FSP API calling condition.

 

@@ -18,14 +17,14 @@
 EFI_STATUS

 EFIAPI

 FspApiCallingCheck (

-  IN UINT8     ApiIdx,

-  IN VOID     *ApiParam

+  IN UINT8  ApiIdx,

+  IN VOID   *ApiParam

   )

 {

-  EFI_STATUS                Status;

-  FSP_GLOBAL_DATA           *FspData;

+  EFI_STATUS       Status;

+  FSP_GLOBAL_DATA  *FspData;

 

-  Status = EFI_SUCCESS;

+  Status  = EFI_SUCCESS;

   FspData = GetFspGlobalDataPointer ();

 

   if (ApiIdx == NotifyPhaseApiIndex) {

diff --git a/IntelFsp2Pkg/FspSecCore/SecMain.c b/IntelFsp2Pkg/FspSecCore/SecMain.c
index 540571187a..d376fb8361 100644
--- a/IntelFsp2Pkg/FspSecCore/SecMain.c
+++ b/IntelFsp2Pkg/FspSecCore/SecMain.c
@@ -8,11 +8,11 @@
 #include "SecMain.h"

 #include "SecFsp.h"

 

-EFI_PEI_TEMPORARY_RAM_SUPPORT_PPI gSecTemporaryRamSupportPpi = {

+EFI_PEI_TEMPORARY_RAM_SUPPORT_PPI  gSecTemporaryRamSupportPpi = {

   SecTemporaryRamSupport

 };

 

-EFI_PEI_PPI_DESCRIPTOR            mPeiSecPlatformInformationPpi[] = {

+EFI_PEI_PPI_DESCRIPTOR  mPeiSecPlatformInformationPpi[] = {

   {

     EFI_PEI_PPI_DESCRIPTOR_PPI,

     &gFspInApiModePpiGuid,

@@ -50,21 +50,21 @@ UINT64  mIdtEntryTemplate = 0xffff8e000008ffe4ULL;
 VOID

 EFIAPI

 SecStartup (

-  IN UINT32                   SizeOfRam,

-  IN UINT32                   TempRamBase,

-  IN VOID                    *BootFirmwareVolume,

-  IN PEI_CORE_ENTRY           PeiCore,

-  IN UINT32                   BootLoaderStack,

-  IN UINT32                   ApiIdx

+  IN UINT32          SizeOfRam,

+  IN UINT32          TempRamBase,

+  IN VOID            *BootFirmwareVolume,

+  IN PEI_CORE_ENTRY  PeiCore,

+  IN UINT32          BootLoaderStack,

+  IN UINT32          ApiIdx

   )

 {

-  EFI_SEC_PEI_HAND_OFF        SecCoreData;

-  IA32_DESCRIPTOR             IdtDescriptor;

-  SEC_IDT_TABLE               IdtTableInStack;

-  UINT32                      Index;

-  FSP_GLOBAL_DATA             PeiFspData;

-  UINT64                      ExceptionHandler;

-  UINTN                       IdtSize;

+  EFI_SEC_PEI_HAND_OFF  SecCoreData;

+  IA32_DESCRIPTOR       IdtDescriptor;

+  SEC_IDT_TABLE         IdtTableInStack;

+  UINT32                Index;

+  FSP_GLOBAL_DATA       PeiFspData;

+  UINT64                ExceptionHandler;

+  UINTN                 IdtSize;

 

   //

   // Process all libraries constructor function linked to SecCore.

@@ -117,10 +117,11 @@ SecStartup (
   IdtTableInStack.PeiService = 0;

   AsmReadIdtr (&IdtDescriptor);

   if (IdtDescriptor.Base == 0) {

-    ExceptionHandler = FspGetExceptionHandler(mIdtEntryTemplate);

-    for (Index = 0; Index < FixedPcdGet8(PcdFspMaxInterruptSupported); Index ++) {

-      CopyMem ((VOID*)&IdtTableInStack.IdtTable[Index], (VOID*)&ExceptionHandler, sizeof (UINT64));

+    ExceptionHandler = FspGetExceptionHandler (mIdtEntryTemplate);

+    for (Index = 0; Index < FixedPcdGet8 (PcdFspMaxInterruptSupported); Index++) {

+      CopyMem ((VOID *)&IdtTableInStack.IdtTable[Index], (VOID *)&ExceptionHandler, sizeof (UINT64));

     }

+

     IdtSize = sizeof (IdtTableInStack.IdtTable);

   } else {

     IdtSize = IdtDescriptor.Limit + 1;

@@ -128,12 +129,13 @@ SecStartup (
       //

       // ERROR: IDT table size from boot loader is larger than FSP can support, DeadLoop here!

       //

-      CpuDeadLoop();

+      CpuDeadLoop ();

     } else {

-      CopyMem ((VOID *) (UINTN) &IdtTableInStack.IdtTable, (VOID *) IdtDescriptor.Base, IdtSize);

+      CopyMem ((VOID *)(UINTN)&IdtTableInStack.IdtTable, (VOID *)IdtDescriptor.Base, IdtSize);

     }

   }

-  IdtDescriptor.Base  = (UINTN) &IdtTableInStack.IdtTable;

+

+  IdtDescriptor.Base  = (UINTN)&IdtTableInStack.IdtTable;

   IdtDescriptor.Limit = (UINT16)(IdtSize - 1);

 

   AsmWriteIdtr (&IdtDescriptor);

@@ -154,18 +156,18 @@ SecStartup (
   // Support FSP reserved temporary memory from the whole temporary memory provided by bootloader.

   // FSP reserved temporary memory will not be given to PeiCore.

   //

-  SecCoreData.TemporaryRamBase       = (UINT8 *)(UINTN) TempRamBase  + PcdGet32 (PcdFspPrivateTemporaryRamSize);

-  SecCoreData.TemporaryRamSize       = SizeOfRam - PcdGet32 (PcdFspPrivateTemporaryRamSize);

+  SecCoreData.TemporaryRamBase = (UINT8 *)(UINTN)TempRamBase  + PcdGet32 (PcdFspPrivateTemporaryRamSize);

+  SecCoreData.TemporaryRamSize = SizeOfRam - PcdGet32 (PcdFspPrivateTemporaryRamSize);

   if (PcdGet8 (PcdFspHeapSizePercentage) == 0) {

-    SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;

-    SecCoreData.PeiTemporaryRamSize    = SecCoreData.TemporaryRamSize;

-    SecCoreData.StackBase              = (VOID *)GetFspEntryStack(); // Share the same boot loader stack

-    SecCoreData.StackSize              = 0;

+    SecCoreData.PeiTemporaryRamBase = SecCoreData.TemporaryRamBase;

+    SecCoreData.PeiTemporaryRamSize = SecCoreData.TemporaryRamSize;

+    SecCoreData.StackBase           = (VOID *)GetFspEntryStack ();   // Share the same boot loader stack

+    SecCoreData.StackSize           = 0;

   } else {

-    SecCoreData.PeiTemporaryRamBase    = SecCoreData.TemporaryRamBase;

-    SecCoreData.PeiTemporaryRamSize    = SecCoreData.TemporaryRamSize * PcdGet8 (PcdFspHeapSizePercentage) / 100;

-    SecCoreData.StackBase              = (VOID*)(UINTN)((UINTN)SecCoreData.TemporaryRamBase + SecCoreData.PeiTemporaryRamSize);

-    SecCoreData.StackSize              = SecCoreData.TemporaryRamSize - SecCoreData.PeiTemporaryRamSize;

+    SecCoreData.PeiTemporaryRamBase = SecCoreData.TemporaryRamBase;

+    SecCoreData.PeiTemporaryRamSize = SecCoreData.TemporaryRamSize * PcdGet8 (PcdFspHeapSizePercentage) / 100;

+    SecCoreData.StackBase           = (VOID *)(UINTN)((UINTN)SecCoreData.TemporaryRamBase + SecCoreData.PeiTemporaryRamSize);

+    SecCoreData.StackSize           = SecCoreData.TemporaryRamSize - SecCoreData.PeiTemporaryRamSize;

   }

 

   DEBUG ((DEBUG_INFO, "Fsp BootFirmwareVolumeBase - 0x%x\n", SecCoreData.BootFirmwareVolumeBase));

@@ -207,22 +209,22 @@ SecStartup (
 EFI_STATUS

 EFIAPI

 SecTemporaryRamSupport (

-  IN CONST EFI_PEI_SERVICES   **PeiServices,

-  IN EFI_PHYSICAL_ADDRESS     TemporaryMemoryBase,

-  IN EFI_PHYSICAL_ADDRESS     PermanentMemoryBase,

-  IN UINTN                    CopySize

+  IN CONST EFI_PEI_SERVICES  **PeiServices,

+  IN EFI_PHYSICAL_ADDRESS    TemporaryMemoryBase,

+  IN EFI_PHYSICAL_ADDRESS    PermanentMemoryBase,

+  IN UINTN                   CopySize

   )

 {

-  IA32_DESCRIPTOR   IdtDescriptor;

-  VOID*             OldHeap;

-  VOID*             NewHeap;

-  VOID*             OldStack;

-  VOID*             NewStack;

-  UINTN             HeapSize;

-  UINTN             StackSize;

+  IA32_DESCRIPTOR  IdtDescriptor;

+  VOID             *OldHeap;

+  VOID             *NewHeap;

+  VOID             *OldStack;

+  VOID             *NewStack;

+  UINTN            HeapSize;

+  UINTN            StackSize;

 

-  UINTN             CurrentStack;

-  UINTN             FspStackBase;

+  UINTN  CurrentStack;

+  UINTN  FspStackBase;

 

   //

   // Override OnSeparateStack to 1 because this function will switch stack to permanent memory

@@ -231,33 +233,31 @@ SecTemporaryRamSupport (
   GetFspGlobalDataPointer ()->OnSeparateStack = 1;

 

   if (PcdGet8 (PcdFspHeapSizePercentage) == 0) {

-

-    CurrentStack = AsmReadEsp();

-    FspStackBase = (UINTN)GetFspEntryStack();

+    CurrentStack = AsmReadEsp ();

+    FspStackBase = (UINTN)GetFspEntryStack ();

 

     StackSize = FspStackBase - CurrentStack;

     HeapSize  = CopySize;

 

-    OldHeap = (VOID*)(UINTN)TemporaryMemoryBase;

-    NewHeap = (VOID*)((UINTN)PermanentMemoryBase);

+    OldHeap = (VOID *)(UINTN)TemporaryMemoryBase;

+    NewHeap = (VOID *)((UINTN)PermanentMemoryBase);

 

-    OldStack = (VOID*)CurrentStack;

+    OldStack = (VOID *)CurrentStack;

     //

-    //The old stack is copied at the end of the stack region because stack grows down.

+    // The old stack is copied at the end of the stack region because stack grows down.

     //

-    NewStack = (VOID*)((UINTN)PermanentMemoryBase - StackSize);

-

+    NewStack = (VOID *)((UINTN)PermanentMemoryBase - StackSize);

   } else {

-    HeapSize   = CopySize * PcdGet8 (PcdFspHeapSizePercentage) / 100 ;

-    StackSize  = CopySize - HeapSize;

+    HeapSize  = CopySize * PcdGet8 (PcdFspHeapSizePercentage) / 100;

+    StackSize = CopySize - HeapSize;

 

-    OldHeap = (VOID*)(UINTN)TemporaryMemoryBase;

-    NewHeap = (VOID*)((UINTN)PermanentMemoryBase + StackSize);

-

-    OldStack = (VOID*)((UINTN)TemporaryMemoryBase + HeapSize);

-    NewStack = (VOID*)(UINTN)PermanentMemoryBase;

+    OldHeap = (VOID *)(UINTN)TemporaryMemoryBase;

+    NewHeap = (VOID *)((UINTN)PermanentMemoryBase + StackSize);

 

+    OldStack = (VOID *)((UINTN)TemporaryMemoryBase + HeapSize);

+    NewStack = (VOID *)(UINTN)PermanentMemoryBase;

   }

+

   //

   // Migrate Heap

   //

@@ -268,7 +268,6 @@ SecTemporaryRamSupport (
   //

   CopyMem (NewStack, OldStack, StackSize);

 

-

   //

   // We need *not* fix the return address because currently,

   // The PeiCore is executed in flash.

@@ -293,8 +292,8 @@ SecTemporaryRamSupport (
   // permanent memory.

   //

   SecSwitchStack (

-    (UINT32) (UINTN) OldStack,

-    (UINT32) (UINTN) NewStack

+    (UINT32)(UINTN)OldStack,

+    (UINT32)(UINTN)NewStack

     );

 

   return EFI_SUCCESS;

diff --git a/IntelFsp2Pkg/FspSecCore/SecMain.h b/IntelFsp2Pkg/FspSecCore/SecMain.h
index f6333b0ffb..7794255af1 100644
--- a/IntelFsp2Pkg/FspSecCore/SecMain.h
+++ b/IntelFsp2Pkg/FspSecCore/SecMain.h
@@ -8,7 +8,6 @@
 #ifndef _SEC_CORE_H_

 #define _SEC_CORE_H_

 

-

 #include <PiPei.h>

 #include <Ppi/TemporaryRamSupport.h>

 

@@ -27,7 +26,7 @@
 typedef VOID (*PEI_CORE_ENTRY) ( \

   IN CONST  EFI_SEC_PEI_HAND_OFF    *SecCoreData, \

   IN CONST  EFI_PEI_PPI_DESCRIPTOR  *PpiList \

-);

+  );

 

 typedef struct _SEC_IDT_TABLE {

   //

@@ -36,8 +35,8 @@ typedef struct _SEC_IDT_TABLE {
   // Note: For IA32, only the 4 bytes immediately preceding IDT is used to store

   // EFI_PEI_SERVICES**

   //

-  UINT64            PeiService;

-  UINT64            IdtTable[FixedPcdGet8 (PcdFspMaxInterruptSupported)];

+  UINT64    PeiService;

+  UINT64    IdtTable[FixedPcdGet8 (PcdFspMaxInterruptSupported)];

 } SEC_IDT_TABLE;

 

 /**

@@ -52,8 +51,8 @@ typedef struct _SEC_IDT_TABLE {
 VOID

 EFIAPI

 SecSwitchStack (

-  IN UINT32   TemporaryMemoryBase,

-  IN UINT32   PermenentMemoryBase

+  IN UINT32  TemporaryMemoryBase,

+  IN UINT32  PermenentMemoryBase

   );

 

 /**

@@ -75,13 +74,12 @@ SecSwitchStack (
 EFI_STATUS

 EFIAPI

 SecTemporaryRamSupport (

-  IN CONST EFI_PEI_SERVICES   **PeiServices,

-  IN EFI_PHYSICAL_ADDRESS     TemporaryMemoryBase,

-  IN EFI_PHYSICAL_ADDRESS     PermanentMemoryBase,

-  IN UINTN                    CopySize

+  IN CONST EFI_PEI_SERVICES  **PeiServices,

+  IN EFI_PHYSICAL_ADDRESS    TemporaryMemoryBase,

+  IN EFI_PHYSICAL_ADDRESS    PermanentMemoryBase,

+  IN UINTN                   CopySize

   );

 

-

 /**

 

   Entry point to the C language phase of SEC. After the SEC assembly

@@ -102,12 +100,12 @@ SecTemporaryRamSupport (
 VOID

 EFIAPI

 SecStartup (

-  IN UINT32                   SizeOfRam,

-  IN UINT32                   TempRamBase,

-  IN VOID                    *BootFirmwareVolume,

-  IN PEI_CORE_ENTRY           PeiCore,

-  IN UINT32                   BootLoaderStack,

-  IN UINT32                   ApiIdx

+  IN UINT32          SizeOfRam,

+  IN UINT32          TempRamBase,

+  IN VOID            *BootFirmwareVolume,

+  IN PEI_CORE_ENTRY  PeiCore,

+  IN UINT32          BootLoaderStack,

+  IN UINT32          ApiIdx

   );

 

 /**