[PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64 build.

[PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64 build.

[Chiu, Chasel]

FSP_STATUS_RESET_REQUIRED_* will be either 32bit or 64bit basing on
the build and this patch series extended the support for 64bit build.

Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Isaac Oram <isaac.w.oram@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>

V2: Change ResetType from UINTN to EFI_STATUS

Chasel Chiu (2):
  MinPlatformPkg/FspWrapper: FspResetType support for X64 build.
  WhitleySiliconPkg: FspResetType support for X64 build.

 Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c | 2 +-
 Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c | 2 +-
 Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c                     | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

--
2.35.0.windows.1

[PATCH v2 1/2] MinPlatformPkg/FspWrapper: FspResetType support for X64 build.

[Chiu, Chasel]

FSP_STATUS_RESET_REQUIRED_* will be either 32bit or 64bit basing
on the build and this patch extended the support for 64bit build.

Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Isaac Oram <isaac.w.oram@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
---
 Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c | 2 +-
 Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c
index b4d509db85..832027d3ee 100644
--- a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c
+++ b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLib/DxeFspWrapperPlatformResetLib.c
@@ -22,7 +22,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 VOID
 EFIAPI
 CallFspWrapperResetSystem (
-  UINT32    ResetType
+  EFI_STATUS    ResetType
   )
 {
   EFI_RESET_TYPE             EfiResetType;
diff --git a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c
index 191bcb2fbb..2f6f7d058c 100644
--- a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c
+++ b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib/PeiFspWrapperPlatformResetLib.c
@@ -21,7 +21,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 VOID
 EFIAPI
 CallFspWrapperResetSystem (
-  UINT32    ResetType
+  EFI_STATUS    ResetType
   )
 {
   EFI_RESET_TYPE             EfiResetType;
-- 
2.35.0.windows.1

[PATCH v2 2/2] WhitleySiliconPkg: FspResetType support for X64 build.

[Chiu, Chasel]

FSP_STATUS_RESET_REQUIRED_* will be either 32bit or 64bit basing
on the build and this patch extended the support for 64bit build.

Cc: Isaac Oram <isaac.w.oram@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
---
 Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c b/Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c
index 95a1f2a33c..e61e8178ed 100644
--- a/Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c
+++ b/Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlatformLib.c
@@ -190,7 +190,7 @@ GetS3MemoryInfo (
 VOID
 EFIAPI
 CallFspWrapperResetSystem (
-  IN UINT32    FspStatusResetType
+  IN EFI_STATUS    FspStatusResetType
   )
 {
   //
-- 
2.35.0.windows.1

Re: [PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64 build.

[Nate DeSimone]

For the series...

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>

> -----Original Message-----
> From: Chiu, Chasel <chasel.chiu@intel.com>
> Sent: Monday, July 18, 2022 5:37 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Oram, Isaac W
> <isaac.w.oram@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>;
> Dong, Eric <eric.dong@intel.com>
> Subject: [PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64
> build.
> 
> FSP_STATUS_RESET_REQUIRED_* will be either 32bit or 64bit basing on the
> build and this patch series extended the support for 64bit build.
> 
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Isaac Oram <isaac.w.oram@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> 
> V2: Change ResetType from UINTN to EFI_STATUS
> 
> Chasel Chiu (2):
>   MinPlatformPkg/FspWrapper: FspResetType support for X64 build.
>   WhitleySiliconPkg: FspResetType support for X64 build.
> 
> 
> Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatfor
> mLib/DxeFspWrapperPlatformResetLib.c | 2 +-
> Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatfor
> mLib/PeiFspWrapperPlatformResetLib.c | 2 +-
> 
> Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapper
> PlatformLib.c                     | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> --
> 2.35.0.windows.1

Re: [PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64 build.

[Chiu, Chasel]

This patch series has been merged:
cfd7e5f231fd644a5cb585d8ab68aa701af6a286
0f836bd4e3e18b9bd0d1a73b849a554519ee82fa

Thanks,
Chasel



> -----Original Message-----
> From: Chiu, Chasel <chasel.chiu@intel.com>
> Sent: Monday, July 18, 2022 5:37 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>;
> Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>
> Subject: [PATCH v2 0/2] MinPlatformPkg: FspResetType support for X64 build.
> 
> FSP_STATUS_RESET_REQUIRED_* will be either 32bit or 64bit basing on the
> build and this patch series extended the support for 64bit build.
> 
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Isaac Oram <isaac.w.oram@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> 
> V2: Change ResetType from UINTN to EFI_STATUS
> 
> Chasel Chiu (2):
>   MinPlatformPkg/FspWrapper: FspResetType support for X64 build.
>   WhitleySiliconPkg: FspResetType support for X64 build.
> 
> 
> Platform/Intel/MinPlatformPkg/FspWrapper/Library/DxeFspWrapperPlatformLi
> b/DxeFspWrapperPlatformResetLib.c | 2 +-
> Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperPlatformLib
> /PeiFspWrapperPlatformResetLib.c | 2 +-
> 
> Silicon/Intel/WhitleySiliconPkg/Library/FspWrapperPlatformLib/FspWrapperPlat
> formLib.c                     | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> --
> 2.35.0.windows.1