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

Trim down xtensa crates #2357

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

Trim down xtensa crates #2357

wants to merge 12 commits into from

Conversation

bugadani
Copy link
Contributor

@bugadani bugadani commented Oct 16, 2024

This PR removes some legacy code from the xtensa crates.

@MabezDev
Copy link
Member

MabezDev commented Nov 1, 2024

esp-rs/esp-pacs#282 has been merged, I think we can get this ready now.

@bugadani
Copy link
Contributor Author

bugadani commented Nov 4, 2024

What should we do about esp_hal::sync? cortex-m and riscv define their own critical sections, so if we can somehow configure a core count for xtensa-lx we might do the same and move the impl there.

Or should we just provide a single-core feature and implemet a single-core critical section in xtensa-lx?

I'm not entirely sure how far we want to generalize this crate at this moment. I'm also not sure whether that question is in scope of this PR :)

@bugadani

This comment was marked as resolved.

@bugadani

This comment was marked as resolved.

@bugadani bugadani added the skip-changelog No changelog modification needed label Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No changelog modification needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants