app: Received a fault after doing nRF52832 DFU

RE: app: Received a fault after doing nRF52832 DFU

snowuyl — Fri, 05 Aug 2022 09:31:10 GMT

Thanks for your suggestion! I have merged the code and app doesn't received a fault.

RE: app: Received a fault after doing nRF52832 DFU

Sigurd — Fri, 05 Aug 2022 07:36:56 GMT

[quote user="snowuyl"]Do I need to upgrade to SDK 17.0.0 for this issue or just merge code you list?[/quote]

Just merge the code, and see if it fixes the problem you are having.

RE: app: Received a fault after doing nRF52832 DFU

snowuyl — Fri, 05 Aug 2022 00:27:41 GMT

Do I need to upgrade to SDK 17.0.0 for this issue or just merge code you list?

RE: app: Received a fault after doing nRF52832 DFU

snowuyl — Fri, 05 Aug 2022 00:25:51 GMT

SDK v16.0.0

RE: app: Received a fault after doing nRF52832 DFU

Sigurd — Thu, 04 Aug 2022 14:50:34 GMT

Hi,

What SDK version is this?

Might be that you are encountering a bug which which we fixed in nRF5 SDK v17. It was quite briefly mentioned in the release notes:

- Fixed a false positive assert that could sometimes happen after a reset
during an upgrade.

But you can see the fix implemented here in SDK v17.0.0:

diff --git a/../../../../nRF5_SDK_15.2.0_9412b96/components/libraries/bootloader/nrf_bootloader_fw_activation.c b/nrf_bootloader_fw_activation.c
index bf7ac87..5892b98 100644
--- a/../../../../nRF5_SDK_15.2.0_9412b96/components/libraries/bootloader/nrf_bootloader_fw_activation.c
+++ b/nrf_bootloader_fw_activation.c
@@ -1,5 +1,5 @@
 /**
- * Copyright (c) 2016 - 2018, Nordic Semiconductor ASA
+ * Copyright (c) 2016 - 2020, Nordic Semiconductor ASA
  *
  * All rights reserved.
  *
@@ -72,13 +72,16 @@ static uint32_t image_copy(uint32_t dst_addr,
 {
     if (src_addr == dst_addr)
     {
-        NRF_LOG_DEBUG("No copy needed src_addr: 0x%x, dst_addr: 0x%x", src_addr, dst_addr);
+        NRF_LOG_DEBUG("No copy needed");
         return NRF_SUCCESS;
     }
 
     ASSERT(src_addr >= dst_addr);
     ASSERT(progress_update_step > 0);
-    ASSERT((dst_addr % CODE_PAGE_SIZE) == 0);
+    if (size != 0)
+    {
+        ASSERT((dst_addr % CODE_PAGE_SIZE) == 0);
+    }
 
     uint32_t max_safe_progress_upd_step = (src_addr - dst_addr)/CODE_PAGE_SIZE;
     ASSERT(max_safe_progress_upd_step > 0);

As you can see from the 'diff', the solution is to ignore the assert when size is 0 (size should always be '0' after an unexpected reset). The copy routine will then continue the copy routine and complete the FW activation.

RE: app: Received a fault after doing nRF52832 DFU

snowuyl — Thu, 04 Aug 2022 07:49:16 GMT

RE: app: Received a fault after doing nRF52832 DFU

snowuyl — Thu, 04 Aug 2022 07:48:52 GMT

devzone.nordicsemi.com/.../2620.app_5F00_dfu_5F00_package.zip devzone.nordicsemi.com/.../3301.secure_5F00_bootloader.rar