From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: hao.a.wu@intel.com) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by groups.io with SMTP; Tue, 16 Apr 2019 22:24:34 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Apr 2019 22:24:33 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,360,1549958400"; d="scan'208";a="143396391" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga003.jf.intel.com with ESMTP; 16 Apr 2019 22:24:33 -0700 Received: from FMSMSX110.amr.corp.intel.com (10.18.116.10) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.408.0; Tue, 16 Apr 2019 22:24:32 -0700 Received: from shsmsx102.ccr.corp.intel.com (10.239.4.154) by fmsmsx110.amr.corp.intel.com (10.18.116.10) with Microsoft SMTP Server (TLS) id 14.3.408.0; Tue, 16 Apr 2019 22:24:32 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.92]) by shsmsx102.ccr.corp.intel.com ([169.254.2.206]) with mapi id 14.03.0415.000; Wed, 17 Apr 2019 13:24:30 +0800 From: "Wu, Hao A" To: "Gao, Zhichao" , "devel@edk2.groups.io" CC: Bret Barkelew , "Wang, Jian J" , "Ni, Ray" , "Zeng, Star" , "Gao, Liming" , Sean Brogan , Michael Turner Subject: Re: [PATCH 24/25] MdeModulePkg/ResetUtilityLib: Add a new API ResetSystemWithSubtype Thread-Topic: [PATCH 24/25] MdeModulePkg/ResetUtilityLib: Add a new API ResetSystemWithSubtype Thread-Index: AQHU8zhkLyHbAs/JWku9fxYCdujmt6Y/06MA Date: Wed, 17 Apr 2019 05:24:29 +0000 Message-ID: References: <20190415030611.12464-1-zhichao.gao@intel.com> <20190415030611.12464-25-zhichao.gao@intel.com> In-Reply-To: <20190415030611.12464-25-zhichao.gao@intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: hao.a.wu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: Gao, Zhichao > Sent: Monday, April 15, 2019 11:06 AM > To: devel@edk2.groups.io > Cc: Bret Barkelew; Wang, Jian J; Wu, Hao A; Ni, Ray; Zeng, Star; Gao, Lim= ing; > Sean Brogan; Michael Turner > Subject: [PATCH 24/25] MdeModulePkg/ResetUtilityLib: Add a new API > ResetSystemWithSubtype >=20 > From: Bret Barkelew >=20 > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1458 >=20 > Add a new API ResetSystemWithSubtype's prototype declaration in > header file. Also add the required data type header file. >=20 > Cc: Jian J Wang > Cc: Hao Wu > Cc: Ray Ni > Cc: Star Zeng > Cc: Liming Gao > Cc: Sean Brogan > Cc: Michael Turner > Cc: Bret Barkelew > Signed-off-by: Zhichao Gao > --- > .../Include/Library/ResetUtilityLib.h | 22 +++++++++++++++++++ > 1 file changed, 22 insertions(+) >=20 > diff --git a/MdeModulePkg/Include/Library/ResetUtilityLib.h > b/MdeModulePkg/Include/Library/ResetUtilityLib.h > index ca310cdb01..9bb4adc1ad 100644 > --- a/MdeModulePkg/Include/Library/ResetUtilityLib.h > +++ b/MdeModulePkg/Include/Library/ResetUtilityLib.h > @@ -10,6 +10,28 @@ > #ifndef _RESET_UTILITY_LIB_H_ > #define _RESET_UTILITY_LIB_H_ >=20 > +#include > + > +/** > + This is a shorthand helper function to reset with a subtype so that > + the caller doesn't have to bother with a function that has half a doze= n > + parameters. > + > + This will generate a reset with status EFI_SUCCESS, a NULL string, and > + no custom data. The subtype will be formatted in such a way that it ca= n be > + picked up by notification registrations and custom handlers. I found that the function description is almost the same with the existing API 'ResetPlatformSpecificGuid' (except lacking the 3rd paragraph starting with 'NOTE:', which I think should be applied to 'ResetSystemWithSubtype' as well). Seems to me that 'ResetSystemWithSubtype' is more generic. Could you help to refine one of their descriptions to reflect their difference? Best Regards, Hao Wu > + > + @param[in] ResetType The default EFI_RESET_TYPE of the reset. > + @param[in] ResetSubtype GUID pointer for the reset subtype to be use= d. > + > +**/ > +VOID > +EFIAPI > +ResetSystemWithSubtype ( > + IN EFI_RESET_TYPE ResetType, > + IN CONST GUID *ResetSubtype > + ); > + > /** > This is a shorthand helper function to reset with a subtype so that > the caller doesn't have to bother with a function that has half a doze= n > -- > 2.21.0.windows.1