From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) (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 C1CC681FAD for ; Mon, 5 Dec 2016 22:28:51 -0800 (PST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP; 05 Dec 2016 22:28:51 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,751,1477983600"; d="scan'208";a="1095145206" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga002.fm.intel.com with ESMTP; 05 Dec 2016 22:28:51 -0800 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.248.2; Mon, 5 Dec 2016 22:28:51 -0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.239]) by SHSMSX104.ccr.corp.intel.com ([169.254.5.142]) with mapi id 14.03.0248.002; Tue, 6 Dec 2016 14:28:49 +0800 From: "Wei, David" To: Leif Lindholm , "edk2-devel@lists.01.org" CC: "Guo, Mang" , "Wei, David" Thread-Topic: [PATCH 7/7] Vlv2TbltDevicePkg: Remove use of IntelFrameworkModulePkg legacy libs Thread-Index: AQHSS+sfAS6j3vgrukSiL0/iAKAcI6D5HUuAgAFenGA= Date: Tue, 6 Dec 2016 06:28:48 +0000 Message-ID: <89954A0B46707A448411A627AD4EEE34663B0AA2@SHSMSX101.ccr.corp.intel.com> References: <20161201155326.21716-1-leif.lindholm@linaro.org> <20161201155326.21716-8-leif.lindholm@linaro.org> <20161205173259.GQ27069@bivouac.eciton.net> In-Reply-To: <20161205173259.GQ27069@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiOWI2NTRlZmQtZTNlOC00Yzc2LWIzYzktMTAwYzgzZWRiNTkzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6Im1jWmNDN3h6eUY3c3BUSUpcL2p6bzZcL2ZMMHVpSVwvSGw4VjJuK09XRXVRZ0U9In0= x-ctpclassification: CTP_IC x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 7/7] Vlv2TbltDevicePkg: Remove use of IntelFrameworkModulePkg legacy libs X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Dec 2016 06:28:51 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable It looks good. Reviewed-by: David Wei =20 Thanks, David Wei =20 -----Original Message----- From: Leif Lindholm [mailto:leif.lindholm@linaro.org]=20 Sent: Tuesday, December 06, 2016 1:33 AM To: edk2-devel@lists.01.org Cc: Wei, David ; Guo, Mang Subject: Re: [PATCH 7/7] Vlv2TbltDevicePkg: Remove use of IntelFrameworkMod= ulePkg legacy libs Hi David, Mang - any comments? All the other parts of the series have now been pushed. Regards, Leif On Thu, Dec 01, 2016 at 03:53:26PM +0000, Leif Lindholm wrote: > LzmaCustomDecompressLib and PeiDxeDebugLibReportStatusCode were copied > from IntelFrameworkModulePkg to MdeModulePkg, but the originals were > kept for compatibility. >=20 > Since the libraries are identical, move Vlv2TbltDevicePkg to use the > MdeModulePkg versions instead. >=20 > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Leif Lindholm > --- > Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 4 ++-- > Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 6 +++--- > Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 6 +++--- > 3 files changed, 8 insertions(+), 8 deletions(-) >=20 > diff --git a/Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc b/Vlv2TbltDevicePkg/= PlatformPkgGccX64.dsc > index 6da2a8a..80d75d1 100644 > --- a/Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc > +++ b/Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc > @@ -1064,7 +1064,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf { > > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaCus= tomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecompre= ssLib.inf > !endif > } > =20 > @@ -1121,7 +1121,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > NULL|MdeModulePkg/Library/DxeCrc32GuidedSectionExtractLib/DxeCrc32= GuidedSectionExtractLib.inf > !endif > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaC= ustomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecomp= ressLib.inf > !endif > !if $(TARGET) !=3D RELEASE > DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialP= ort.inf > diff --git a/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc b/Vlv2TbltDevicePkg/Pl= atformPkgIA32.dsc > index 5b5523f..44ec378 100644 > --- a/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > +++ b/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > @@ -332,7 +332,7 @@ > DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf > SerialPortLib|MdePkg/Library/BaseSerialPortLibNull/BaseSerialPortLibNu= ll.inf > !else > - DebugLib|IntelFrameworkModulePkg/Library/PeiDxeDebugLibReportStatusCod= e/PeiDxeDebugLibReportStatusCode.inf > + DebugLib|MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/PeiDxeDeb= ugLibReportStatusCode.inf > SerialPortLib|$(PLATFORM_PACKAGE)/Library/SerialPortLib/SerialPortLib.= inf > !endif > =20 > @@ -1058,7 +1058,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf { > > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaCus= tomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecompre= ssLib.inf > !endif > } > =20 > @@ -1115,7 +1115,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > NULL|MdeModulePkg/Library/DxeCrc32GuidedSectionExtractLib/DxeCrc32= GuidedSectionExtractLib.inf > !endif > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaC= ustomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecomp= ressLib.inf > !endif > !if $(TARGET) !=3D RELEASE > DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialP= ort.inf > diff --git a/Vlv2TbltDevicePkg/PlatformPkgX64.dsc b/Vlv2TbltDevicePkg/Pla= tformPkgX64.dsc > index 54d2b81..40568f6 100644 > --- a/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > +++ b/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > @@ -332,7 +332,7 @@ > DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf > SerialPortLib|MdePkg/Library/BaseSerialPortLibNull/BaseSerialPortLibNu= ll.inf > !else > - DebugLib|IntelFrameworkModulePkg/Library/PeiDxeDebugLibReportStatusCod= e/PeiDxeDebugLibReportStatusCode.inf > + DebugLib|MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/PeiDxeDeb= ugLibReportStatusCode.inf > SerialPortLib|$(PLATFORM_PACKAGE)/Library/SerialPortLib/SerialPortLib.= inf > !endif > =20 > @@ -1058,7 +1058,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf { > > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaCus= tomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecompre= ssLib.inf > !endif > } > =20 > @@ -1115,7 +1115,7 @@ $(PLATFORM_BINARY_PACKAGE)/$(DXE_ARCHITECTURE)$(TAR= GET)/IA32/fTPMInitPeim.inf > NULL|MdeModulePkg/Library/DxeCrc32GuidedSectionExtractLib/DxeCrc32= GuidedSectionExtractLib.inf > !endif > !if $(LZMA_ENABLE) =3D=3D TRUE > - NULL|IntelFrameworkModulePkg/Library/LzmaCustomDecompressLib/LzmaC= ustomDecompressLib.inf > + NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecomp= ressLib.inf > !endif > !if $(TARGET) !=3D RELEASE > DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialP= ort.inf > --=20 > 2.10.2 >=20