With one comment that IntelFsp2WrapperPkg.dec file needs a new issue opened to resolve 2 library classes pointing to same include file.
Series Reviewed-by: Michael D Kinney <michael.d.kinney> Mike > -----Original Message----- > From: [email protected] <[email protected]> > Sent: Thursday, September 15, 2022 11:55 AM > To: [email protected] > Cc: Chiu, Chasel <[email protected]>; Gao, Liming > <[email protected]>; Kinney, Michael D <[email protected]>; > Desimone, Nathaniel L <[email protected]>; Sean Brogan > <[email protected]>; Zeng, Star <[email protected]> > Subject: [PATCH v2 0/6] Enable CI in Intel FSP Packages > > From: Michael Kubacki <[email protected]> > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4048 > > - Enables CI in IntelFsp2Pkg and IntelFsp2WrapperPkg. > - Fixes several pre-existing issues that impact common CI checks. > > You can find the CI results for the packages with this change > in the following edk2 PR: > https://github.com/tianocore/edk2/pull/3347 > > V2 Changes: > > 1. The pre-existing compilation issue in IntelFsp2Pkg that caused > the following BZ to be filed in v1 is now resolved. Therefore, > the compiler CI plugin is enabled in IntelFsp2Pkg now. > > https://bugzilla.tianocore.org/show_bug.cgi?id=4049 > > 2. The following patch is dropped from v2: > > [PATCH v1 5/7] IntelFsp2WrapperPkg.dec: Remove duplicate > LibraryClasses entry > > Chasel indicated this is an intentional design decision so > platforms do not have to override the entire library instance > during platform integration. > > Consequently, "FspWrapperPlatformMultiPhaseLib" was added to the > ignore list for the "LibraryClassCheck" CI plugin in > IntelFspWrapperPkg.ci.yaml. > > Rebased series on f46c7d1e36c9 and added v1 R-b tags. > > Cc: Chasel Chiu <[email protected]> > Cc: Liming Gao <[email protected]> > Cc: Michael D Kinney <[email protected]> > Cc: Nate DeSimone <[email protected]> > Cc: Sean Brogan <[email protected]> > Cc: Star Zeng <[email protected]> > Signed-off-by: Michael Kubacki <[email protected]> > > Michael Kubacki (6): > IntelFsp2Pkg: Fix code formatting errors > IntelFsp2Pkg/BaseFspMultiPhaseLib: Replace duplicate GUID > IntelFsp2Pkg: Add CI YAML file > IntelFsp2WrapperPkg: Fix code formatting errors > IntelFsp2WrapperPkg: Add CI YAML file > .azurepipelines: Add IntelFsp2Pkg and IntelFsp2WrapperPkg to CI > > IntelFsp2Pkg/Library/BaseFspDebugLibSerialPort/DebugLib.c > | 9 +- > IntelFsp2Pkg/Library/BaseFspSwitchStackLib/FspSwitchStackLib.c > | 2 +- > IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/FspWrapperApiLib.c > | 4 + > IntelFsp2WrapperPkg/Library/BaseFspWrapperApiLib/IA32/DispatchExecute.c > | 1 - > > IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecRamInitData.c > | 8 +- > .azurepipelines/templates/pr-gate-build-job.yml > | 3 + > .pytool/CISettings.py > | 2 + > IntelFsp2Pkg/Include/Ppi/Variable.h > | 8 +- > IntelFsp2Pkg/IntelFsp2Pkg.ci.yaml > | 90 ++++++++++++++++++ > IntelFsp2Pkg/Library/BaseFspMultiPhaseLib/BaseFspMultiPhaseLib.inf > | 2 +- > IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.ci.yaml > | 96 ++++++++++++++++++++ > 11 files changed, 210 insertions(+), 15 deletions(-) > create mode 100644 IntelFsp2Pkg/IntelFsp2Pkg.ci.yaml > create mode 100644 IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.ci.yaml > > -- > 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94716): https://edk2.groups.io/g/devel/message/94716 Mute This Topic: https://groups.io/mt/93707361/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
