Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make PlatformInitPreMem independent to IntelSiliconPkg and IntelFsp2WrapperPkg #310

Conversation

makubacki
Copy link
Member

Description

This driver is shared across different Silicon vendors. So it should not use any Intel specific packages.

In order to achieve this, added a new lib FspSupportLib to isolate silicon related PCDs


Note: This change targeted the "release" branch in #293 before the "dev" branch was created. This adds the commit to the dev branch for parity with the release branch.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

CI build passed.

Integration Instructions

No additional change is required for the consuming platforms.

…rapperPkg (microsoft#293)

## Description

This driver is shared across different Silicon vendors. So it should not
use any Intel specific packages.

In order to achieve this, added a new lib `FspSupportLib` to isolate
silicon related PCDs

- [ ] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

CI build passed.

## Integration Instructions

No additional change is required for the consuming platforms.

---------

Co-authored-by: Michael Kubacki <[email protected]>
@makubacki makubacki requested review from kenlautner and apop5 November 5, 2024 18:42
@makubacki makubacki self-assigned this Nov 5, 2024
@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Nov 5, 2024
@makubacki makubacki merged commit 8cbbc6e into microsoft:dev/202405 Nov 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:non-functional Does not have a functional impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants