FspInitPei function calling parameters not matching with the function definition.
authorYao, Jiewen <Jiewen.yao@intel.com>
Tue, 7 Jul 2015 05:47:08 +0000 (05:47 +0000)
committerjyao1 <jyao1@Edk2>
Tue, 7 Jul 2015 05:47:08 +0000 (05:47 +0000)
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
IntelFspWrapperPkg/FspInitPei/FspInitPeiV2.c

index da7f823..b145b50 100644 (file)
@@ -47,7 +47,7 @@ ContinuationFunc (
   //\r
   BootMode = GetBootMode ();\r
 \r
-  GetStackInfo (BootMode, TRUE, &StackBase, &StackSize);\r
+  GetStackInfo (BootMode, TRUE, &StackSize, &StackBase);\r
   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));\r
   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));\r
   CallPeiCoreEntryPoint (\r
@@ -81,7 +81,7 @@ PeiFspInit (
   PeiServicesGetBootMode (&BootMode);\r
   DEBUG ((DEBUG_INFO, "BootMode - 0x%x\n", BootMode));\r
 \r
-  GetStackInfo (BootMode, FALSE, &StackBase, &StackSize);\r
+  GetStackInfo (BootMode, FALSE, &StackSize, &StackBase);\r
   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));\r
   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));\r
 \r
index 35a994e..50b150a 100644 (file)
@@ -148,7 +148,7 @@ PeiFspMemoryInit (
   PeiServicesGetBootMode (&BootMode);\r
   DEBUG ((DEBUG_INFO, "BootMode - 0x%x\n", BootMode));\r
 \r
-  GetStackInfo (BootMode, FALSE, &StackBase, &StackSize);\r
+  GetStackInfo (BootMode, FALSE, &StackSize, &StackBase);\r
   DEBUG ((DEBUG_INFO, "StackBase - 0x%x\n", StackBase));\r
   DEBUG ((DEBUG_INFO, "StackSize - 0x%x\n", StackSize));\r
 \r