[edk2-devel] [PATCH v2 0/6] Enable CI in Intel FSP Packages

[Michael Kubacki]

From: Michael Kubacki <michael.kubacki at microsoft.com>

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 <chasel.chiu at intel.com>
Cc: Liming Gao <gaoliming at byosoft.com.cn>
Cc: Michael D Kinney <michael.d.kinney at intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone at intel.com>
Cc: Sean Brogan <sean.brogan at microsoft.com>
Cc: Star Zeng <star.zeng at intel.com>
Signed-off-by: Michael Kubacki <michael.kubacki at microsoft.com>

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 (#93859): https://edk2.groups.io/g/devel/message/93859
Mute This Topic: https://groups.io/mt/93707361/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-