Update ESP32P4 defconfig with Wi-Fi and LWIP performance settings #323
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added configuration options for Wi-Fi and LWIP buffers.
The thread you've mentioned is specific to SP32-P4-Function-EV-Board product that uses ESP32-C6 as a slave Wi-Fi and BLE co-processor However, ESP32-C5 is also in use by ESP32-P4 boards to provide Wi-Fi and BLE features
Limited RAM of ESP32-C5 is known to cause issues. One known example is XiaoZhi Voice Assistant https://github.com/78/xiaozhi-esp32 The buffers has to be reduced as follows to fix the boot loop My suggestion is to revert this PR until the Arduino Core will distinguish ESP32-C5 slave from ESP32-C6 one and adjust Wi-Fi buffers correspondingly ! |
|
@lyusupov unless we can control those at runtime, we will not have a way to change them based on the slave chip, so we either go with the lower values to cover both chips, or we keep what is default |
This ESP-HOSTED commit that was referenced in this PR (see above) contains this 
This commit contains the following values for ESP32-C5 slave : 
For ESP32-C61 slave: 
Note Regardless of what particular approach will be chosen - it makes sense to revert this commit 7685397 before it will break next Release of the Arduino Core. |
|
@Jason2866 any comments? |
|
@me-no-dev Fine with changes which works for all slave MCUs |
|
@lyusupov @Jason2866 here it is: #324 |
Current Arduino Core and Libraries still use ESP-HOSTED version 2.0.12 which is 3+ months old 
This is location of 2.0.12 in the Espressif registry https://components.espressif.com/components/espressif/esp_hosted/versions/2.0.12/readme?language=en When the Arduino Core 3.3.2 for ESP32-P4 with ESP-HOSTED component 2.0.12 successfully boots up on ESP32-C5 slave, it produces the log You can see there what particular Wi-Fi buffers were successfully allocated by ESP-HOSTED 2.0.12 on the C5 slave 
From my experience with XiaoZhi assistant, when the C5 SLAVE allocates less memory than P4 expects to get - ASSERT() on the P4 HOST will happen followed by executing a panic() procedure.... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
enhance wifi performance by using recommended settings from espressif/esp-hosted-mcu#109 (comment)
@me-no-dev