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

Use flash_mode for memory_type when memory_type is not set for #80

Merged
merged 2 commits into from
Aug 22, 2022

Conversation

Jason2866
Copy link
Contributor

…ESP32, C3 and S2 when using Platformio

Replaces espressif/arduino-esp32#7141

…ESP32, C3 and S2 when using Platformio
@Jason2866
Copy link
Contributor Author

Jason2866 commented Aug 16, 2022

This change is tested and verified. Builded framework is working

@me-no-dev
Copy link
Member

Can you please replace dout with dio as dout is not available for all boards, where dio is left for compatibility.

@Jason2866
Copy link
Contributor Author

@me-no-dev Changed. I always thought dout is slowest most compatible mode

@me-no-dev
Copy link
Member

@me-no-dev Changed. I always thought dout is slowest most compatible mode

it is, but it is unnecessary on boards other than one very specific ESP32 based one. So to save on libs and bootloaders, it has been removed on S3 and probably will do for C3 and S2 in the future.

@me-no-dev me-no-dev merged commit e4fd9cf into espressif:master Aug 22, 2022
@Jason2866 Jason2866 deleted the patch-1 branch August 22, 2022 13:26
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.

2 participants