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

Remove binary info header, was crashing picotool #831

Merged
merged 1 commit into from
Sep 3, 2022
Merged

Conversation

earlephilhower
Copy link
Owner

Because OTA has changed the flash map from standard, picotool ends up
crashing or hanging while trying to operate on the current built files.

Remove the binary_info calls and structures completely to avoid any
issue.

Fixes #803

Because OTA has changed the flash map from standard, picotool ends up
crashing or hanging while trying to operate on the current built files.

Remove the binary_info calls and structures completely to avoid any
issue.

Fixes #803
@earlephilhower earlephilhower changed the title Remove bianry info header, was crashing picotool Remove binary info header, was crashing picotool Sep 3, 2022
@fatherseye
Copy link

Sorry it took a while for me to work out how to get that test release into platform IO. I'm sure that there must be an easier way but in the end I had to use command line git.

cd $home/.platformio/packages/framework-arduinopico
git reset --hard origin/noinfo

Which seems to do the trick from what I could tell of the files vs the changes logged int github. I seem to remember reading something about adding @ branch at the end of the platformio entry but it didn't seem to work for the board_build.core entry.

I removed the "upload_protocol = mbed" from platformio.ini. Did a clean and rebuild.
Then I reset a rpi pico to an old version uf2 firmware using bootsel/copy file. Then finally tried that demo program again with all Serial operation commented out. Uploaded multiple times without any problems!!! Hurrah!

Well done that was an impressive turn around.

For for completeness I did it again with the Serial commands uncommented. Again, all good.

@earlephilhower earlephilhower merged commit 36d5ceb into master Sep 3, 2022
@earlephilhower earlephilhower deleted the noinfo branch September 3, 2022 06:17
@earlephilhower
Copy link
Owner Author

Thanks for the feedback. I'll do a point release tomorrow with this, since P.IO uploads seem busted now...

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.

PlatformIO download stops working after 1st download from platformIO
2 participants