Skip to content

Conversation

felipebalbi
Copy link
Contributor

DebugPort might be useful for some uefi applications.

@felipebalbi
Copy link
Contributor Author

This is my first time contributing to a rust project. I'll gladly update anything that needs to be updated. This has been tested with SbsaQemu platform after enabling DebugPortDxe driver

@felipebalbi felipebalbi force-pushed the add-debug-port-protocol branch from b2a731f to a1acd47 Compare April 5, 2023 12:24
Copy link
Member

@nicholasbishop nicholasbishop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Left a few comments

@felipebalbi felipebalbi force-pushed the add-debug-port-protocol branch from a1acd47 to 36b4b64 Compare April 6, 2023 07:45
@felipebalbi
Copy link
Contributor Author

@nicholasbishop the API looks a lot cleaner with your suggestions. PR updated, thanks

@felipebalbi felipebalbi force-pushed the add-debug-port-protocol branch from 36b4b64 to 0016e36 Compare April 6, 2023 07:47
@felipebalbi felipebalbi force-pushed the add-debug-port-protocol branch from 0016e36 to bc44fba Compare April 6, 2023 13:16
DebugPort might be useful for some uefi applications.
@felipebalbi felipebalbi force-pushed the add-debug-port-protocol branch from bc44fba to 6fffe57 Compare April 6, 2023 13:18
Copy link
Member

@nicholasbishop nicholasbishop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks again.

@nicholasbishop nicholasbishop merged commit 9777fd4 into rust-osdev:main Apr 6, 2023
@felipebalbi felipebalbi deleted the add-debug-port-protocol branch April 6, 2023 14:46
debug_port
.reset()
.expect("Error while resetting debug port");
let data: Vec<_> = r##"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol, nice 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants