Hi,
On 08/12/19 05:07, Gao, Zhichao wrote:
> Change the comment of EfiResetSystem base on the
> UEFI spec 2.8
>
> Cc: Michael D Kinney
> Cc: Liming Gao
> Signed-off-by: Zhichao Gao
> ---
> MdePkg/Library/UefiRuntimeLib/RuntimeLib.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/MdePkg/Library/UefiRuntimeLib/RuntimeLib.c
> b/MdePkg/Library/UefiRuntimeLib/RuntimeLib.c
> index 933e0099ceed..28cbf493b665 100644
> --- a/MdePkg/Library/UefiRuntimeLib/RuntimeLib.c
> +++ b/MdePkg/Library/UefiRuntimeLib/RuntimeLib.c
> @@ -218,10 +218,10 @@ EfiGoneVirtual (
>@param DataSizeThe size, in bytes, of ResetData.
>@param ResetData For a ResetType of EfiResetCold, EfiResetWarm, or
> EfiResetShutdown the data buffer starts with a
>Null-terminated Unicode 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 then
> EFI_SUCCESS. This pointer must be a physical
> - address. For a ResetType of EfiRestUpdate the data
> buffer also starts with a Null-terminated string
> - that is followed by a physical VOID * to an
> EFI_CAPSULE_HEADER.
> + description that the caller may use to further
> indicate the reason for the system reset. This
> + pointer must be a physical address. 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.
>
> **/
> VOID
>
given that UefiRuntimeLib is a single-instance library class -- that is,
my understanding is that we never plan to introduce a different instance
for this class --, I would suggest fusing the following two patches into
one:
- [PATCH v4 14/15] MdePkg/UefiRuntimeLib.h: Change the comment
- [PATCH v4 15/15] MdePkg/UefiRuntimeLib: Change the comment
Also, can we please use a better (more specific) subject line?
Obviously I'm not an MdePkg maintainer (or even reviewer), so I defer to
Liming and Mike.
Thanks
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#45439): https://edk2.groups.io/g/devel/message/45439
Mute This Topic: https://groups.io/mt/32836909/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-