[riot-notifications] [RIOT-OS/RIOT] boards/common/nrf52: add openocd support for 'nordic_softdevice_ble' (#11470)

Alexandre Abadie notifications at github.com
Wed Sep 11 08:25:22 CEST 2019


aabadie commented on this pull request.

With all related PRs merged, I retested flashing with OpenOCD and it now works like a charm.

I have one question regarding Murdock integration though.


> @@ -41,8 +41,24 @@ else ifeq (openocd,$(PROGRAMMER))
   # for nrf52dk and nrf52840dk boards. To use OpenOCD with these a version
   # build from source (master > 2018, August the 13rd) is required.
   ifneq (,$(filter nordic_softdevice_ble,$(USEPKG)))
-    # openocd doesn't work (yet) with softdevice
-    $(error Cannot use OpenOCD with nordic_softdevice module)
+    LINKER_SCRIPT ?= $(RIOTCPU)/$(CPU)/ldscripts/$(CPU_MODEL)_sd.ld
+
+    export OPENOCD_PRE_FLASH_CMDS += -c 'flash write_image erase "$(BINDIR)/softdevice.hex"'
+    export OPENOCD_PRE_VERIFY_CMDS += -c 'verify_image "$(BINDIR)/softdevice.hex"'
+
+    # Cannot use the 'ELFFILE' here
+    # The segments are 0x10000 aligned so padding bytes are copied and verified
+    #
+    # Using 'LINKFLAGS += -Wl,--nmagic' prevents it but would require
+    # dedicated testing/review
+    FLASHFILE ?= $(HEXFILE)
+
+    # murdock: softdevice.hex file is used for flashing

Why here and not in the package Makefile ?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11470#pullrequestreview-286575779
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190910/6b0fa579/attachment.htm>


More information about the notifications mailing list