Skip to content

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

Closed
wants to merge 4 commits into from
Closed

Conversation

tobozo
Copy link
Contributor

@tobozo tobozo commented May 31, 2023

followup to #8161

  • adjusted default build values
    1. cdc default enabled
    2. 16MB flash + partition scheme copied from Core2 default selected
  • added missing sections
  • added two custom partitions schemes to the variant folder

the new custom partitions layout is open for discussions.

link to clean diff

@imliubo
Copy link

imliubo commented May 31, 2023

LGTM :)

@me-no-dev
Copy link
Member

boards.txt change went wrong... shows 23k lines changed

@tobozo
Copy link
Contributor Author

tobozo commented May 31, 2023

@me-no-dev it's just github being github :-) link to clean diff

@me-no-dev
Copy link
Member

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

@me-no-dev
Copy link
Member

@me-no-dev it's just github being github :-) link to clean diff

maybe your editor changed some whitespaces? I do not think that github is at fault here

@tobozo
Copy link
Contributor Author

tobozo commented May 31, 2023

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?

@tobozo
Copy link
Contributor Author

tobozo commented May 31, 2023

@me-no-dev thanks for your input!

if you require more than 2 OTA partitions, you are a very special case and should have the partition scheme in your sketch folder.

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.

Our code assumes 2 such partitions and does not allow for APP switching for example

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.

@tobozo tobozo closed this May 31, 2023
@tobozo
Copy link
Contributor Author

tobozo commented Jun 1, 2023

resubmitted as #8276

tobozo added a commit to tobozo/arduino-esp32 that referenced this pull request Sep 11, 2023
me-no-dev pushed a commit that referenced this pull request Sep 15, 2023
* M5CoreS3

followup to #8161 #8274 #8276

* Add files via upload

* Fix partition names
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.

3 participants