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

Update firmware and modernize generation scripts #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

i509VCB
Copy link

@i509VCB i509VCB commented Sep 11, 2024

By commit:

  1. Allow my hardware setup to work - I use a 5340dk with a 7002ek shield.
  2. Modernize gen.py to automatically fetch headers for bindings and firmware blobs
  3. Update the code to adjust for the new firmware version. This gets to the current state of the repo where the the init event is sent to the host and watch dog irqs get handled.

Obviously still more to go, but I want to get some feedback on the firmware update related changes before I try to get scanning to work.

This makes updating the firmware headers and firmware binaries easier since running the script automatically fetches and bindgens the headers along with the new fireware blobs
And the related changes due to new firmware
@i509VCB
Copy link
Author

i509VCB commented Sep 18, 2024

@Dirbaio poke?

Minimal changes for now as I am probably going to have a few more pull requests after this one and would prefer to not dump a 10k patch on you at once.

@diondokter
Copy link
Collaborator

diondokter commented Sep 18, 2024

@Dirbaio poke?

Minimal changes for now as I am probably going to have a few more pull requests after this one and would prefer to not dump a 10k patch on you at once.

Hey there! So Dario can have his own opinion of course. But I'd say that making one huge PR is better than several smaller ones.

This project doesn't work yet and it's not used by anyone. So unless you make it work, all the work in the middle is not that relevant and worth having an opinion on.

Dario has spent a lot of time on this and I have too and both of us have given up.

Now, if you come with a mega PR, but you're now able to connect to an AP or even ping a device in the network, then hell yeah, I wanna see how you did that!

Snipe me hard enough and I'll grab my DK from my endless drawer of devboards again and test it out and help out landing the PR.

But until then... You know I/we stopped working on it for a reason...

You get that hopefully!

Btw, maybe Dario can just give you access to the repo, just like he gave it to me...

@i509VCB
Copy link
Author

i509VCB commented Sep 19, 2024

I guess on that topic I'll continue working until something more interesting is done.

@Dirbaio
Copy link
Member

Dirbaio commented Sep 24, 2024

@diondokter is right, there's no need to be afraid of breaking anything when nothing works at all :D

@i509VCB I've added you to the repo, feel free to self-merge PRs or push to main directly. I would love to help but my available time is zero, so I'm excited to see this carried forward!

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.

3 participants