From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=fail (domain: intel.com, ip: , mailfrom: zhichao.gao@intel.com) Received: from mga04.intel.com (mga04.intel.com []) by groups.io with SMTP; Sun, 14 Apr 2019 20:07:47 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Apr 2019 20:07:47 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,351,1549958400"; d="scan'208";a="150867090" Received: from gaozhic-mobl.ccr.corp.intel.com ([10.239.198.173]) by orsmga002.jf.intel.com with ESMTP; 14 Apr 2019 20:07:45 -0700 From: "Gao, Zhichao" To: devel@edk2.groups.io Cc: Ard Biesheuvel , Leif Lindholm , Michael D Kinney , Liming Gao Subject: [edk2-platforms PATCH] Platform/ResetSystemLib: Add a new API ResetSystem Date: Mon, 15 Apr 2019 11:07:35 +0800 Message-Id: <20190415030739.14328-3-zhichao.gao@intel.com> X-Mailer: git-send-email 2.16.2.windows.1 In-Reply-To: <20190415030739.14328-1-zhichao.gao@intel.com> References: <20190415030739.14328-1-zhichao.gao@intel.com> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1460 Add a new API ResetSystem to this ResetSystemLib instance. It only adds the basic functions from ResetSystemRuntimeDxe. Lacking of this interface may cause link error, if some drivers use this new API and link to this library instance. Make the ResetPlatformSpecific's parameters same with the interface in Edk2 repo. Notes: This library API only provide a basic function of reset. If the consumers want full functions, they should use the instance in the MdeModulePkg and make sure the depex driver is dispatched. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Michael D Kinney Cc: Liming Gao Signed-off-by: Zhichao Gao --- .../Library/ResetSystemLib/ResetSystemLib.c | 44 +++++++++++++++++++ 1 file changed, 44 insertions(+) diff --git a/Platform/ARM/VExpressPkg/Library/ResetSystemLib/ResetSystemLib.c b/Platform/ARM/VExpressPkg/Library/ResetSystemLib/ResetSystemLib.c index d2bc4a88fa..c4f4eedc45 100644 --- a/Platform/ARM/VExpressPkg/Library/ResetSystemLib/ResetSystemLib.c +++ b/Platform/ARM/VExpressPkg/Library/ResetSystemLib/ResetSystemLib.c @@ -6,6 +6,7 @@ Copyright (c) 2008 - 2009, Apple Inc. All rights reserved.
Copyright (c) 2013, ARM Ltd. All rights reserved.
Copyright (c) 2017, Linaro Ltd. All rights reserved.
+ Copyright (c) 2019, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License @@ -109,3 +110,46 @@ ResetPlatformSpecific ( { ResetCold (); } + +/** + The ResetSystem function resets the entire platform. + + @param[in] ResetType The type of reset to perform. + @param[in] ResetStatus The status code for the reset. + @param[in] DataSize The size, in bytes, of ResetData. + @param[in] ResetData For a ResetType of EfiResetCold, EfiResetWarm, or 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. +**/ +VOID +EFIAPI +ResetSystem ( + IN EFI_RESET_TYPE ResetType, + IN EFI_STATUS ResetStatus, + IN UINTN DataSize, + IN VOID *ResetData OPTIONAL + ) +{ + switch (ResetType) { + case EfiResetWarm: + ResetWarm (); + break; + + case EfiResetCold: + ResetCold (); + break; + + case EfiResetShutdown: + ResetShutdown (); + return ; + + case EfiResetPlatformSpecific: + ResetPlatformSpecific (DataSize, ResetData); + return; + + default: + return ; + } +} -- 2.21.0.windows.1