-
Notifications
You must be signed in to change notification settings - Fork 7.6k
M5CoreS3 board and partitions update #8274
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
Conversation
LGTM :) |
boards.txt change went wrong... shows 23k lines changed |
@me-no-dev it's just github being github :-) link to clean diff |
my 2 cents: if you require more than 2 OTA partitions, you are a very special case and should have the partition scheme in your sketch folder. Our code assumes 2 such partitions and does not allow for APP switching for example |
maybe your editor changed some whitespaces? I do not think that github is at fault here |
The editor is github webapp, I edited the boards.txt from the browser and pasted the M5CoreS3 code block inside the existing flow. To my knowledge nothing else but the M5CoreS3 board block was modified in the file. I've repeated the operation on the patch-6 branch but I can't edit the source branch for this PR, can someone with admin permissions do it, or should I close and submit a new PR from that branch? |
@me-no-dev thanks for your input!
This is for a general use case and may eventually be extended to all M5Stack devices that come with 16MB flash and have an attached display.
It's still missing a custom bootloader to load the factory app on factory reset, but app switching is somehow possible with additional code. So far it can boot from any of the app partitions (including the factory) provided there's some binary flashed there. |
resubmitted as #8276 |
followup to espressif#8161 espressif#8274 espressif#8276
followup to #8161
the new custom partitions layout is open for discussions.
link to clean diff