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

Use cyw43 FW file size instead of magic numbers #3575

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

9names
Copy link
Contributor

@9names 9names commented Nov 26, 2024

This should remove the possibility of the firmware slice length being incorrect when the firmware is updated next time.
Would be nice to also have a test that runs the bluetooth firmware so we can have an assert that the firmware blobs don't overlap - I don't expect this to be a serious concern though.
Resolves #3470

@Dirbaio
Copy link
Member

Dirbaio commented Dec 2, 2024

bender run

@Dirbaio
Copy link
Member

Dirbaio commented Dec 2, 2024

cyw43 HIL test is failing for some reason.

Also I don't remember whether the pico w in CI has the latest firmware flashed. if the size doesn't match it's because it has an older one, in which case changing it will break it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rp cyw43 firmware flashing and from_raw_parts len parameter
2 participants