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

limit default partition size #398

Merged
merged 1 commit into from
May 29, 2023
Merged

limit default partition size #398

merged 1 commit into from
May 29, 2023

Conversation

MabezDev
Copy link
Member

@MabezDev MabezDev commented May 4, 2023

Some chips can have more than 16MB of flash, mine has 32MB. Historically espflash failed to detect this flash size (and defaulted to 4MB) but now it can. Without this change, the board will boot loop because the esp idf bootloader max size is only 16MB.

Closes #393

Some chips can have more than 16MB of flash, mine as 32MB. Historically
espflash failed to detect this flash size but now can. Without this
change the board will boot loop because the esp idf bootloader max size
is only 16MB.
Copy link
Member

@SergioGasquez SergioGasquez left a comment

Choose a reason for hiding this comment

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

LGTM! But, cant test it because I dont think I have a >16MB flash chip

@maxwase
Copy link
Contributor

maxwase commented May 7, 2023

Tested this on my ESP32-S3 32MB flash, works fine
I had a bootloop only when I explicitly set >16MB partition, maybe we should add a warning for it?
In 2 paces: flash and partition-table sub-commands

Copy link
Member

@jessebraham jessebraham left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jessebraham jessebraham merged commit 33e32c1 into main May 29, 2023
@jessebraham jessebraham deleted the limit-partition-size branch July 13, 2023 18:14
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.

Flashing with ESP32-S3 with Octal flash flashes an unbootable image
4 participants