nrf52840_pca10059 executables do not work without mcuboot

[markrages]

Description

The LED blink example build for nrf52840_pca10059 does not work when loaded with nRF Connect.

To Reproduce
Steps to reproduce the behavior:

1. west build -d build/nRF52/blinky -s zephyrproject/zephyr/samples/basic/blinky -b nrf52840_pca10059 -- -DCONFIG_FLASH_BASE_ADDRESS=0x1000 (the define is necessitated by config BOARD_HAS_NRF5_BOOTLOADER not honored for nrf52840_pca10059 #14943)
2. Load executable with nrfutil:

nrfutil pkg generate --application $BUILDDIR/zephyr/zephyr.hex --hw-version 52 --sd-req 0 --application-version=0  t.zip

nrfutil dfu usb-serial --package=t.zip --serial-number=EC4938BFC6DB --connect-delay=0

3. The LED is not blinking on the target board.

Expected behavior
The LED blinks.

Actual behavior
The LED doesn't blink.

Impact
Prevents simple use of nRF-Connect app, or distribution of images.

Workaround
The LED can blinks if I use this workaround, suggested by @mike-scott on IRC.

1. west build -d build/nRF52/mcuboot -s mcuboot/boot/zephyr -b nrf52840_pca10059 -- -DCONFIG_FLASH_LOAD_OFFSET=0x1000 -DCONFIG_FLASH_LOAD_SIZE=0xf000
2. west build -d build/nRF52/blinky -s zephyrproject/zephyr/samples/basic/blinky -b nrf52840_pca10059 -- -DCONFIG_FLASH_LOAD_OFFSET=0x10000 -DCONFIG_FLASH_LOAD_SIZE=0x5E000
3. west sign -d build/nRF52/blinky -t imgtool -- --key mcuboot/root-rsa-2048.pem
4. load build/nRF52/mcuboot/zephyr/zephyr.hex and zephyr.signed.hex simultaneously with nRF Connect app.

Environment

• OS: Ubuntu 18.04
• Toolchain: zephyr-sdk-0.10.0
• Commit: c3cf543

[lemrey]

This issue does not seem valid, since the "nRF Connect App" is not documented as a supported DFU method for this board. The supported DFU method is via nrfutil, and the steps to reproduce are not what is indicated in the documentation, either.

[markrages]

@lemrey edited to show nrfutil use. The file is linked incorrectly, so the programmer script makes no difference.

[lemrey]

This is issue is still confusing, since you apply a workaround and argue that the result is not what expected; you are not supposed to manually override the FLASH_LOAD_OFFSET.

I think all the issues you have opened are rooted in one, real issue: #14943,
for which I have opened a PR: #14978.

Please check if that fixes it.

[markrages]

@lemrey I agree, my three issues all stem from #14943 (and #14875.)

I will close them all when #14978 is merged.

[carlescufi]

Closing, #14978 is merged