Skip to content

add defmt-itm#335

Merged
bors[bot] merged 5 commits intomainfrom
itm
Jan 12, 2021
Merged

add defmt-itm#335
bors[bot] merged 5 commits intomainfrom
itm

Conversation

@japaric
Copy link
Copy Markdown
Member

@japaric japaric commented Jan 12, 2021

this is a new defmt transport crate

static ENABLED: AtomicBool = AtomicBool::new(false);

/// Enables defmt logging over the ITM stimulus port 0
pub fn enable(itm: ITM) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

we may want to make the stimulus port configurable in the feature that can be added backwards compatibility by adding a second function with signature fn(ITM, port_number: u8)

Comment thread firmware/defmt-itm/src/lib.rs
Comment thread firmware/defmt-itm/src/lib.rs Outdated
Comment thread firmware/defmt-itm/src/lib.rs
Comment thread firmware/defmt-itm/src/lib.rs Outdated
japaric and others added 2 commits January 12, 2021 15:35
Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>
@jonas-schievink
Copy link
Copy Markdown
Contributor

bors r+

@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Jan 12, 2021

Build succeeded:

@bors bors Bot merged commit 5dc83b2 into main Jan 12, 2021
@bors bors Bot deleted the itm branch January 12, 2021 15:18
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