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

Increased HCI_RX_BUF_QUEUE_SIZE to 16 to Resolve OTA DFU #90

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chrisingis
Copy link

@chrisingis chrisingis commented Dec 13, 2019

Using a Segger J-Link, I traced the following error, thrown by hci_mem_pool_rx_produce():
NRF_ERROR_NO_MEM due to m_rx_buffer_queue.free_window_count == 0

In that trace, the value of length, being passed to hci_mem_pool_rx_produce() was 14. Alas, increasing HCI_RX_BUF_QUEUE_SIZE resolved the issue. I have quite a handful of these boards, and the error was consistently resolved across the board in both my iOS and Android app.

nRF52 Device Information:

MCU Variant: nRF52840	0x41414330
Memory (FLASH): 1024 KB
Memory (RAM): 256 KB
ARM Cortex-M4 Processor Revision: v1
MDK Version: v8.24.1
TinyUSB Version: v0.5.0
LittleFS Version: v1.1
Bootloader/DFU Revision: s140 6.1.1
SoftDevice Version: v6.1.1
BSP Library: v0.14.0

Desktop Platform Information: macOS 10.14, as well as 10.15
Mobile Devices Used: An iPhone 8 (iOS 11 + 12) as well as an iPhone X (iOS 12 + 13)

Using a Segger J-Link, I traced the following error, thrown by hci_mem_pool_rx_produce():
"NRF_ERROR_NO_MEM due to m_rx_buffer_queue.free_window_count == 0"
In that trace, the value of @Length, being passed into hci_mem_pool_rx_produce() was 14.

nRF52 Device Information:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
MCU Variant												= nRF52840	0x41414330
Memory (FLASH)										= 1024 KB
Memory (RAM)											= 256 KB
ARM Cortex-M4 Processor Revision	= v1
MDK Version												= v8.24.1
TinyUSB Version										= v0.5.0
LittleFS Version									= v1.1
Bootloader/DFU Revision						= s140 6.1.1
SoftDevice Version								= v6.1.1
BSP Library												= v0.14.0
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Desktop Platform Information: macOS 10.14, as well as 10.15
Mobile Devices Used: An iPhone 8 (iOS 11 + 12) as well as an iPhone X (iOS 12 + 13)
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.

1 participant