You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "acassis (via GitHub)" <gi...@apache.org> on 2023/02/08 15:12:49 UTC
[GitHub] [nuttx] acassis opened a new issue, #8477: ESP32 BLE driver is crashing
acassis opened a new issue, #8477:
URL: https://github.com/apache/nuttx/issues/8477
After the PR https://github.com/apache/nuttx/pull/7160 ESP32 BLE driver is crashing.
@gustavonihei figure-out that the issue is caused by the change with the source pointer `_data_start_btdm_rom`
And he come up with this small fix:
```
--- a/arch/xtensa/src/esp32/esp32_ble_adapter.c
+++ b/arch/xtensa/src/esp32/esp32_ble_adapter.c
@@ -451,8 +451,8 @@ extern uint8_t _bss_start_btdm[];
extern uint8_t _bss_end_btdm[];
extern uint8_t _data_start_btdm[];
extern uint8_t _data_end_btdm[];
-extern const uint8_t _data_start_btdm_rom[];
-extern const uint8_t _data_end_btdm_rom[];
+extern uint32_t _data_start_btdm_rom;
+extern uint32_t _data_end_btdm_rom;
extern uint8_t _bt_bss_start[];
extern uint8_t _bt_bss_end[];
@@ -2027,11 +2027,11 @@ static void btdm_controller_mem_init(void)
/* initialise .data section */
- memcpy(_data_start_btdm, _data_start_btdm_rom,
+ memcpy(_data_start_btdm, (void *)_data_start_btdm_rom,
_data_end_btdm - _data_start_btdm);
wlinfo(".data initialise [0x%08x] <== [0x%08x]\n",
- (uint32_t)_data_start_btdm, (uint32_t)_data_start_btdm_rom);
+ (uint32_t)_data_start_btdm, _data_start_btdm_rom);
/* initial em, .bss section */
```
@xiaoxiang781216 could you please helps us to understand what is going on after that PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] acassis commented on issue #8477: ESP32 BLE driver is crashing
Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on issue #8477:
URL: https://github.com/apache/nuttx/issues/8477#issuecomment-1422759012
You can test it on NuttX this way:
```
nsh> bt bnep0 scan start
nsh> bt bnep0 scan stop
nsh> bt bnep0 scan get
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] xiaoxiang781216 commented on issue #8477: ESP32 BLE driver is crashing
Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 commented on issue #8477:
URL: https://github.com/apache/nuttx/issues/8477#issuecomment-1422813037
could you print out all memcpy argument before and after revert?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [nuttx] btashton closed issue #8477: ESP32 BLE driver is crashing
Posted by "btashton (via GitHub)" <gi...@apache.org>.
btashton closed issue #8477: ESP32 BLE driver is crashing
URL: https://github.com/apache/nuttx/issues/8477
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org