Skip to content

sdk: Implement ProtocolInterface for TimerProtocol - #1774

Merged
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:impl_protint_for_prottimer
Sep 11, 2026
Merged

sdk: Implement ProtocolInterface for TimerProtocol#1774
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:impl_protint_for_prottimer

Conversation

@makubacki

Copy link
Copy Markdown
Collaborator

Description

Associate the TimerProtocol structure with the PI-defined GUID so that the protocol can be used more safely in code.

Note: This is an improvement on its own, but it's being made to support backing the upcoming Timer Services.

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

How This Was Tested

  • cargo make all

Integration Instructions

  • N/A

@makubacki makubacki self-assigned this Sep 11, 2026
@patina-automation

patina-automation Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

✅ QEMU Validation Passed

All QEMU validation jobs completed successfully.

Note: Q35 is only built on Windows hosts (QEMU boot is disabled due to a QEMU vfat issue).

Workflow run: https://github.com/OpenDevicePartnership/patina/actions/runs/34652142922

Boot Time to EFI Shell

Platform Elapsed
ArmVirt (Linux Host) 21.2s
Q35 (Linux Host) 24.5s

Dependencies

Repository Ref
patina abd55d5
patina-dxe-core-qemu 17620b3
patina-fw-patcher c28ab7d
patina-qemu firmware v4.0.5
patina-qemu build script a8b6551

This comment was automatically generated by the Patina QEMU PR Validation Post workflow.

@github-actions github-actions Bot added the impact:non-functional Does not have a functional impact label Sep 11, 2026
@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Associate the `TimerProtocol` structure with the PI-defined GUID so
that the protocol can be used more safely in code.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki
makubacki force-pushed the impl_protint_for_prottimer branch from b726d9e to abd55d5 Compare September 11, 2026 21:50
@makubacki
makubacki enabled auto-merge (squash) September 11, 2026 21:51
@makubacki
makubacki merged commit ae7e830 into OpenDevicePartnership:main Sep 11, 2026
11 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.

3 participants