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

The Ieee802154 driver does not use the PeripheralRef metaphor #3238

Open
ivmarkov opened this issue Mar 8, 2025 · 2 comments
Open

The Ieee802154 driver does not use the PeripheralRef metaphor #3238

ivmarkov opened this issue Mar 8, 2025 · 2 comments
Labels
bug Something isn't working status:needs-attention This should be prioritized

Comments

@ivmarkov
Copy link

ivmarkov commented Mar 8, 2025

Bug description

Since the driver takes the IEEE802154 and the RADIO_CLK peripherals by value, it is not really possible to mutably borrow the peripherals into the radio and then release them once the radio is dropped.

I believe this is just an ommission, as the IEEE802154 radio is not used that widely yet.

Expected behavior

Use impl Peripheral<P = IEEE802154> + 'd, ...` like everywhere else. :)

Environment

  • Target device: esp32c6, esp32h2
  • Crate name and version: main branch
@ivmarkov ivmarkov added bug Something isn't working status:needs-attention This should be prioritized labels Mar 8, 2025
@github-project-automation github-project-automation bot moved this to Todo in esp-rs Mar 8, 2025
@bugadani
Copy link
Contributor

bugadani commented Mar 8, 2025

We're simply not there yet

@ivmarkov
Copy link
Author

ivmarkov commented Mar 8, 2025

I know, and IEEE802154::steal() is a good workaround in the meantime.

The bigger trouble is #3239

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status:needs-attention This should be prioritized
Projects
Status: Todo
Development

No branches or pull requests

2 participants