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.

Sign up for GitHub

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

Jump to bottom

Add fspiwrapper peim #6292

Closed
wants to merge 5 commits into from
Closed

Add fspiwrapper peim #6292

wants to merge 5 commits into from

Conversation

hongbin123
Copy link
Contributor

  • Breaking change?
    • Breaking change - Does this PR cause a break in build or boot behavior?
    • Examples: Does it add a new library class or move a module to a different repo.
  • Impacts security?
    • Security - Does this PR have a direct security impact?
    • Examples: Crypto algorithm change or buffer overflow fix.
  • Includes tests?
    • Tests - Does this PR include any explicit test code?
    • Examples: Unit tests or integration tests.

@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch 5 times, most recently from a0d8a3d to c15430c Compare October 8, 2024 10:21
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch from c15430c to f55141c Compare October 9, 2024 00:30
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch from f55141c to fbc537a Compare October 10, 2024 03:30
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch from fbc537a to 5548968 Compare October 11, 2024 11:12
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch 5 times, most recently from 73da281 to 835e23e Compare October 11, 2024 13:20
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch 3 times, most recently from 3053726 to aa190ad Compare October 16, 2024 01:03
@ChaselChiu ChaselChiu added the push Auto push patch series in PR if all checks pass label Oct 16, 2024
@ChaselChiu ChaselChiu added push Auto push patch series in PR if all checks pass and removed push Auto push patch series in PR if all checks pass labels Oct 16, 2024
@niruiyu niruiyu added the push Auto push patch series in PR if all checks pass label Dec 17, 2024
@hongbin123 hongbin123 force-pushed the add_fspiwrapper_peim branch 3 times, most recently from 76f381c to 5cffc3e Compare December 18, 2024 00:59
@liyi77
Copy link
Contributor

liyi77 commented Dec 20, 2024

@mergify rebase

@hongbin123 @liyi77
IntelFsp2WrapperPkg/FspiWrapperPeim : FSP-I wrapper PEIM entrypoint
b90033c 
FSP-I wrapper PEIM entrypoint is for including FSP-SMM code for both
dispatch and API mode.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Chasel Chiu <[email protected]>
Cc: Nate DeSimone <[email protected]>
Cc: Duggapu Chinni B <[email protected]>
Cc: Chen Gang C <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Ted Kuo <[email protected]>
Cc: Ashraf Ali S <[email protected]>
Cc: Ray Ni <[email protected]>
@hongbin123 @liyi77
IntelFsp2WrapperPkg/FspiWrapperPeim : Support dispatch mode
edfae38 
Add FSP-SMM code for dispatch mode.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Chasel Chiu <[email protected]>
Cc: Nate DeSimone <[email protected]>
Cc: Duggapu Chinni B <[email protected]>
Cc: Chen Gang C <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Ted Kuo <[email protected]>
Cc: Ashraf Ali S <[email protected]>
Cc: Ray Ni <[email protected]>
@hongbin123 @liyi77
IntelFsp2Pkg : Add fsp status code for fspsmm init
d810ee5 
Add fsp status code for fspsmm init interface.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Chasel Chiu <[email protected]>
Cc: Nate DeSimone <[email protected]>
Cc: Duggapu Chinni B <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Ted Kuo <[email protected]>
Cc: Ashraf Ali S <[email protected]>
Cc: Ray Ni <[email protected]>
@hongbin123 @liyi77
IntelFsp2WrapperPkg/FspiWrapperPeim : Support API mode
fb1e87d 
Add fspsmm init interface for API mode.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Chasel Chiu <[email protected]>
Cc: Nate DeSimone <[email protected]>
Cc: Duggapu Chinni B <[email protected]>
Cc: Chen Gang C <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Ted Kuo <[email protected]>
Cc: Ashraf Ali S <[email protected]>
Cc: Ray Ni <[email protected]>
@hongbin123 @liyi77
IntelFsp2WrapperPkg/FspiWrapperPeim : Support FSP-I measurement
faca4ae 
Add code to support FSP-I binary measurement.

Signed-off-by: Hongbin1 Zhang <[email protected]>
Cc: Chasel Chiu <[email protected]>
Cc: Nate DeSimone <[email protected]>
Cc: Duggapu Chinni B <[email protected]>
Cc: Chen Gang C <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Ted Kuo <[email protected]>
Cc: Ashraf Ali S <[email protected]>
Cc: Ray Ni <[email protected]>
Cc: Jiewen Yao <[email protected]>
@mergify Mergify
Copy link

mergify bot commented Dec 20, 2024

rebase

✅ Branch has been successfully rebased

@liyi77 liyi77 force-pushed the add_fspiwrapper_peim branch from 215e34f to faca4ae Compare December 20, 2024 02:36
@mergify mergify bot mentioned this pull request Dec 20, 2024
Merged
3 tasks
@hongbin123
Copy link
Contributor Author

This PR could not be merged due to git issue.
#9406 was created to replace this

@hongbin123 hongbin123 closed this Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AshrafAliS AshrafAliS AshrafAliS approved these changes

@tedkuo1 tedkuo1 tedkuo1 approved these changes

@ChaselChiu ChaselChiu ChaselChiu approved these changes

@niruiyu niruiyu niruiyu approved these changes

@jiaxinwu jiaxinwu jiaxinwu approved these changes

@cbduggap cbduggap Awaiting requested review from cbduggap

@chengangc chengangc Awaiting requested review from chengangc

@lzeng14 lzeng14 Awaiting requested review from lzeng14

@nate-desimone nate-desimone Awaiting requested review from nate-desimone

Assignees
No one assigned
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@hongbin123 @liyi77 @jiaxinwu @niruiyu @AshrafAliS @ChaselChiu @tedkuo1