-
Notifications
You must be signed in to change notification settings - Fork 8.4k
boards: esp32c3: Increase heap when BT is enabled #80180
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
Conversation
The default heap is not enough not even to initialize the Bluetooth stack. Increase the memory heap when Bluetooth is selected to be able at least to do basic things. Signed-off-by: Flavio Ceolin <[email protected]>
|
@ceolin hum, which revision you have tested? Just tested with latest main and I get both *** Booting Zephyr OS build (tainted) v3.7.0-4826-g372b3a9b8da2 ***
[00:00:00.046,000] <inf> fs_nvs: 8 Sectors of 4096 bytes
[00:00:00.046,000] <inf> fs_nvs: alloc wra: 0, fe8
[00:00:00.046,000] <inf> fs_nvs: data wra: 0, 0
[00:00:00.047,000] <inf> esp32_bt_adapter: BT controller compile version [963cad4]
[00:00:00.442,000] <inf> bt_hci_core: No ID address. App must call settings_load()
Bluetooth initialized
[00:00:00.447,000] <inf> bt_hci_core: Identity: 64:E8:33:98:6D:3C (public)
[00:00:00.448,000] <inf> bt_hci_core: HCI: version 5.0 (0x09) revision 0x0016, manufacturer 0x02e5
[00:00:00.448,000] <inf> bt_hci_core: LMP: version 5.0 (0x09) subver 0x0016
Advertising successfully started
Indicate VND attr 0x3c0306d4 (UUID 12345678-1234-5678-1234-56789abcdef1) |
|
Curious, I am using Building with: That is the error I get: The controller revision is the same too. Btw, if I enable boot loader protection I get: Looking the HAL, the initialization failed in |
|
@ceolin I tried the same commands you did. Still got it working. Would you mind sharing the initial log output so I can get more info about the esp32c3 memory stuff? Is |
|
|
||
| config HEAP_MEM_POOL_ADD_SIZE_BOARD | ||
| int | ||
| default 32768 if BT |
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.
BT and WiFi should be using the runtime heap, so increasing the system heap doesn't make sense now.
|
It was indeed fixed. Thank you guys for the support. |
The default heap is not enough not even to initialize the Bluetooth stack. Increase the memory heap when Bluetooth is selected to be able at least to do basic things.