Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Use esp-wifi-sys in place of blobs/binary module, use PAC instead of RAL #25

Merged
merged 5 commits into from
May 14, 2024

Conversation

jessebraham
Copy link
Member

@jessebraham jessebraham commented May 9, 2024

This is a pretty substantial update, but the code itself has not changed much. This still relies on a couple branches from my forks, so is a draft for now until those PRs get merged. This is all untested but, in theory, should work; once the relevant dependencies are updated I will do a full round of testing.

Essentially the changes boil down to:

  • Remove the binary blobs and headers, use esp-wifi-sys instead of generating our own bindings
  • Remove the RAL, use the registers which are now present in the PAC for IEEE802154
  • Make any other updates required to get all examples building again

Once this gets merged, I will add the esp-ieee802154 package to the esp-hal repository and we can archive this repo.

@jessebraham jessebraham marked this pull request as ready for review May 13, 2024 14:34
@jessebraham jessebraham requested a review from bjoernQ May 13, 2024 14:34
Copy link
Collaborator

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Cannot really test currently but the changes look good and straight-forward enough

@bjoernQ bjoernQ merged commit 5822912 into esp-rs:main May 14, 2024
2 checks passed
@jessebraham jessebraham deleted the feature/overhaul branch May 14, 2024 13:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants