Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Esp32 support #43

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Esp32 support #43

wants to merge 2 commits into from

Conversation

sh0rez
Copy link
Member

@sh0rez sh0rez commented Feb 4, 2022

This library already works really well on the embedded ESP32 SOC, but that platform being 32-bit means we cannot use AtomicU32.

This PR adds xtensa to the list of 32-bit architectures, so that it compiles.
It also implements Atomic for f32, so that Gauges become usable.

IMHO we really should use rust-lang/rust#32976 once available, but that is for another day.

Signed-off-by: sh0rez <me@shorez.de>
Signed-off-by: sh0rez <me@shorez.de>
Copy link
Member

@mxinden mxinden 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 providing a patch!

Would you mind adding a changelog entry? Something along the lines of:

## [0.15.2] - unreleased

### Added

- Support for `f32` `Gauge` and `Counter`. See [PR xx].
- Support for `xtensa` architecture. See [PR xx].

And bump the patch version in Cargo.toml?

Also does Rust support cross compilation for xtensa? If so, want to add it to the matrix in our CI config?

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.

None yet

2 participants