> -----Original Message----- > From: Gao, Zhichao > Sent: Wednesday, April 17, 2019 2:27 PM > To: devel@edk2.groups.io > Cc: Bret Barkelew; Wang, Jian J; Wu, Hao A; Ni, Ray; Zeng, Star; Gao, Liming; > Sean Brogan; Michael Turner > Subject: [PATCH V2 24/25] MdeModulePkg/ResetUtilityLib: Add a new API > ResetSystemWithSubtype > > From: Bret Barkelew <bret.barke...@microsoft.com> > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1458 > > Add a new API ResetSystemWithSubtype's prototype declaration in > header file. Also add the required data type header file. > > 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> > --- > .../Include/Library/ResetUtilityLib.h | 34 ++++++++++++++++--- > 1 file changed, 30 insertions(+), 4 deletions(-) > > diff --git a/MdeModulePkg/Include/Library/ResetUtilityLib.h > b/MdeModulePkg/Include/Library/ResetUtilityLib.h > index ca310cdb01..656703e782 100644 > --- a/MdeModulePkg/Include/Library/ResetUtilityLib.h > +++ b/MdeModulePkg/Include/Library/ResetUtilityLib.h > @@ -1,7 +1,7 @@ > /** @file > This header describes various helper functions for resetting the system. > > - Copyright (c) 2017 Intel Corporation. All rights reserved.<BR> > + Copyright (c) 2017 - 2019 Intel Corporation. All rights reserved.<BR> > Copyright (c) 2016 Microsoft Corporation. All rights reserved.<BR> > > SPDX-License-Identifier: BSD-2-Clause-Patent > @@ -10,10 +10,36 @@ > #ifndef _RESET_UTILITY_LIB_H_ > #define _RESET_UTILITY_LIB_H_ > > +#include <Uefi/UefiMultiPhase.h> > + > +/** > + This is a shorthand helper function to reset with reset type and a subtype > + so that the caller doesn't have to bother with a function that has half > + a dozen 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 can be > + picked up by notification registrations and custom handlers. > + > + NOTE: This call will fail if the architectural ResetSystem underpinnings > + are not initialized. For DXE, you can add gEfiResetArchProtocolGuid > + to your DEPEX. > + > + @param[in] ResetType The default EFI_RESET_TYPE of the reset. > + @param[in] ResetSubtype GUID pointer for the reset subtype to be used. > + > +**/ > +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 dozen > - parameters. > + This is a shorthand helper function to reset with the reset type > + 'EfiResetPlatformSpecific' and a subtype so that the caller doesn't > + have to bother with a function that has half a dozen parameters.
Thanks for addressing the comments. Reviewed-by: Hao Wu <hao.a...@intel.com> Best Regards, Hao Wu > > 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 can be > -- > 2.21.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39232): https://edk2.groups.io/g/devel/message/39232 Mute This Topic: https://groups.io/mt/31209252/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-