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

Nordic DFU OTA Compatibility #80

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

Conversation

zwells
Copy link

@zwells zwells commented Sep 13, 2019

HCI_RX_BUF_QUEUE_SIZE must be 16 for Nordic DFU to prevent overflow

HCI_RX_BUF_QUEUE_SIZE must be 16 for Nordic DFU to prevent overflow
@zwells
Copy link
Author

zwells commented Sep 13, 2019

Solves Issue #63

@zwells zwells changed the title Update sdk_config.h Nordic DFU OTA Compatibility Sep 13, 2019
@hathach
Copy link
Member

hathach commented Sep 14, 2019

Can you provide your set up inclduing app used, mobile hardware and os, bootloader, board and step to reproduce the issue so we could try to reproduce the issue first.

@zwells
Copy link
Author

zwells commented Sep 14, 2019

Any Arduino IDE generated zip package for Adafruit Feather nRF52832 bootloader can be given directly to nRF Toolbox, nRF Connect, or fengqiangboy/flutter-nordic-dfu (iOS and Android)

Hangs and fails on Starting DFU when HCI_RX_BUF_QUEUE_SIZE < 14

Succeeds when HCI_RX_BUF_QUEUE_SIZE >= 14

@hathach hathach added this to To do in Next Release Nov 28, 2019
@mattratto
Copy link

The issue with this is that changing the HCI_RX_BUF_QUEUE_SIZE = 16 generates the following error on make for Adafruit Feather nRF52832:
c:/program files (x86)/gnu tools arm embedded/8 2019-q3-update/bin/../lib/gcc/arm-none-eabi/8.3.1/../../../../arm-none-eabi/bin/ld.exe: region RAM overflowed with stack

This does appear to work for 52340-based boards.

@evandavey
Copy link

I required this change for OTA DFU to work using nRF Connect on Android and a Sparkfun MicroMod nRF52840.

@hyx0329 hyx0329 mentioned this pull request May 26, 2023
@ericlangel
Copy link

Same Issue here....i cant Update with NRF DFU and NRF Connect

and this issue/PR is 4 years old!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Next Release
  
To do
Development

Successfully merging this pull request may close these issues.

None yet

6 participants