edk2 icon indicating copy to clipboard operation
edk2 copied to clipboard

Published 20 hours ago • verified publisher icon tianocore

Create a new MmCommunicationDxe driver

Open td36 opened this issue 1 year ago • 6 comments

Description

StandaloneMmPkg: Add a new MmCommunicationDxe driver

Add a new MmCommunicationDxe driver. The driver is to: 1.Install gEfiMmCommunication2ProtocolGuid 2.Install gEfiMmCommunicationProtocolGuid 3.Create the notifications of some protocols and event that the Standalone Mm requires

How This Was Tested

Tested in Intel internal platform

td36 avatar Jun 11 '24 03:06 td36

Hi @jiaxinwu , @niruiyu , could you help to review the PR to create MmCommunicationDxe driver?

td36 avatar Jun 11 '24 03:06 td36

Please provide more background on why this new driver is required.

mdkinney avatar Jun 15 '24 00:06 mdkinney

Some of the commit messages have more than one Signed-off-by. There should only be one of those tags in each commit. If more than one author contributed to the change, then additional Co-authored-by tags can be used.

mdkinney avatar Jun 15 '24 00:06 mdkinney

@mdkinney , I guess it's a "draft" patch aiming to do a CI check. There will be a complete set of patches in one PR to enable standalone MM for X86.

niruiyu avatar Jun 17 '24 02:06 niruiyu

Some of the commit messages have more than one Signed-off-by. There should only be one of those tags in each commit. If more than one author contributed to the change, then additional Co-authored-by tags can be used.

Thanks for the comments Mike. It's a draft PR for CI check and internal review.

td36 avatar Jun 17 '24 02:06 td36

PR can not be merged due to conflict. Please rebase and resubmit

mergify[bot] avatar Jun 27 '24 10:06 mergify[bot]

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

github-actions[bot] avatar Sep 07 '24 23:09 github-actions[bot]

PR can not be merged due to conflict. Please rebase and resubmit

mergify[bot] avatar Sep 07 '24 23:09 mergify[bot]