commit dd4e2bcfaa068fd958169944e99149fa06b58354
author Yao, Jiewen <Jiewen.yao@intel.com> Tue Jul 07 05:47:08 2015 +0000
committer jyao1 <jyao1@Edk2> Tue Jul 07 05:47:08 2015 +0000
tree 94c6592253a19595c6a29632e6bc1e4208d0a9f6
parent ebf735f11907e31e1fa80f21737d277d8a964e17

FspInitPei function calling parameters not matching with the function definition.

IntelFspWrapperPkg, In the file FspInitPeiV1.c and FspInitPeiV2.c, there are function calling:
GetStackInfo (BootMode, FALSE, &StackBase, &StackSize);

But the function GetStackInfo() defined as EFI_STATUS EFIAPI GetStackInfo (
  IN  UINT32               BootMode,
  IN  BOOLEAN              FspInitDone,
  OUT UINT64               *StackSize,
  OUT EFI_PHYSICAL_ADDRESS *StackBase
  )

It should be GetStackInfo (BootMode, FALSE, &StackSize, &StackBase); 

Contributed-under: TianoCore Contribution Agreement 1.0

Signed-off-by: "Yao, Jiewen" <Jiewen.yao@intel.com>
Reviewed-by: "Ma, Maurice" <maurice.ma@intel.com>
Reviewed-by: "Rangarajan, Ravi P" <ravi.p.rangarajan@intel.com>


git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@17850 6f19259b-4bc3-4df7-8a09-765794883524

IntelFspWrapperPkg/FspInitPei/FspInitPeiV1.c

diff --git a/IntelFspWrapperPkg/FspInitPei/FspInitPeiV1.c b/IntelFspWrapperPkg/FspInitPei/FspInitPeiV1.c
index da7f823..b145b50 100644
--- a/IntelFspWrapperPkg/FspInitPei/FspInitPeiV1.c
+++ b/IntelFspWrapperPkg/FspInitPei/FspInitPeiV1.c
@@ -47,7 +47,7 @@
   //

   BootMode = GetBootMode ();

 

-  GetStackInfo (BootMode, TRUE, &StackBase, &StackSize);

+  GetStackInfo (BootMode, TRUE, &StackSize, &StackBase);

   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));

   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));

   CallPeiCoreEntryPoint (

@@ -81,7 +81,7 @@
   PeiServicesGetBootMode (&BootMode);

   DEBUG ((DEBUG_INFO, "BootMode - 0x%x\n", BootMode));

 

-  GetStackInfo (BootMode, FALSE, &StackBase, &StackSize);

+  GetStackInfo (BootMode, FALSE, &StackSize, &StackBase);

   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));

   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));

 

IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c

diff --git a/IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c b/IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c
index 35a994e..50b150a 100644
--- a/IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c
+++ b/IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c
@@ -148,7 +148,7 @@
   PeiServicesGetBootMode (&BootMode);

   DEBUG ((DEBUG_INFO, "BootMode - 0x%x\n", BootMode));

 

-  GetStackInfo (BootMode, FALSE, &StackBase, &StackSize);

+  GetStackInfo (BootMode, FALSE, &StackSize, &StackBase);

   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));

   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));