Update several dependencies to their latest versions#56
Update several dependencies to their latest versions#56ivmarkov merged 4 commits intoesp-rs:mainfrom
Conversation
b5491ef to
25b5251
Compare
3bca3bd to
ed4bc50
Compare
There was a problem hiding this comment.
Pull request overview
Updates the ESP 802.15.4 radio integration and default radio configuration to improve OpenThread stability (notably around RX while idle and TX failure reporting), and adjusts the ESP example build to use an updated esp-hal/esp-radio source.
Changes:
- Change default radio config to
rx_when_idle = trueand TX power to20dBm. - Update ESP radio driver integration to handle TX failure callbacks and attempt to surface received ACK frames.
- Patch ESP example dependencies to pull
esp-hal/esp-radiofrom a git branch containing the updated 802.15.4 driver.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
openthread/src/radio.rs |
Updates default Config values (TX power, rx-when-idle). |
openthread/src/esp.rs |
Adds TX-failed callback handling, uses CCA param, and tries to extract ACK + RSSI. |
examples/esp/Cargo.toml |
Patches ESP-related crates to a git branch for updated 802.15.4 driver behavior. |
examples/esp/Cargo.lock |
Locks updated dependency graph consistent with the patched ESP crates. |
Cargo.toml |
Adds a workspace-level patch for esp-hal to a specific git revision. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1c6b361 to
62d9668
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
49c1f26 to
9cadacd
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
503b6be to
0a1f728
Compare
b57aea7 to
b87ffe9
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
=== ESP-specific:
esp-halfrommain(rev e156b5453a8adc123b7a8b6aa08e0698ec87dfe8) which does have an updated 802.15.4 driver (Update the 802.15.4 driver to match the ESP-IDF C one w.r.t. behavior esp-hal#5006)Overall, addresses most of #53
=== NRF-specific:
=== edge-nal-specific: