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

Refer to Uefi spec 2.8, the ResetData is valid while ResetStatus
is EFI_SUCCESS regardless of the ResetType is EfiResetPlatformSpecific
or not.

Cc: Jian J Wang <jian.j.w...@intel.com>
Cc: Hao Wu <hao.a...@intel.com>
Cc: Ray Ni <ray...@intel.com>
Cc: Star Zeng <star.z...@intel.com>
Cc: Liming Gao <liming....@intel.com>
Cc: Sean Brogan <sean.bro...@microsoft.com>
Cc: Michael Turner <michael.tur...@microsoft.com>
Cc: Bret Barkelew <bret.barke...@microsoft.com>
Signed-off-by: Zhichao Gao <zhichao....@intel.com>
Reviewed-by: Hao Wu <hao.a...@intel.com>
---
 MdeModulePkg/Core/Pei/PeiMain.h     | 4 +---
 MdeModulePkg/Core/Pei/Reset/Reset.c | 6 ++----
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/MdeModulePkg/Core/Pei/PeiMain.h b/MdeModulePkg/Core/Pei/PeiMain.h
index 0aed4f4685..f2aa97c664 100644
--- a/MdeModulePkg/Core/Pei/PeiMain.h
+++ b/MdeModulePkg/Core/Pei/PeiMain.h
@@ -1097,9 +1097,7 @@ PeiResetSystem (
                             the data buffer starts with a Null-terminated 
string, optionally
                             followed by additional binary data. The string is 
a description
                             that the caller may use to further indicate the 
reason for the
-                            system reset. ResetData is only valid if 
ResetStatus is something
-                            other than EFI_SUCCESS unless the ResetType is 
EfiResetPlatformSpecific
-                            where a minimum amount of ResetData is always 
required.
+                            system reset.
 
 **/
 VOID
diff --git a/MdeModulePkg/Core/Pei/Reset/Reset.c 
b/MdeModulePkg/Core/Pei/Reset/Reset.c
index 263eace239..0c3cfad035 100644
--- a/MdeModulePkg/Core/Pei/Reset/Reset.c
+++ b/MdeModulePkg/Core/Pei/Reset/Reset.c
@@ -1,7 +1,7 @@
 /** @file
   Pei Core Reset System Support
 
-Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.<BR>
 SPDX-License-Identifier: BSD-2-Clause-Patent
 
 **/
@@ -72,9 +72,7 @@ PeiResetSystem (
                             the data buffer starts with a Null-terminated 
string, optionally
                             followed by additional binary data. The string is 
a description
                             that the caller may use to further indicate the 
reason for the
-                            system reset. ResetData is only valid if 
ResetStatus is something
-                            other than EFI_SUCCESS unless the ResetType is 
EfiResetPlatformSpecific
-                            where a minimum amount of ResetData is always 
required.
+                            system reset.
 
 **/
 VOID
-- 
2.21.0.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39218): https://edk2.groups.io/g/devel/message/39218
Mute This Topic: https://groups.io/mt/31209232/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to