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

compatible bootloader with Adafruit nRF52 Core #23

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

gb88
Copy link

@gb88 gb88 commented Dec 12, 2023

I create a build with SDK v15.3.0 and SD 6.1.1 to be fully compatible with Adafruit nRF52 Arduino Core, i've added some configuration in the docker build and the possibilities to create the secure and unsecure build, i've tested only on nrf52840, the Adafruit nrf52 Arduino Core has beed edited to be compatible with some small linker modification, you can see it here https://github.com/gb88/Adafruit_nRF52_Arduino_for_secure_bootloader, to be fully compatible with Arduino environment a build toolchain that use the nrfutils instead of nrfutils of Adafruits needs to be integrated, should not be too difficult, maybe i can try in the spare time.

@charlesportwoodii
Copy link
Collaborator

Thanks for providing this. I'm a lot behind but I'll take a look and give you my feedback.

Quick questions:

  1. Is the Bluefruit stuff locked to SD6.1 or can we stick with SD7? I don't mind rolling back if it adds functionality, but would like to know if it's possible. Same question for SDK 15.3 vs 16.
  2. I see you added some Nordic SDK files. Are those custom, or taken straight from the 15.3 SDK? For licensing reasons we can include those indirectly rather than committing them. I'm mainly interested if they are patched in any way.
  3. Do you have a test Bluefruit sketch you can provide with this for testing purposes?

Thanks!

@gb88
Copy link
Author

gb88 commented Dec 13, 2023

  1. I thought that was better to work with 15.3 and SD6.1.1 to be fully compliant with the Adafruit Core and the compatibility matrix of Nordic, but i not exclude that could work with the newer version.
  2. Almost all can be patched all, i fix a bug that even if you don't want the signature, is required to sign the dfu packet with some signature even with a wrong key, i'm not very able to create patch, i've also add some include to define some sdk configuration, maybe can be moved to the docker enviroment.
  3. I've tested the Bluefruit with a custom board with nrf52840 i've use a custom services and characteristichs and the DIS service, i can create some test, but theoretically all the examples should work.

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.

3 participants