Skip to content

Pico 2 W OTA Update Example #559

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Pico 2 W OTA Update Example #559

wants to merge 18 commits into from

Conversation

will-v-pi
Copy link
Contributor

Adds an example of Pico 2 W OTA update

Requires raspberrypi/pico-sdk#1969

@will-v-pi will-v-pi added this to the 2.0.1 milestone Nov 12, 2024
@kilograham kilograham removed this from the 2.1.0 milestone Nov 22, 2024
@will-v-pi will-v-pi added this to the 2.2.0 milestone Apr 7, 2025
@andygpz11
Copy link
Contributor

I think the README.md will need to be updated to include this example.

@will-v-pi will-v-pi modified the milestones: 2.2.0, 2.1.2 Jul 15, 2025
@will-v-pi will-v-pi marked this pull request as ready for review July 18, 2025 08:42
@peterharperuk
Copy link
Contributor

1/ How do I know this is really working? I really think this needs a readme with the example to tell someone how to run the example to perform an update etc. I see it downloading the update and rebooting but it doesn't change anything.
2/ I'm not sure why the python script is in the python_test_tcp folder (not really sure I remember why we have that folder). If we're going to stick all python scripts in there we should probably rename the folder. Or just stick the script with the ota_update
source?
3/ Do we need poll and background versions of the firmware uf2s? Maybe it would be less confusing to just build one version of this example (just use background threadsafe?)

kilograham
kilograham reviewed Jul 20, 2025
View reviewed changes
@will-v-pi
Copy link
Contributor Author

Thanks - I've added a readme with details of how to use it, moved the python script, and removed the poll version

@peterharperuk
Copy link
Contributor

Thanks - works nicely following the instructions

peterharperuk
peterharperuk previously approved these changes Jul 21, 2025
View reviewed changes
kilograham
kilograham reviewed Jul 23, 2025
View reviewed changes
pico_w/wifi/CMakeLists.txt
else()
add_subdirectory_exclude_platforms(tls_client)
add_subdirectory_exclude_platforms(ota_update rp2040)
Copy link
Contributor

Choose a reason for hiding this comment

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

note ota_update doesn't seem to declare pico_mbedtls as a dependency; if it needs it it probably should

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That check is being used similar to the encrypted examples, to check that picotool has been compiled with mbedtls - it’s not directly used by the example

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(See bootloaders/CMakeLists.txt)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kilograham kilograham kilograham left review comments

@peterharperuk peterharperuk peterharperuk left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.2.0
Development

Successfully merging this pull request may close these issues.
4 participants
@will-v-pi @andygpz11 @peterharperuk @kilograham
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy