-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 ESP-IDF to 5.2.1 #9071
Update ESP-IDF to 5.2.1 #9071
Conversation
6ea19b2
to
bd5d693
Compare
I tried this on my memento. It runs but the camera stops working. I'll look tomorrow if @jepler doesn't get to it first. |
Still draft because it needs: adafruit/esp32-camera#8 |
Just out of curiosity, why is 120m unstable? |
From here:
|
Ok, @dhalbert this is ready for review. |
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.
Port waveshare_esp32s2_pico
still has CIRCUITPY_ESP_PSRAM_FREQ = 120m
.
I ran a string search just in case there are others, but no.
I left that in because it is marked as quad not octal. |
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.
OK, looks good! Looking forward to 9.1.0-beta.0!
This updates the underlying I2C API and allocates memory. So, I switched busio.I2C to using a finalizer for reset instead of bulk reset.