Reviewed-by: Chasel Chiu <chasel.c...@intel.com>
> -----Original Message----- > From: Kubacki, Michael A <michael.a.kuba...@intel.com> > Sent: Tuesday, October 8, 2019 1:17 PM > To: devel@edk2.groups.io > Cc: Chaganty, Rangasai V <rangasai.v.chaga...@intel.com>; Chiu, Chasel > <chasel.c...@intel.com> > Subject: [edk2-platforms][PATCH V1 10/17] CoffeelakeSiliconPkg: Add DSC > include file section tags > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2246 > > The DSC LibraryClass files in CoffeelakeSiliconPkg that are intended to be > included elsewhere such as SiPkgPeiLib.dsc, SiPkgDxeLib.dsc, and > SiPkgCommonLib.dsc should have section tags so that they are not > dependent on the top-level DSC file to place the include file in the correct > location in the DSC file and better define the applicability of their library > content. > > This change adds section tags for the library class related files. > The component files may be built differently in the consuming package > depending on their architecture requirements so those are not modified. > > Cc: Sai Chaganty <rangasai.v.chaga...@intel.com> > Cc: Chasel Chiu <chasel.c...@intel.com> > Signed-off-by: Michael Kubacki <michael.a.kuba...@intel.com> > --- > Silicon/Intel/CoffeelakeSiliconPkg/SiPkgCommonLib.dsc | 1 + > Silicon/Intel/CoffeelakeSiliconPkg/SiPkgDxeLib.dsc | 1 + > Silicon/Intel/CoffeelakeSiliconPkg/SiPkgPeiLib.dsc | 1 + > 3 files changed, 3 insertions(+) > > diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgCommonLib.dsc > b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgCommonLib.dsc > index 2df08c6d01..7a9911e825 100644 > --- a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgCommonLib.dsc > +++ b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgCommonLib.dsc > @@ -13,6 +13,7 @@ > # > DEFINE PCH = Cnl > > +[LibraryClasses.common] > # > # Cpu > # > diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgDxeLib.dsc > b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgDxeLib.dsc > index 214de06d58..e21004c993 100644 > --- a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgDxeLib.dsc > +++ b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgDxeLib.dsc > @@ -7,6 +7,7 @@ > # > ## > > +[LibraryClasses.common.DXE_CORE,LibraryClasses.common.DXE_SMM_DRIV > ER,Li > +braryClasses.common.SMM_CORE,LibraryClasses.common.DXE_DRIVER,Libr > aryCl > +asses.common.DXE_RUNTIME_DRIVER,LibraryClasses.common.UEFI_DRIVER > ] > # > # Silicon Init Dxe Library > # > diff --git a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgPeiLib.dsc > b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgPeiLib.dsc > index 6e244a6ded..bc3fcabd4d 100644 > --- a/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgPeiLib.dsc > +++ b/Silicon/Intel/CoffeelakeSiliconPkg/SiPkgPeiLib.dsc > @@ -7,6 +7,7 @@ > # > ## > > +[LibraryClasses] > # > # Silicon Init Pei Library > # > -- > 2.16.2.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48598): https://edk2.groups.io/g/devel/message/48598 Mute This Topic: https://groups.io/mt/34439446/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-