-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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 adafruit boards tinyuf2 and partition scheme #6770
Conversation
platform.txt
Outdated
build.custom_bootloader=bootloader | ||
build.custom_partitions=partitions | ||
build.custom_bootloader= | ||
build.custom_partitions= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes the build search for variant/.bin
, expanded from "{build.variant.path}"/{build.custom_bootloader}.bin
. Maybe it should be left as is?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, this is meant to expanded to non-existed files by default in the variant/board folder so that one of the espressif default parition csv can be used if an TinyUF2 scheme isn't selected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it will still expand to non-existing if you name yours to something else. Default values are there to allow variants to add bootloader.bin and partitions.csv and be done, without having to specify anything in boards.txt. Please rename your bootloaders to something else and the defaults here will work as expected :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update PR to revert this changes and rename the bootloader/partition in the variant folder as suggested.
- add tinyuf2 suffix to variant bootloader, cvs - also update adafruit esp32s3 menu to match latest esp32s3dev one
ab8abc6
to
417f45f
Compare
Thanks @hathach |
I am the one to say thanks :) |
Description of Change
It address one of the walk-around @UnexpectedMaker need to do is renaming the tinyuf2.bin/bootloader.bin in the variant folder.Tests scenarios
Related links
@ladyada