Reviewed-by: Chao Zhang <chao.b.zh...@intel.com>

-----Original Message-----
From: Gao, Zhichao 
Sent: Wednesday, April 17, 2019 2:27 PM
To: devel@edk2.groups.io
Cc: Zhang, Chao B <chao.b.zh...@intel.com>; Yao, Jiewen <jiewen....@intel.com>; 
Gao, Liming <liming....@intel.com>; Sean Brogan <sean.bro...@microsoft.com>; 
Michael Turner <michael.tur...@microsoft.com>; Bret Barkelew 
<bret.barke...@microsoft.com>
Subject: [PATCH V2 05/25] SecurityPkg/Tcg2Dxe: Change comments of 
ShutdownTpmOnReset

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: Chao Zhang <chao.b.zh...@intel.com>
Cc: Jiewen Yao <jiewen....@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>
---
 SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c 
b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
index b3e21f0947..a2729457b7 100644
--- a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
+++ b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
@@ -1,7 +1,7 @@
 /** @file
   This module implements Tcg2 Protocol.
 
-Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
 (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
 SPDX-License-Identifier: BSD-2-Clause-Patent
 
@@ -2419,10 +2419,7 @@ OnExitBootServicesFailed (
                                 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 (#39297): https://edk2.groups.io/g/devel/message/39297
Mute This Topic: https://groups.io/mt/31209224/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to