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>
---
 MdeModulePkg/Universal/ResetSystemPei/ResetSystem.c | 5 +----
 MdeModulePkg/Universal/ResetSystemPei/ResetSystem.h | 5 +----
 2 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.c 
b/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.c
index 068e4e1130..0a60387373 100644
--- a/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.c
+++ b/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.c
@@ -265,10 +265,7 @@ InitializeResetSystem (
                                 EfiResetShutdown 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.
+                                indicate the reason for the system reset.
                                 For a ResetType of EfiResetPlatformSpecific 
the data buffer
                                 also starts with a Null-terminated string that 
is followed
                                 by an EFI_GUID that describes the specific 
type of reset to perform.
diff --git a/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.h 
b/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.h
index 6c96ddaa84..4d49a308af 100644
--- a/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.h
+++ b/MdeModulePkg/Universal/ResetSystemPei/ResetSystem.h
@@ -58,10 +58,7 @@ typedef struct {
                                 EfiResetShutdown 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.
+                                indicate the reason for the system reset.
                                 For a ResetType of EfiResetPlatformSpecific 
the data buffer
                                 also starts with a Null-terminated string that 
is followed
                                 by an EFI_GUID that describes the specific 
type of reset to perform.
-- 
2.21.0.windows.1


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

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

Reply via email to