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. So change the comments of RuntimeServiceResetSystem.
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/ResetSystemRuntimeDxe/ResetSystem.c | 5 +----
MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.h | 5 +----
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c
b/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c
index 399104ca8e..77aedcba6e 100644
--- a/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c
+++ b/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c
@@ -210,10 +210,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/ResetSystemRuntimeDxe/ResetSystem.h
b/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.h
index 25ae5dac27..87d9c572aa 100644
--- a/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.h
+++ b/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.h
@@ -80,10 +80,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.
--
2.21.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#39033): https://edk2.groups.io/g/devel/message/39033
Mute This Topic: https://groups.io/mt/31137347/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
