Skip to content

Commit fd612d5

Browse files
committed
sysbuild: Fix compliance issues with Kconfig
Fixes some issues that the sysbuild Kconfig check has identified Signed-off-by: Jamie McCrae <[email protected]>
1 parent f3f0249 commit fd612d5

File tree

4 files changed

+10
-18
lines changed

4 files changed

+10
-18
lines changed

sysbuild/Kconfig.appcore

+4-12
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ config APPCORE_REMOTE_BOARD_TARGET_CPUCLUSTER
1212

1313
config APPCORE_REMOTE_DOMAIN
1414
string
15-
default "CPUAPP" if APPCORE_REMOTE_BOARD_NAME != ""
15+
default "CPUAPP" if APPCORE_REMOTE_BOARD_TARGET_CPUCLUSTER != ""
1616

1717
menu "Application core configuration"
1818
depends on SUPPORT_APPCORE
@@ -33,9 +33,9 @@ config NRF_DEFAULT_REMOTE_SHELL
3333

3434
choice APPCORE
3535
prompt "Appcore image"
36-
default APPCORE_REMOTE_SHELL if NRF_DEFAULT_REMOTE_SHELL && APPCORE_REMOTE_BOARD_NAME != ""
37-
default APPCORE_EMPTY if NRF_DEFAULT_EMPTY && APPCORE_REMOTE_BOARD_NAME != ""
38-
depends on SUPPORT_APPCORE && !EXTERNAL_CONFIGURED_APPCORE
36+
default APPCORE_REMOTE_SHELL if NRF_DEFAULT_REMOTE_SHELL
37+
default APPCORE_EMPTY if NRF_DEFAULT_EMPTY
38+
depends on SUPPORT_APPCORE
3939

4040
config APPCORE_NONE
4141
bool "None"
@@ -72,14 +72,6 @@ config APPCORE_IMAGE_PATH
7272
help
7373
Source directory of appcore image.
7474

75-
config APPCORE_IMAGE_BOARD
76-
string
77-
default APPCORE_REMOTE_BOARD_NAME
78-
depends on APPCORE_EMPTY
79-
help
80-
Remote board when building this sample.
81-
If not set, the board given to sysbuild is used.
82-
8375
config APPCORE_IMAGE_DOMAIN
8476
string
8577
default APPCORE_REMOTE_DOMAIN

sysbuild/Kconfig.netcore

+4-1
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,13 @@
22
#
33
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
44

5+
# Zephyr Kconfig for some samples
6+
config NET_CORE_IMAGE_HCI_IPC
7+
bool
8+
59
config EXTERNAL_CONFIGURED_NETCORE
610
bool
711
default y
8-
select NETCORE_REMOTE_NRF5340_CPUNET if NET_CORE_BOARD = "nrf5340dk/nrf5340/cpunet"
912
depends on NET_CORE_IMAGE_HCI_IPC # Zephyr Kconfig enabling HCI IPC on netcore
1013

1114
config SUPPORT_NETCORE

sysbuild/Kconfig.secureboot

+1-4
Original file line numberDiff line numberDiff line change
@@ -278,11 +278,9 @@ choice SECURE_BOOT_SIGNING
278278

279279
config SECURE_BOOT_SIGNING_PYTHON
280280
bool "Sign with Python ecdsa library"
281-
select SECURE_BOOT_PRIVATE_KEY_PROVIDED
282281

283282
config SECURE_BOOT_SIGNING_OPENSSL
284283
bool "Sign with openssl command line tool"
285-
select SECURE_BOOT_PRIVATE_KEY_PROVIDED
286284

287285
config SECURE_BOOT_SIGNING_CUSTOM
288286
bool "Sign with custom command"
@@ -343,7 +341,7 @@ config SECURE_BOOT_PUBLIC_KEY_FILES
343341

344342
config SUPPORT_SECURE_BOOT_BOOTCONF_LOCK_WRITES
345343
bool
346-
depends on !FPROTECT_ALLOW_COMBINED_REGIONS
344+
depends on !MCUBOOT_FPROTECT_ALLOW_COMBINED_REGIONS
347345
default y if SOC_NRF54L15_CPUAPP
348346

349347
config SECURE_BOOT_BOOTCONF_LOCK_WRITES
@@ -354,7 +352,6 @@ config SECURE_BOOT_BOOTCONF_LOCK_WRITES
354352

355353
config SECURE_BOOT_DEBUG_SIGNATURE_PUBLIC_KEY_LAST
356354
bool "[DEBUG] Place signing public key last"
357-
default n
358355
help
359356
Place the public key used for signing last in the list instead of first. This is meant to
360357
be used for testing looping through the public keys.

sysbuild/Kconfig.xip

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ config SUPPORT_QSPI_XIP
88
default y if SOC_NRF5340_CPUAPP
99
default y if SOC_SERIES_NRF91X
1010

11-
menuconfig QSPI_XIP_SPLIT_IMAGE
11+
config QSPI_XIP_SPLIT_IMAGE
1212
bool "QSPI XIP split image"
1313
depends on SUPPORT_QSPI_XIP && BOOTLOADER_MCUBOOT && PARTITION_MANAGER
1414
depends on PM_EXTERNAL_FLASH_MCUBOOT_SECONDARY

0 commit comments

Comments
 (0)