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.web10.3217.1574404611107866576 for ; Thu, 21 Nov 2019 22:36:51 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: rangasai.v.chaganty@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 21 Nov 2019 22:36:50 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,228,1571727600"; d="scan'208";a="232576933" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by fmsmga004.fm.intel.com with ESMTP; 21 Nov 2019 22:36:50 -0800 Received: from fmsmsx114.amr.corp.intel.com (10.18.116.8) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 21 Nov 2019 22:36:50 -0800 Received: from fmsmsx104.amr.corp.intel.com ([169.254.3.245]) by FMSMSX114.amr.corp.intel.com ([169.254.6.239]) with mapi id 14.03.0439.000; Thu, 21 Nov 2019 22:36:50 -0800 From: "Chaganty, Rangasai V" To: "Desimone, Nathaniel L" , "devel@edk2.groups.io" CC: "Chiu, Chasel" , "Kubacki, Michael A" Subject: Re: [edk2-platforms] [PATCH V2 01/14] KabylakeSiliconPkg: Change MODULE_TYPE of SiliconInitLib to PEIM Thread-Topic: [edk2-platforms] [PATCH V2 01/14] KabylakeSiliconPkg: Change MODULE_TYPE of SiliconInitLib to PEIM Thread-Index: AQHVoEn7BFrXWiG3U0+LFG1i2y9d06eWvcQQ Date: Fri, 22 Nov 2019 06:36:49 +0000 Message-ID: References: <20191121085853.2626-1-nathaniel.l.desimone@intel.com> <20191121085853.2626-2-nathaniel.l.desimone@intel.com> In-Reply-To: <20191121085853.2626-2-nathaniel.l.desimone@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: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNzg0YjFmNjItODFkMC00NzRjLWFmOGQtMDdkMDRhYWYxZjE2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiUElmZzI0VlppU0Vod2lUY1A4VmRHWVhVeXhXVmFmaVdXUzNSUWZNcHk1NW5ZQTFKMmUwMW40UXpZME44NWhldCJ9 x-ctpclassification: CTP_NT x-originating-ip: [10.1.200.106] MIME-Version: 1.0 Return-Path: rangasai.v.chaganty@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Sai Chaganty -----Original Message----- From: Desimone, Nathaniel L=20 Sent: Thursday, November 21, 2019 12:59 AM To: devel@edk2.groups.io Cc: Chiu, Chasel ; Kubacki, Michael A ; Chaganty, Rangasai V Subject: [edk2-platforms] [PATCH V2 01/14] KabylakeSiliconPkg: Change MODUL= E_TYPE of SiliconInitLib to PEIM This library contains PEI phase only code. For example, ClearIchSmiAndWake() calls PeiServicesGetBootMode(). Accordingly, this library should have never been marked as a BASE library. Cc: Chasel Chiu Cc: Michael Kubacki Cc: Sai Chaganty Signed-off-by: Nate DeSimone --- .../PeiSiliconInitLib.inf} | 5 +++-- .../{SiliconInitLib =3D> PeiSiliconInitLib}/SiliconInit.c | 0 .../SiliconInitPreMem.c | 0 3 files changed, 3 insertions(+), 2 deletions(-) rename Silicon/Intel/Kab= ylakeSiliconPkg/Library/{SiliconInitLib/SiliconInitLib.inf =3D> PeiSiliconI= nitLib/PeiSiliconInitLib.inf} (86%) rename Silicon/Intel/KabylakeSiliconPk= g/Library/{SiliconInitLib =3D> PeiSiliconInitLib}/SiliconInit.c (100%) ren= ame Silicon/Intel/KabylakeSiliconPkg/Library/{SiliconInitLib =3D> PeiSilico= nInitLib}/SiliconInitPreMem.c (100%) diff --git a/Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/Silico= nInitLib.inf b/Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/P= eiSiliconInitLib.inf similarity index 86% rename from Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/Silicon= InitLib.inf rename to Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/PeiSil= iconInitLib.inf index 132080e876..bd77b081e5 100644 --- a/Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/SiliconInitLi= b.inf +++ b/Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/PeiSili +++ conInitLib.inf @@ -11,8 +11,9 @@ BASE_NAME =3D SiliconInitLib FILE_GUID = =3D 72CD3A7B-FEA5-4F5E-9165-4DD12187BB13 VERSION_STRING = =3D 1.0- MODULE_TYPE =3D BASE- LIBRARY_CLASS= =3D SiliconInitLib+ MODULE_TYPE =3D P= EIM+ LIBRARY_CLASS =3D SiliconInitLib|SEC PEIM+ # # The f= ollowing information is for reference only and not required by the build to= ols. #diff --git a/Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/= SiliconInit.c b/Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/= SiliconInit.c similarity index 100% rename from Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/Silicon= Init.c rename to Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/Silico= nInit.c diff --git a/Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/Silico= nInitPreMem.c b/Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/= SiliconInitPreMem.c similarity index 100% rename from Silicon/Intel/KabylakeSiliconPkg/Library/SiliconInitLib/Silicon= InitPreMem.c rename to Silicon/Intel/KabylakeSiliconPkg/Library/PeiSiliconInitLib/Silico= nInitPreMem.c -- 2.24.0.windows.2