Skip to content

Commit 30e7326

Browse files
committed
[nrf noup] boot: Use NCS_ prefix for sdk-nrf specific Kconfigs
nrf-squash! [nrf noup] boot: Add retry for image verification Use NCS_ prefix, for Kconfigs added in noups, to avoid collisions. The NRF_ prefix will remain reserved for Kconfigs related to products or sdk-nrf subsystems. Signed-off-by: Dominik Ermel <[email protected]>
1 parent 8b2d04c commit 30e7326

File tree

2 files changed

+22
-22
lines changed

2 files changed

+22
-22
lines changed

boot/bootutil/src/loader.c

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,9 @@
5757

5858
#ifdef __ZEPHYR__
5959
#include <zephyr/sys/reboot.h>
60-
#if defined(CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS)
60+
#if defined(CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS)
6161
#include <zephyr/kernel.h>
62-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS */
62+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS */
6363
#endif
6464

6565
#if defined(CONFIG_SOC_NRF5340_CPUAPP) && defined(PM_CPUNET_B0N_ADDRESS) && defined(CONFIG_PCD_APP)
@@ -916,10 +916,10 @@ boot_image_check(struct boot_loader_state *state, struct image_header *hdr,
916916
}
917917
#endif
918918

919-
for (int i = 1; i <= CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT; i++ ) {
920-
#if CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
921-
BOOT_LOG_DBG("Image validation attempt %d/%d", i, CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT);
922-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
919+
for (int i = 1; i <= CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT; i++ ) {
920+
#if CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
921+
BOOT_LOG_DBG("Image validation attempt %d/%d", i, CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT);
922+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
923923

924924
#if defined(MCUBOOT_SWAP_USING_OFFSET) && defined(MCUBOOT_SERIAL_RECOVERY)
925925
FIH_CALL(bootutil_img_validate, fih_rc, state, hdr, fap, tmpbuf, BOOT_TMPBUF_SZ,
@@ -930,25 +930,25 @@ boot_image_check(struct boot_loader_state *state, struct image_header *hdr,
930930
#endif
931931

932932
if (FIH_EQ(fih_rc, FIH_SUCCESS)) {
933-
#if CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
934-
BOOT_LOG_DBG("Image validation attempt %d/%d success", i, CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT);
935-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
933+
#if CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
934+
BOOT_LOG_DBG("Image validation attempt %d/%d success", i, CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT);
935+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
936936
break;
937937
} else {
938-
#if CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
938+
#if CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
939939
BOOT_LOG_WRN("Image validation attempt %d/%d failure: %d",
940940
i,
941-
CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT, fih_rc);
942-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
941+
CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT, fih_rc);
942+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
943943

944-
if (i < CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT) {
945-
#if defined(CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS)
946-
#if CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
944+
if (i < CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT) {
945+
#if defined(CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS)
946+
#if CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
947947
BOOT_LOG_DBG("Waiting %d ms before next attempt",
948-
CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS);
949-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
950-
k_busy_wait(CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS * 1000);
951-
#endif /* CONFIG_NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS */
948+
CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS);
949+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1 */
950+
k_busy_wait(CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS * 1000);
951+
#endif /* CONFIG_NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS */
952952
}
953953
}
954954
}

boot/zephyr/Kconfig

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1309,7 +1309,7 @@ config MCUBOOT_VERIFY_IMG_ADDRESS
13091309
also be useful when BOOT_DIRECT_XIP is enabled, to ensure that the image
13101310
linked at the correct address is loaded.
13111311

1312-
config NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT
1312+
config NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT
13131313
int "Number of image validation attempts"
13141314
default 1
13151315
help
@@ -1318,9 +1318,9 @@ config NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT
13181318
This can prevent erasing an image when initial validation fails.
13191319
Wait time is controlled by MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS.
13201320

1321-
config NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS
1321+
config NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_WAIT_MS
13221322
int "Time between image validation attempts"
1323-
depends on NRF_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
1323+
depends on NCS_MCUBOOT_IMG_VALIDATE_ATTEMPT_COUNT > 1
13241324
default 5000
13251325
help
13261326
Time between image validation attempts, in milliseconds.

0 commit comments

Comments
 (0)