From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web09.5538.1575010804323725867 for ; Thu, 28 Nov 2019 23:00:04 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: chasel.chiu@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Nov 2019 23:00:03 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,256,1571727600"; d="scan'208";a="221524322" Received: from pgsmsx108.gar.corp.intel.com ([10.221.44.103]) by orsmga002.jf.intel.com with ESMTP; 28 Nov 2019 23:00:02 -0800 Received: from pgsmsx111.gar.corp.intel.com ([169.254.2.24]) by PGSMSX108.gar.corp.intel.com ([10.221.44.103]) with mapi id 14.03.0439.000; Fri, 29 Nov 2019 15:00:01 +0800 From: "Chiu, Chasel" To: "Kubacki, Michael A" , "devel@edk2.groups.io" CC: "Desimone, Nathaniel L" Subject: Re: [edk2-platforms][PATCH V2 35/47] KabylakeOpenBoardPkg/GalagoPro3: Enable advanced features Thread-Topic: [edk2-platforms][PATCH V2 35/47] KabylakeOpenBoardPkg/GalagoPro3: Enable advanced features Thread-Index: AQHVpYhbrrzbvpkXMEqLCdy4uz2txqehuhUA Date: Fri, 29 Nov 2019 07:00:01 +0000 Message-ID: <3C3EFB470A303B4AB093197B6777CCEC505B1F7C@PGSMSX111.gar.corp.intel.com> References: <20191128010614.43628-1-michael.a.kubacki@intel.com> <20191128010614.43628-36-michael.a.kubacki@intel.com> In-Reply-To: <20191128010614.43628-36-michael.a.kubacki@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjYxZTRmY2ItMTBmZC00MjQxLThlOGQtZmZhNWIxNzViM2RiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiUkx4OFZmdWl2emNPd2pcLzBkYVRhOWNuK0xrVlY0dUJDT013Rit3VjFqOGhFMmpqTEQwZ1ZXakRXdGdWMlY3S3kifQ== x-ctpclassification: CTP_NT x-originating-ip: [172.30.20.206] MIME-Version: 1.0 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Chasel Chiu > -----Original Message----- > From: Kubacki, Michael A > Sent: Thursday, November 28, 2019 9:06 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > > Subject: [edk2-platforms][PATCH V2 35/47] > KabylakeOpenBoardPkg/GalagoPro3: Enable advanced features >=20 > This change enables advanced features to be used by the > GalagoPro3 board in KabylakeOpenBoardPkg. >=20 > Some platform macros in OpenBoardPkg.dsc are consolidated so all values > are set at the top of the file before including advanced feature code. >=20 > Cc: Chasel Chiu > Cc: Nate DeSimone > Signed-off-by: Michael Kubacki > Reviewed-by: Nate DeSimone > --- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > | 46 +++++++++----------- > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc | > 1 + > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf | > 4 ++ > 3 files changed, 25 insertions(+), 26 deletions(-) >=20 > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > index d4ca2daa18..a39547f41f 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > +++ b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc > @@ -16,38 +16,32 @@ > DEFINE PROJECT =3D > $(PLATFORM_BOARD_PACKAGE)/$(BOARD) > DEFINE PEI_ARCH =3D IA32 > DEFINE DXE_ARCH =3D X64 > - > - # > - # Include PCD configuration for this board. > - # > - !include OpenBoardPkgPcd.dsc > - > -############################################################### > ################# > -# > -# Defines Section - statements that will be processed to create a Makefi= le. > -# > -############################################################### > ################# > -[Defines] > - PLATFORM_NAME =3D $(PLATFORM_PACKAGE) > - PLATFORM_GUID =3D > 7324F33D-4E96-4F8B-A550-544DE6162AB7 > - PLATFORM_VERSION =3D 0.1 > - DSC_SPECIFICATION =3D 0x00010005 > - OUTPUT_DIRECTORY =3D Build/$(PROJECT) > - SUPPORTED_ARCHITECTURES =3D IA32|X64 > - BUILD_TARGETS =3D DEBUG|RELEASE > - SKUID_IDENTIFIER =3D ALL > - > - > - FLASH_DEFINITION =3D > $(PROJECT)/OpenBoardPkg.fdf > - > - FIX_LOAD_TOP_MEMORY_ADDRESS =3D 0x0 > - DEFINE TOP_MEMORY_ADDRESS =3D 0x0 > + DEFINE TOP_MEMORY_ADDRESS =3D 0x0 >=20 > # > # Default value for OpenBoardPkg.fdf use > # > DEFINE BIOS_SIZE_OPTION =3D SIZE_60 >=20 > + PLATFORM_NAME =3D > $(PLATFORM_PACKAGE) > + PLATFORM_GUID =3D > 7324F33D-4E96-4F8B-A550-544DE6162AB7 > + PLATFORM_VERSION =3D 0.1 > + DSC_SPECIFICATION =3D 0x00010005 > + OUTPUT_DIRECTORY =3D Build/$(PROJECT) > + SUPPORTED_ARCHITECTURES =3D IA32|X64 > + BUILD_TARGETS =3D DEBUG|RELEASE > + SKUID_IDENTIFIER =3D ALL > + > + FLASH_DEFINITION =3D > $(PROJECT)/OpenBoardPkg.fdf > + FIX_LOAD_TOP_MEMORY_ADDRESS =3D 0x0 > + > + # > + # Include PCD configuration for this board. > + # > + !include > + > AdvancedFeaturePkg/TemporaryBuildWorkaround/TemporaryBuildWorkaro > und.d > + sc > + !include OpenBoardPkgPcd.dsc > + !include AdvancedFeaturePkg/Include/AdvancedFeatures.dsc > + >=20 > ################################################################ > ################ > # > # SKU Identification section - list of all SKU IDs supported by this boa= rd. > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > index c6f5ae7534..7393897249 100644 > --- > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > +++ > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkgPcd.dsc > @@ -24,6 +24,7 @@ > # Stage 3 - boot to shell only > # Stage 4 - boot to OS > # Stage 5 - boot to OS with security boot enabled > + # Stage 6 - boot with advanced features enabled > # > gMinPlatformPkgTokenSpaceGuid.PcdBootStage|4 >=20 > diff --git > a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf > b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf > index 0a9b933e40..da94ed0cd1 100644 > --- a/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf > +++ b/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf > @@ -631,6 +631,8 @@ READ_LOCK_CAP =3D TRUE > READ_LOCK_STATUS =3D TRUE > FvNameGuid =3D 6053D78A-457E-4490-A237-31D0FBE2F305 >=20 > +!include AdvancedFeaturePkg/Include/PreMemory.fdf > + > !if gKabylakeOpenBoardPkgTokenSpaceGuid.PcdTbtEnable =3D=3D TRUE INF > $(PLATFORM_BOARD_PACKAGE)/Features/Tbt/TbtInit/Pei/PeiTbtInit.inf > !endif > @@ -657,6 +659,8 @@ READ_LOCK_CAP =3D TRUE > READ_LOCK_STATUS =3D TRUE > FvNameGuid =3D BE3DF86F-E464-44A3-83F7-0D27E6B88C27 >=20 > +!include AdvancedFeaturePkg/Include/PostMemory.fdf > + > !if gKabylakeOpenBoardPkgTokenSpaceGuid.PcdTbtEnable =3D=3D TRUE INF > $(PLATFORM_BOARD_PACKAGE)/Features/Tbt/TbtInit/Dxe/TbtDxe.inf > INF $(PLATFORM_BOARD_PACKAGE)/Features/PciHotPlug/PciHotPlug.inf > -- > 2.16.2.windows.1