From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.136; helo=mga12.intel.com; envelope-from=michael.d.kinney@intel.com; receiver=edk2-devel@lists.01.org Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 21EBF210C2D92 for ; Wed, 1 Aug 2018 16:20:42 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Aug 2018 16:20:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,433,1526367600"; d="scan'208";a="77964746" Received: from orsmsx110.amr.corp.intel.com ([10.22.240.8]) by orsmga001.jf.intel.com with ESMTP; 01 Aug 2018 16:20:36 -0700 Received: from orsmsx151.amr.corp.intel.com (10.22.226.38) by ORSMSX110.amr.corp.intel.com (10.22.240.8) with Microsoft SMTP Server (TLS) id 14.3.319.2; Wed, 1 Aug 2018 16:20:36 -0700 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.96]) by ORSMSX151.amr.corp.intel.com ([169.254.7.226]) with mapi id 14.03.0319.002; Wed, 1 Aug 2018 16:20:36 -0700 From: "Kinney, Michael D" To: "Zeng, Star" , "edk2-devel@lists.01.org" , "Kinney, Michael D" Thread-Topic: [PATCH] FmpDevicePkg: Remove useless lib mapping in dsc Thread-Index: AQHUKYKs47HSs2iSmkCvsIWuptvR1qSq8BTwgACZCrA= Date: Wed, 1 Aug 2018 23:20:35 +0000 Message-ID: References: <1533119422-9408-1-git-send-email-star.zeng@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.22.254.139] MIME-Version: 1.0 Subject: Re: [PATCH] FmpDevicePkg: Remove useless lib mapping in dsc X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 01 Aug 2018 23:20:44 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Star, I have verified that removing the rest of the lib mappings still builds, so those changes look good. I have integrated these changes except the ARM/AARCH64 changes into a V6 version of the patch series. Thanks, Mike > -----Original Message----- > From: Kinney, Michael D > Sent: Wednesday, August 1, 2018 7:15 AM > To: Zeng, Star ; edk2- > devel@lists.01.org; Kinney, Michael D > > Subject: RE: [PATCH] FmpDevicePkg: Remove useless lib > mapping in dsc >=20 > Star, >=20 > The ARM and AARCH64 specific lib mappings should not be > removed. > That will break the FmpDevicePkg build for ARM and > AARCH64. >=20 > Mike >=20 > > -----Original Message----- > > From: Zeng, Star > > Sent: Wednesday, August 1, 2018 3:30 AM > > To: edk2-devel@lists.01.org > > Cc: Zeng, Star ; Kinney, Michael > D > > > > Subject: [PATCH] FmpDevicePkg: Remove useless lib > > mapping in dsc > > > > Cc: Michael D Kinney > > Contributed-under: TianoCore Contribution Agreement > 1.1 > > Signed-off-by: Star Zeng > > > > NOTE: This patch is based on thread > > https://lists.01.org/pipermail/edk2-devel/2018- > > July/027810.html. > > > > --- > > FmpDevicePkg/FmpDevicePkg.dsc | 34 ----------------- > -- > > --------------- > > 1 file changed, 34 deletions(-) > > > > diff --git a/FmpDevicePkg/FmpDevicePkg.dsc > > b/FmpDevicePkg/FmpDevicePkg.dsc > > index 8d2436627dae..c04cb0efe86e 100644 > > --- a/FmpDevicePkg/FmpDevicePkg.dsc > > +++ b/FmpDevicePkg/FmpDevicePkg.dsc > > @@ -48,58 +48,24 @@ [Defines] > > > > [LibraryClasses] > > > > > UefiDriverEntryPoint|MdePkg/Library/UefiDriverEntryPoin > > t/UefiDriverEntryPoint.inf > > - > > > UefiApplicationEntryPoint|MdePkg/Library/UefiApplicatio > > nEntryPoint/UefiApplicationEntryPoint.inf > > > > > UefiBootServicesTableLib|MdePkg/Library/UefiBootService > > sTableLib/UefiBootServicesTableLib.inf > > UefiLib|MdePkg/Library/UefiLib/UefiLib.inf > > > > > UefiRuntimeServicesTableLib|MdePkg/Library/UefiRuntimeS > > ervicesTableLib/UefiRuntimeServicesTableLib.inf > > - > > > UefiRuntimeLib|MdePkg/Library/UefiRuntimeLib/UefiRuntim > > eLib.inf > > > > > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocation > > Lib/UefiMemoryAllocationLib.inf > > > > > DevicePathLib|MdePkg/Library/UefiDevicePathLib/UefiDevi > > cePathLib.inf > > - > UefiUsbLib|MdePkg/Library/UefiUsbLib/UefiUsbLib.inf > > - > > > UefiScsiLib|MdePkg/Library/UefiScsiLib/UefiScsiLib.inf > > BaseLib|MdePkg/Library/BaseLib/BaseLib.inf > > > > > BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLi > > b.inf > > - > > > SynchronizationLib|MdePkg/Library/BaseSynchronizationLi > > b/BaseSynchronizationLib.inf > > > > PrintLib|MdePkg/Library/BasePrintLib/BasePrintLib.inf > > > > > DebugLib|MdePkg/Library/UefiDebugLibStdErr/UefiDebugLib > > StdErr.inf > > > > > DebugPrintErrorLevelLib|MdePkg/Library/BaseDebugPrintEr > > rorLevelLib/BaseDebugPrintErrorLevelLib.inf > > - > > > PostCodeLib|MdePkg/Library/BasePostCodeLibPort80/BasePo > > stCodeLibPort80.inf > > > > > PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf > > - > > > DxeServicesLib|MdePkg/Library/DxeServicesLib/DxeService > > sLib.inf > > - > > - > > > UefiRuntimeServicesTableLib|MdePkg/Library/UefiRuntimeS > > ervicesTableLib/UefiRuntimeServicesTableLib.inf > > - UefiLib|MdePkg/Library/UefiLib/UefiLib.inf > > - > > > DevicePathLib|MdePkg/Library/UefiDevicePathLib/UefiDevi > > cePathLib.inf > > - > > > UefiBootManagerLib|MdeModulePkg/Library/UefiBootManager > > Lib/UefiBootManagerLib.inf > > - > > > DxeServicesTableLib|MdePkg/Library/DxeServicesTableLib/ > > DxeServicesTableLib.inf > > - > > > MemoryAllocationLib|MdePkg/Library/UefiMemoryAllocation > > Lib/UefiMemoryAllocationLib.inf > > - HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf > > > > > OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLib.inf > > > > > IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLi > > b.inf > > > > > BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLi > > b.inf > > > > > FmpAuthenticationLib|SecurityPkg/Library/FmpAuthenticat > > ionLibPkcs7/FmpAuthenticationLibPkcs7.inf > > - > > > > > CapsuleUpdatePolicyLib|FmpDevicePkg/Library/CapsuleUpda > > tePolicyLibNull/CapsuleUpdatePolicyLibNull.inf > > > > > FmpPayloadHeaderLib|FmpDevicePkg/Library/FmpPayloadHead > > erLibV1/FmpPayloadHeaderLibV1.inf > > > > > FmpDeviceLib|FmpDevicePkg/Library/FmpDeviceLibNull/FmpD > > eviceLibNull.inf > > - > > > BmpSupportLib|MdeModulePkg/Library/BaseBmpSupportLib/Ba > > seBmpSupportLib.inf > > - > > > SafeIntLib|MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib > > .inf > > - > > > CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCap > > suleLib.inf > > - > > -[LibraryClasses.ARM, LibraryClasses.AARCH64] > > - # > > - # It is not possible to prevent the ARM compiler > for > > generic intrinsic functions. > > - # This library provides the instrinsic functions > > generate by a given compiler. > > - # [LibraryClasses.ARM, LibraryClasses.AARCH64] and > > NULL mean link this library > > - # into all ARM and AARCH64 images. > > - # > > - > > > NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntri > > nsicsLib.inf > > - > > - # Add support for stack protector > > - > > > NULL|MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib > > .inf > > - > > -[LibraryClasses.ARM] > > - > > > ArmSoftFloatLib|ArmPkg/Library/ArmSoftFloatLib/ArmSoftF > > loatLib.inf > > > > [Components] > > # > > -- > > 2.7.0.windows.1