Skip to content

Add a no_std target, an esp32c3 node - #22

Merged
mcbridejc merged 4 commits into
mcbridejc:mainfrom
etiennedm:example-no-std-esp-node
Sep 4, 2025
Merged

Add a no_std target, an esp32c3 node#22
mcbridejc merged 4 commits into
mcbridejc:mainfrom
etiennedm:example-no-std-esp-node

Conversation

@etiennedm

Copy link
Copy Markdown
Contributor

Here is a working esp32c3 node, that is doing nothing but should appear on a scan with the CLI.

⚠️ Do note I had to bump the rust version to 1.88, I don't know if that's OK.

I could not get it to play nicely with the workspace for at least 2 reasons:

  1. critical-section conflicting features
  2. .cargo/config.toml would be ignored in esp-node when compiling from the root dir (expected behaviour, see cargo doc)

@mcbridejc

Copy link
Copy Markdown
Owner

Thanks!

We should be able to just bump the rust version for the example app, if it is something in the example that requires it, right?

@etiennedm

Copy link
Copy Markdown
Contributor Author

Oh yes you're right, that was probably from when I was trying to get it to work with the workspace. Let me try.

@etiennedm
etiennedm force-pushed the example-no-std-esp-node branch from a0d228a to 5bdf80c Compare September 3, 2025 16:45
@etiennedm

Copy link
Copy Markdown
Contributor Author

Yep that works, and avoids clippy warnings 🤷

@mcbridejc

Copy link
Copy Markdown
Owner

You can opt out of the global workspace by putting [workspace] in esp-node/Cargo.toml. Seems better to me than keeping a list of exluded projects at the top.

@etiennedm
etiennedm force-pushed the example-no-std-esp-node branch from cccf9b3 to 4c226b3 Compare September 3, 2025 16:52
Comment thread examples/esp-node/src/main.rs Outdated
Comment thread examples/esp-node/zencan_config.toml Outdated
@etiennedm
etiennedm force-pushed the example-no-std-esp-node branch from 4c226b3 to a328385 Compare September 4, 2025 10:02
@mcbridejc
mcbridejc merged commit 183a2d8 into mcbridejc:main Sep 4, 2025
1 check passed
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