Skip to content

Add defmt feature for riscv crate - #423

Merged
romancardenas merged 5 commits into
rust-embedded:masterfrom
TechnoPorg:push-kmmvsqlxspru
Oct 5, 2026
Merged

romancardenas merged 5 commits into
rust-embedded:masterfrom
TechnoPorg:push-kmmvsqlxspru

Conversation

@TechnoPorg

Copy link
Copy Markdown
Contributor

This is a follow-up to #335 which adds the same feature to riscv and implements the trait on commonly used enums.

@TechnoPorg
TechnoPorg requested a review from a team as a code owner October 2, 2026 16:25
@TechnoPorg

Copy link
Copy Markdown
Contributor Author

One other thing I've run into in my error logging is the lack of defmt support for riscv_types::result::Error. Should I add that here or in a separate PR?

@romancardenas

Copy link
Copy Markdown
Contributor

I think you can include It in this same PR

@TechnoPorg

Copy link
Copy Markdown
Contributor Author

Done! This is ready for review now 😄

@romancardenas romancardenas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks! I added 3 comments that all address to the same issue.

Comment thread riscv-rt/Cargo.toml Outdated
Comment thread riscv-types/Cargo.toml Outdated
Comment thread riscv/Cargo.toml Outdated

@romancardenas romancardenas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@romancardenas
romancardenas added this pull request to the merge queue Oct 5, 2026
Merged via the queue into rust-embedded:master with commit 110d3f7 Oct 5, 2026
165 checks passed
@TechnoPorg
TechnoPorg deleted the push-kmmvsqlxspru branch October 5, 2026 13:28
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.

2 participants