From: "Jin, Eric" <eric.jin@intel.com>
To: Supreeth Venkatesh <supreeth.venkatesh@arm.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Jin, Eric" <eric.jin@intel.com>
Subject: Re: [edk2-test][PATCH v1 30/30] UEFI/UEFI.dec: Add missing protocol GUIDs in declaration file.
Date: Fri, 29 Mar 2019 09:03:31 +0000 [thread overview]
Message-ID: <DA72DC7456565B47808A57108259571F6376F721@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <20190328231222.4717-30-supreeth.venkatesh@arm.com>
Hi Supreeth,
Thanks a lot to fix this build issue at first.
Comments for this series of patches are replied below.
Patch 1/2 :
1. Could the two patches can be re-organized? For example, merge the change of *.dsc in patch 2 to patch 1 to make it clear.
2. The change of CommonGenFramework.sh in patch 2 will remove the ebc from binary package.
Patch 24 :
1. Please don't forget change the copyright part
2. Do you meet the similar issue that "UEFI_CONFIG_LANG" : macro redefinition while build?
Other Patches :
1. Please remove "COMPONENT_TYPE = BS_DRIVER"
With that
Reviewed-by: Eric Jin <eric.jin@intel.com>
Best Regards
Eric
-----Original Message-----
From: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
Sent: Friday, March 29, 2019 7:12 AM
To: edk2-devel@lists.01.org
Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>; Jin, Eric <eric.jin@intel.com>
Subject: [edk2-test][PATCH v1 30/30] UEFI/UEFI.dec: Add missing protocol GUIDs in declaration file.
Fix compilation issues in inf files when compiled against edk2 stable tag edk2-stable201903.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
---
uefi-sct/SctPkg/UEFI/UEFI.dec | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/uefi-sct/SctPkg/UEFI/UEFI.dec b/uefi-sct/SctPkg/UEFI/UEFI.dec index 8495a4e1..bdf3323f 100644
--- a/uefi-sct/SctPkg/UEFI/UEFI.dec
+++ b/uefi-sct/SctPkg/UEFI/UEFI.dec
@@ -1,7 +1,7 @@
## @file
#
# Copyright 2004 - 2017 Unified EFI, Inc.<BR> -# Copyright (c) 2014 - 2018, ARM Limited. All rights reserved.
+# Copyright (c) 2014 - 2019, ARM Limited. All rights reserved.
# Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR> # (C) Copyright 2017 Hewlett Packard Enterprise Development LP<BR> # @@ -119,14 +119,22 @@
gBlackBoxEfiHIIStringProtocolGuid = {0xfd96974, 0x23aa, 0x4cdc, { 0xb9, 0xcb, 0x98, 0xd1, 0x77, 0x50, 0x32, 0x2a }}
gBlackBoxEfiHIIImageProtocolGuid = { 0x31a6406a, 0x6bdf, 0x4e46, {0xb2, 0xa2, 0xeb, 0xaa, 0x89, 0xc4, 0x9, 0x20 }}
gBlackBoxEfiHIIImageExProtocolGuid = { 0x1a1241e6, 0x8f19, 0x41a9, { 0xbc, 0xe, 0xe8, 0xef,0x39, 0xe0, 0x65, 0x46 }}
+ gBlackBoxEfiHIIDatabaseProtocolGuid = { 0xef9fc172, 0xa1b2, 0x4693, {
+ 0xb3, 0x27, 0x6d, 0x32, 0xfc, 0x41, 0x60, 0x42 }}
+ gBlackBoxEfiHIIPackageListProtocolGuid = { 0x6a1ee763, 0xd47a, 0x43b4,
+ { 0xaa, 0xbe, 0xef, 0x1d, 0xe2, 0xab, 0x56, 0xfc }}
+ gBlackBoxEfiHIIStringProtocolGuid = {0xfd96974, 0x23aa, 0x4cdc, {
+ 0xb9, 0xcb, 0x98, 0xd1, 0x77, 0x50, 0x32, 0x2a }}
+ gBlackBoxEfiHIIConfigAccessProtocolGuid = { 0x330d4706, 0xf2a0,
+ 0x4e4f, { 0xa3, 0x69, 0xb6, 0x6f, 0xa8, 0xd5, 0x43, 0x85 }}
+ gBlackBoxEfiHIIConfigRoutingProtocolGuid = { 0x587e72d7, 0xcc50,
+ 0x4f79, { 0x82, 0x09, 0xca, 0x29, 0x1f, 0xc1, 0xa1, 0x0f }}
gBlackBoxEfiHIIFontProtocolGuid = { 0xe9ca4775, 0x8657, 0x47fc, { 0x97, 0xe7, 0x7e, 0xd6, 0x5a, 0x8, 0x43, 0x24 }}
gBlackBoxEfiHIIFontExProtocolGuid = { 0x849e6875, 0xdb35, 0x4df8, { 0xb4, 0x1e, 0xc8, 0xf3, 0x37, 0x18, 0x7, 0x3f }}
gBlackBoxEfiHttpProtocolGuid = { 0x7A59B29B, 0x910B, 0x4171, { 0x82, 0x42, 0xA8, 0x5A, 0x0D, 0xF2, 0x5B, 0x5B }}
gBlackBoxEfiHttpServiceBindingProtocolGuid = { 0xbdc8e6af, 0xd9bc, 0x4379, { 0xa7, 0x2a, 0xe0, 0xc4, 0xe7, 0x5d, 0xae, 0x1c }}
gBlackBoxEfiIp4ServiceBindingProtocolGuid = { 0xc51711e7, 0xb4bf, 0x404a, {0xbf, 0xb8, 0x0a, 0x04, 0x8e, 0xf1, 0xff, 0xe4 } }
gBlackBoxEfiIp4ProtocolGuid = { 0x41d94cd2, 0x35b6, 0x455a, {0x82, 0x58, 0xd4, 0xe5, 0x13, 0x34, 0xaa, 0xdd } }
+ gBlackBoxEfiIp4ConfigProtocolGuid = { 0x3B95AA31, 0x3793, 0x434B,
+ {0x86, 0x67, 0xC8, 0x07, 0x08, 0x92, 0xE0, 0x5E } }
+ gBlackBoxEfiIp4Config2ProtocolGuid = { 0x5b446ed1, 0xe30b, 0x4faa,
+ {0x87, 0x1a, 0x36, 0x54, 0xec, 0xa3, 0x60, 0x80 } }
gBlackBoxEfiIp6ServiceBindingProtocolGuid = { 0xec835dd3, 0xfe0f, 0x617b, {0xa6, 0x21, 0xb3, 0x50, 0xc3, 0xe1, 0x33, 0x88 } }
gBlackBoxEfiIp6ProtocolGuid = { 0xec835dd3, 0xfe0f, 0x617b, {0xa6, 0x21, 0xb3, 0x50, 0xc3, 0xe1, 0x33, 0x88 } }
+ gBlackBoxEfiIp6ConfigProtocolGuid = { 0x937fe521, 0x95ae, 0x4d1a,
+ {0x89, 0x29, 0x48, 0xbc, 0xd9, 0x0a, 0xd3, 0x1a } }
gBlackBoxEfiIPsec2ProtocolGuid = {0xa3979e64, 0xace8, 0x4ddc, { 0xbc, 0x7, 0x4d, 0x66, 0xb8, 0xfd, 0x9, 0x77 }}
gBlackBoxEfiIPsecConfigProtocolGuid = { 0xce5e5929, 0xc7a3, 0x4602, { 0xad, 0x9e, 0xc9, 0xda, 0xf9, 0x4e, 0xbf, 0xcf }}
gBlackBoxEfiIScsiInitiatorNameProtocolGuid = { 0x59324945, 0xec44, 0x4c0d, {0xb1, 0xcd, 0x9d, 0xb1, 0x39, 0xdf, 0x7, 0xc }}
--
2.17.1
next prev parent reply other threads:[~2019-03-29 9:03 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-28 23:11 [edk2-test][PATCH v1 01/30] SctPkg/UEFI: Modify description files to ignore Ebc for AArch64 Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 02/30] SctPkg: Ignore Ebc protocol test " Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 03/30] IHV/Protocol: Fix inf files for IHV protocol tests Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 04/30] RIVL/Protocol: Fix inf files for RIVL Arp " Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 05/30] RIVL/Protocol: Fix inf files for Dhcp4 " Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 06/30] RIVL/Protocol: Fix inf files for Dhcp6 " Supreeth Venkatesh
2019-03-28 23:11 ` [edk2-test][PATCH v1 07/30] RIVL/Protocol: Fix inf files for Http " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 08/30] RIVL/Protocol: Fix inf files for InternetProtocol4 tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 09/30] RIVL/Protocol: Fix inf files for Ip6 tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 10/30] RIVL/Protocol: Fix inf files for Managed network protocol tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 11/30] RIVL/Protocol: Fix inf files for Mtftp4 " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 12/30] RIVL/Protocol: Fix inf files for Mtftp6 " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 13/30] RIVL/Protocol: fix inf file for PXE protocol test Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 14/30] RIVL/Protocol: Fix inf files for Simple Network " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 15/30] RIVL/Protocol: Fix inf files for Tcp4 protocol tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 16/30] RIVL/Protocol: Fix inf file for Tcp6 " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 17/30] RIVL/Protocol: Fix inf files for Udp4 " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 18/30] RIVL/Protocol: Fix inf files for Udp6 " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 19/30] BootServices: Fix inf files for ImageServices dependency data files Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 20/30] EFI/Protocol: Fix inf file for DebugSupport protocol test Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 21/30] EFI/Protocol: Fix inf files for Decompress protocol dependency data Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 22/30] EFI/Protocol: Fix inf files for DevicePath related protocol tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 23/30] EFI/Protocol: Fix inf files for HII " Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 24/30] UEFI/Protocol: Remove struct and enum in Hii defines file Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 25/30] EFI/Protocol: Fix inf files for SimpleText protocol tests Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 26/30] EFI/Protocol: Fix inf files for Pci protocol related dependency data Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 27/30] EFI/Protocol: Fix inf file for LoadedImage protocol test Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 28/30] EFI/Protocol: Fix inf file for PxeBase dependency data Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 29/30] EFI/Protocol: Fix inf file for SimpleFileSystem protocol test Supreeth Venkatesh
2019-03-28 23:12 ` [edk2-test][PATCH v1 30/30] UEFI/UEFI.dec: Add missing protocol GUIDs in declaration file Supreeth Venkatesh
2019-03-29 9:03 ` Jin, Eric [this message]
2019-03-29 14:12 ` Supreeth Venkatesh
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=DA72DC7456565B47808A57108259571F6376F721@SHSMSX103.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