From: "Yao, Jiewen" <jiewen.yao@intel.com>
To: Pete Batard <pete@akeo.ie>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [PATCH 1/1] MdeModulePkg/EbcDxe: add EBC Debugger configuration application
Date: Thu, 24 Nov 2016 12:40:46 +0000 [thread overview]
Message-ID: <74D8A39837DF1E4DA445A8C0B3885C50386DA4D5@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <20161124112956.3960-2-pete@akeo.ie>
Thank you Peter. I like this porting work.
One comment is that we have special restriction on package dependency.
-- The MdeModulePkg should not depend upon ShellPkg.
-- The MdeModulePkg can only depend upon MdePkg.
I know you want to parse the shell argument, such as Argc and Argv.
If an application has ShellPkg dependency issue, the recommended way is to use EFI_SHELL_PARAMETERS_PROTOCOL, which is defined in UEFI SHELL 2.x specification.
-- It is located in MdePkg\Include\Protocol\ShellParameters.h
An application may use ShellLib, if and only if it can depend upon ShellPkg, such as a platform Package, or ShellPkg itself.
You can refer to MdeModulePkg\Application\CapsuleApp\AppSupport.c, GetArg() to see how to get Argv and Argc via EFI_SHELL_PARAMETERS_PROTOCOL. :-)
Other part looks good to me.
Thank you
Yao Jiewen
> -----Original Message-----
> From: Pete Batard [mailto:pete@akeo.ie]
> Sent: Thursday, November 24, 2016 7:30 PM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Pete Batard <pete@akeo.ie>
> Subject: [PATCH 1/1] MdeModulePkg/EbcDxe: add EBC Debugger
> configuration application
>
> * Introduce a generic Debugger Configuration protocol.
> * Add private configuration data in the EBC Debugger and make it
> register the Debugger Configuration protocol on initialization.
> * Add a shell application that uses the protocol above to access
> the private data in order to configure the EBC debugger.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Pete Batard <pete@akeo.ie>
> ---
> .../Include/Protocol/DebuggerConfiguration.h | 32 ++++
> MdeModulePkg/MdeModulePkg.dec | 4 +
> MdeModulePkg/MdeModulePkg.dsc | 4 +
> MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf | 1 +
> .../EbcDxe/EbcDebugger/EbcDebuggerConfig.c | 207
> +++++++++++++++++++++
> MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 4 +
> .../Universal/EbcDxe/EbcDebugger/EdbCommon.h | 3 +
> .../Universal/EbcDxe/EbcDebugger/EdbHook.c | 11 ++
> .../Universal/EbcDxe/EbcDebuggerConfig.inf | 57 ++++++
> .../Universal/EbcDxe/EbcDebuggerConfig.uni | 18 ++
> .../Universal/EbcDxe/EbcDebuggerConfigExtra.uni | 17 ++
> 11 files changed, 358 insertions(+)
> create mode 100644
> MdeModulePkg/Include/Protocol/DebuggerConfiguration.h
> create mode 100644
> MdeModulePkg/Universal/EbcDxe/EbcDebugger/EbcDebuggerConfig.c
> create mode 100644
> MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf
> create mode 100644
> MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.uni
> create mode 100644
> MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfigExtra.uni
>
> diff --git a/MdeModulePkg/Include/Protocol/DebuggerConfiguration.h
> b/MdeModulePkg/Include/Protocol/DebuggerConfiguration.h
> new file mode 100644
> index 000000000000..eaaa6b925fad
> --- /dev/null
> +++ b/MdeModulePkg/Include/Protocol/DebuggerConfiguration.h
> @@ -0,0 +1,32 @@
> +/** @file
> + EBC Debugger configuration protocol.
> +
> + Copyright (c) 2007-2016, Intel Corporation. All rights reserved.<BR>
> +
> + This program and the accompanying materials
> + are licensed and made available under the terms and conditions
> + of the BSD License which accompanies this distribution. The
> + full text of the license may be found at
> + http://opensource.org/licenses/bsd-license.php
> +
> + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> +
> +**/
> +
> +#ifndef __EFI_DEBUGGER_CONFIGURATION_H__
> +#define __EFI_DEBUGGER_CONFIGURATION_H__
> +
> +#define EFI_DEBUGGER_CONFIGURATION_PROTOCOL_GUID \
> + { 0x577d959c, 0xe967, 0x4546, 0x86, 0x20, 0xc7, 0x78, 0xfa, 0xe5, 0xda,
> 0x5 }
> +
> +#define EFI_DEBUGGER_CONFIGURATION_VERSION 0x00000001
> +
> +typedef struct _EFI_DEBUGGER_CONFIGURATION_PROTOCOL {
> + UINT32 DebuggerConfigurationRevision;
> + VOID *DebuggerPrivateData;
> +} EFI_DEBUGGER_CONFIGURATION_PROTOCOL;
> +
> +extern EFI_GUID gEfiDebuggerConfigurationProtocolGuid;
> +
> +#endif
> diff --git a/MdeModulePkg/MdeModulePkg.dec
> b/MdeModulePkg/MdeModulePkg.dec
> index 22898e3f7487..08c3e40e930c 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -420,6 +420,10 @@ [Protocols]
> # Include/Protocol/GenericMemoryTest.h
> gEfiGenericMemTestProtocolGuid = { 0x309DE7F1, 0x7F5E, 0x4ACE,
> { 0xB4, 0x9C, 0x53, 0x1B, 0xE5, 0xAA, 0x95, 0xEF }}
>
> + ## This protocol defines the Debugger Configuration protocol.
> + # Include/Protocol/DebuggerConfiguration.h
> + gEfiDebuggerConfigurationProtocolGuid = { 0x577d959c, 0xe967, 0x4546,
> { 0x86, 0x20, 0xc7, 0x78, 0xfa, 0xe5, 0xda, 0x05 }}
> +
> ## Include/Protocol/Dpc.h
> gEfiDpcProtocolGuid = {0x480f8ae9, 0xc46, 0x4aa9,
> { 0xbc, 0x89, 0xdb, 0x9f, 0xba, 0x61, 0x98, 0x6 }}
>
> diff --git a/MdeModulePkg/MdeModulePkg.dsc
> b/MdeModulePkg/MdeModulePkg.dsc
> index bfecb3078737..60202d65600c 100644
> --- a/MdeModulePkg/MdeModulePkg.dsc
> +++ b/MdeModulePkg/MdeModulePkg.dsc
> @@ -163,6 +163,9 @@ [LibraryClasses.common.UEFI_APPLICATION]
> HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf
>
> MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocationLib/UefiMem
> oryAllocationLib.inf
> DebugLib|MdePkg/Library/UefiDebugLibStdErr/UefiDebugLibStdErr.inf
> + FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf
> + ShellLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf
> +
> ShellCEntryLib|ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLib.inf
>
> [LibraryClasses.ARM, LibraryClasses.AARCH64]
> ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
> @@ -432,6 +435,7 @@ [Components.IA32, Components.X64,
> Components.IPF, Components.AARCH64]
> MdeModulePkg/Universal/DebugSupportDxe/DebugSupportDxe.inf
> MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
> MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf
> + MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf
>
> [Components.IA32, Components.X64, Components.Ebc]
> MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf
> {
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf
> index 7e0166672b99..ce413c0b25f6 100644
> --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger.inf
> @@ -102,6 +102,7 @@ [LibraryClasses]
> [Protocols]
> gEfiDebugSupportProtocolGuid ## PRODUCES
> gEfiEbcProtocolGuid ## PRODUCES
> + gEfiDebuggerConfigurationProtocolGuid ## PRODUCES
> gEfiEbcVmTestProtocolGuid ##
> SOMETIMES_PRODUCES
> gEfiEbcSimpleDebuggerProtocolGuid ##
> SOMETIMES_CONSUMES
> gEfiPciRootBridgeIoProtocolGuid ##
> SOMETIMES_CONSUMES
> diff --git
> a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EbcDebuggerConfig.c
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EbcDebuggerConfig.c
> new file mode 100644
> index 000000000000..8e9fca9d3ef8
> --- /dev/null
> +++
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EbcDebuggerConfig.c
> @@ -0,0 +1,207 @@
> +/** @file
> + Configuration application for the EBC Debugger.
> +
> + Copyright (c) 2007 - 2016, Intel Corporation. All rights reserved.<BR>
> + This program and the accompanying materials
> + are licensed and made available under the terms and conditions of the
> BSD License
> + which accompanies this distribution. The full text of the license may be
> found at
> + http://opensource.org/licenses/bsd-license.php
> +
> + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> +
> +**/
> +
> +#include <Uefi.h>
> +#include <Protocol/Shell.h>
> +#include <Library/ShellCEntryLib.h>
> +
> +#include "EdbCommon.h"
> +#include "EdbSupport.h"
> +
> +VOID
> +PrintUsage (
> + VOID
> + )
> +{
> + Print (
> + L"EbcDebuggerConfig Version 0.2\n"
> + L"Copyright (C) Intel Corp 2007-2016. All rights reserved.\n"
> + L"\n"
> + L"Configure EbcDebugger in EFI Shell Environment.\n"
> + L"\n"
> + L"usage: EdbCfg <Command>\n"
> + L" CommandList:\n"
> + L" BO[C|CX|R|E|T|K] <ON|OFF> - Enable/Disable
> BOC/BOCX/BOR/BOE/BOT/BOK.\n"
> +// L" SHOWINFO - Show Debugger
> Information.\n"
> + L"\n"
> + );
> + return;
> +}
> +
> +VOID
> +EdbShowInfo (
> + EFI_DEBUGGER_CONFIGURATION_PROTOCOL *DebuggerConfiguration
> + )
> +{
> + Print (L"Not supported!\n");
> + return ;
> +}
> +
> +VOID
> +EdbConfigBreak (
> + EFI_DEBUGGER_CONFIGURATION_PROTOCOL *DebuggerConfiguration,
> + CHAR16 *Command,
> + CHAR16 *CommandArg
> + )
> +{
> + EFI_DEBUGGER_PRIVATE_DATA *DebuggerPrivate;
> +
> + DebuggerPrivate = (EFI_DEBUGGER_PRIVATE_DATA
> *)DebuggerConfiguration->DebuggerPrivateData;
> +
> + if (StriCmp (Command, L"BOC") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOC)
> == EFI_DEBUG_FLAG_EBC_BOC) {
> + Print (L"BOC on\n");
> + } else {
> + Print (L"BOC off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOC;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOC;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + } else if (StriCmp (Command, L"BOCX") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOCX)
> == EFI_DEBUG_FLAG_EBC_BOCX) {
> + Print (L"BOCX on\n");
> + } else {
> + Print (L"BOCX off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOCX;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOCX;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + } else if (StriCmp (Command, L"BOR") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOR)
> == EFI_DEBUG_FLAG_EBC_BOR) {
> + Print (L"BOR on\n");
> + } else {
> + Print (L"BOR off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOR;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOR;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + } else if (StriCmp (Command, L"BOE") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOE)
> == EFI_DEBUG_FLAG_EBC_BOE) {
> + Print (L"BOE on\n");
> + } else {
> + Print (L"BOE off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOE;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOE;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + } else if (StriCmp (Command, L"BOT") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOT)
> == EFI_DEBUG_FLAG_EBC_BOT) {
> + Print (L"BOT on\n");
> + } else {
> + Print (L"BOT off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOT;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOT;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + } else if (StriCmp (Command, L"BOK") == 0) {
> + if (CommandArg == NULL) {
> + if ((DebuggerPrivate->FeatureFlags & EFI_DEBUG_FLAG_EBC_BOK)
> == EFI_DEBUG_FLAG_EBC_BOK) {
> + Print (L"BOK on\n");
> + } else {
> + Print (L"BOK off\n");
> + }
> + } else if (StriCmp (CommandArg, L"ON") == 0) {
> + DebuggerPrivate->FeatureFlags |= EFI_DEBUG_FLAG_EBC_BOK;
> + } else if (StriCmp (CommandArg, L"OFF") == 0) {
> + DebuggerPrivate->FeatureFlags &=
> ~EFI_DEBUG_FLAG_EBC_B_BOK;
> + } else {
> + Print (L"Invalid parameter\n");
> + }
> + }
> + return ;
> +}
> +
> +INTN
> +EFIAPI
> +ShellAppMain (
> + IN UINTN Argc,
> + IN CHAR16 **Argv
> + )
> +{
> + EFI_DEBUGGER_CONFIGURATION_PROTOCOL *DebuggerConfiguration;
> + EFI_STATUS Status;
> +
> + if (Argc < 2) {
> + PrintUsage ();
> + return SHELL_INVALID_PARAMETER;
> + }
> +
> + if (Argc == 2) {
> + if ((StrCmp (Argv[1], L"/?") == 0) ||
> + (StrCmp (Argv[1], L"-?") == 0) ||
> + (StrCmp (Argv[1], L"-h") == 0) ||
> + (StrCmp (Argv[1], L"-H") == 0) ) {
> + PrintUsage ();
> + return SHELL_SUCCESS;
> + }
> + }
> +
> + Status = gBS->LocateProtocol (
> + &gEfiDebuggerConfigurationProtocolGuid,
> + NULL,
> + &DebuggerConfiguration
> + );
> + if (EFI_ERROR(Status)) {
> + Print (L"Error: DebuggerConfiguration Protocol not found!\n");
> + return SHELL_SUCCESS;
> + }
> +
> + if (StriCmp (Argv[1], L"SHOWINFO") == 0) {
> + EdbShowInfo (DebuggerConfiguration);
> + return SHELL_SUCCESS;
> + }
> +
> + if (((Argc == 2) || (Argc == 3)) &&
> + ((StriCmp (Argv[1], L"BOC") == 0) ||
> + (StriCmp (Argv[1], L"BOCX") == 0) ||
> + (StriCmp (Argv[1], L"BOR") == 0) ||
> + (StriCmp (Argv[1], L"BOE") == 0) ||
> + (StriCmp (Argv[1], L"BOT") == 0) ||
> + (StriCmp (Argv[1], L"BOK") == 0))) {
> + if (Argc == 3) {
> + EdbConfigBreak (DebuggerConfiguration, Argv[1], Argv[2]);
> + } else {
> + EdbConfigBreak (DebuggerConfiguration, Argv[1], NULL);
> + }
> + return SHELL_SUCCESS;
> + }
> +
> + Print (L"Error: Invalid Command!\n");
> + return SHELL_INVALID_PARAMETER;
> +}
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c
> index e0b85c70eff5..189b88357758 100644
> --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c
> @@ -27,6 +27,10 @@ EFI_DEBUGGER_PRIVATE_DATA mDebuggerPrivate = {
> EBC_DEBUGGER_MINOR_VERSION, //
> EfiDebuggerRevision
> (VM_MAJOR_VERSION << 16) |
> VM_MINOR_VERSION, // EbcVmRevision
> + {
> + EFI_DEBUGGER_CONFIGURATION_VERSION,
> + &mDebuggerPrivate,
> + }, //
> DebuggerConfiguration
> NULL, //
> DebugImageInfoTableHeader
> NULL, // Vol
> NULL, // PciRootBridgeIo
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCommon.h
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCommon.h
> index 45634af9ce39..616c369253b8 100644
> --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCommon.h
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCommon.h
> @@ -30,9 +30,11 @@ Abstract:
> #include <Library/PrintLib.h>
> #include <Library/UefiBootServicesTableLib.h>
> #include <Protocol/Ebc.h>
> +#include <Protocol/EbcVmTest.h>
> #include <Protocol/DebugSupport.h>
> #include <Protocol/PciRootBridgeIo.h>
> #include <Protocol/SimpleFileSystem.h>
> +#include <Protocol/DebuggerConfiguration.h>
> #include <Guid/FileInfo.h>
> #include <Guid/FileSystemInfo.h>
> #include <Guid/FileSystemVolumeLabelInfo.h>
> @@ -227,6 +229,7 @@ typedef struct _EFI_DEBUGGER_PRIVATE_DATA {
> EFI_INSTRUCTION_SET_ARCHITECTURE Isa;
> UINT32
> EfiDebuggerRevision;
> UINT32 EbcVmRevision;
> + EFI_DEBUGGER_CONFIGURATION_PROTOCOL
> DebuggerConfiguration;
> EFI_DEBUG_IMAGE_INFO_TABLE_HEADER
> *DebugImageInfoTableHeader;
> EFI_SIMPLE_FILE_SYSTEM_PROTOCOL *Vol;
> EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL
> *PciRootBridgeIo;
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c
> index 9196adb74ba3..3bfee927e227 100644
> --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c
> @@ -448,6 +448,17 @@ Returns:
> );
>
> //
> + // Register Debugger Configuration Protocol, for config in shell
> + //
> + Status = gBS->InstallProtocolInterface (
> + &Handle,
> + &gEfiDebuggerConfigurationProtocolGuid,
> + EFI_NATIVE_INTERFACE,
> + &mDebuggerPrivate.DebuggerConfiguration
> + );
> +
> + //
> + //
> // Create break event
> //
> Status = gBS->CreateEvent (
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf
> b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf
> new file mode 100644
> index 000000000000..db21f301d87c
> --- /dev/null
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.inf
> @@ -0,0 +1,57 @@
> +## @file
> +# EBC Debugger configuration application.
> +#
> +# Copyright (c) 2007 - 2016, Intel Corporation. All rights reserved.<BR>
> +#
> +# This program and the accompanying materials
> +# are licensed and made available under the terms and conditions of the
> BSD License
> +# which accompanies this distribution. The full text of the license may be
> found at
> +# http://opensource.org/licenses/bsd-license.php
> +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS
> IS" BASIS,
> +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> +#
> +#
> +##
> +
> +[Defines]
> + INF_VERSION = 0x00010005
> + BASE_NAME = EdbCfg
> + MODULE_UNI_FILE = EbcDebuggerConfig.uni
> + FILE_GUID =
> 8CFC5233-23C6-49e3-8A2D-7E581AB305BA
> + MODULE_TYPE = UEFI_APPLICATION
> + VERSION_STRING = 1.0
> + ENTRY_POINT = ShellCEntryLib
> +
> +#
> +# The following information is for reference only and not required by the
> build tools.
> +#
> +# VALID_ARCHITECTURES = IA32 X64 IPF AARCH64
> +#
> +
> +[Sources]
> + EbcDebugger/EbcDebuggerConfig.c
> + EbcDebugger/EdbCommon.h
> + EbcDebugger/EdbSupportString.c
> + EbcDebugger/EdbSupport.h
> +
> +[Packages]
> + AppPkg/AppPkg.dec
> + MdePkg/MdePkg.dec
> + ShellPkg/ShellPkg.dec
> + MdeModulePkg/MdeModulePkg.dec
> +
> +[LibraryClasses]
> + UefiLib
> + BaseLib
> + DebugLib
> + ShellLib
> + ShellCEntryLib
> +
> +[Protocols]
> + gEfiDebuggerConfigurationProtocolGuid ## CONSUMES
> +
> +[Depex]
> + TRUE
> +
> +[UserExtensions.TianoCore."ExtraFiles"]
> + EbcDebuggerConfigExtra.uni
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.uni
> b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.uni
> new file mode 100644
> index 000000000000..5201b9358346
> --- /dev/null
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfig.uni
> @@ -0,0 +1,18 @@
> +// /** @file
> +// EBC Debugger configuration application.
> +//
> +// Copyright (c) 2016, Intel Corporation. All rights reserved.<BR>
> +//
> +// This program and the accompanying materials
> +// are licensed and made available under the terms and conditions of the
> BSD License
> +// which accompanies this distribution. The full text of the license may be
> found at
> +// http://opensource.org/licenses/bsd-license.php
> +// THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> +// WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> +//
> +// **/
> +
> +
> +#string STR_MODULE_ABSTRACT #language en-US "EBC
> Debugger configuration application"
> +
> +#string STR_MODULE_DESCRIPTION #language en-US "This
> application allows configuring the EBC Debugger."
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfigExtra.uni
> b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfigExtra.uni
> new file mode 100644
> index 000000000000..b3ad472c4c44
> --- /dev/null
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebuggerConfigExtra.uni
> @@ -0,0 +1,17 @@
> +// /** @file
> +// EBC Debugger configuration application.
> +//
> +// Copyright (c) 2016, Intel Corporation. All rights reserved.<BR>
> +//
> +// This program and the accompanying materials
> +// are licensed and made available under the terms and conditions of the
> BSD License
> +// which accompanies this distribution. The full text of the license may be
> found at
> +// http://opensource.org/licenses/bsd-license.php
> +// THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> +// WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> +//
> +// **/
> +
> +#string STR_PROPERTIES_MODULE_NAME
> +#language en-US
> +"EBC Debugger Config"
> --
> 2.9.3.windows.2
next prev parent reply other threads:[~2016-11-24 12:40 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-24 11:29 [PATCH 0/1] MdeModulePkg/EbcDxe: add EBC Debugger configuration application Pete Batard
2016-11-24 11:29 ` [PATCH 1/1] " Pete Batard
2016-11-24 12:40 ` Yao, Jiewen [this message]
2016-11-24 13:06 ` Pete Batard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=74D8A39837DF1E4DA445A8C0B3885C50386DA4D5@shsmsx102.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox