public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
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



  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